a52: Ignore start request if we're already running

When trying to start a PCM that's already running, the result is EBADFD.
We might have implicitly started the buffer by filling it up,
so just ignore this request if we're already running.

Reported-by: Chow Loong Jin <hyperair@ubuntu.com>
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

diff --git a/a52/pcm_a52.c b/a52/pcm_a52.c
index b467ec8e5abb40ae69d6572e4af4365e91e59843..59e25cbd5f46419398decc96a1e1d2fdd9be9902 100644 (file)
--- a/a52/pcm_a52.c
+++ b/a52/pcm_a52.c
@@ -482,6 +482,12 @@ static int a52_start(snd_pcm_ioplug_t *io)
 {
        struct a52_ctx *rec = io->private_data;
 
+       /* When trying to start a PCM that's already running, the result is
+          EBADFD. We might have implicitly started the buffer by filling it
+          up, so just ignore this request if we're already running. */
+       if (snd_pcm_state(rec->slave) == SND_PCM_STATE_RUNNING)
+               return 0;
+
        return snd_pcm_start(rec->slave);
 }