ASoC: dpcm: Avoid putting stream state to STOP when FE stream is paused
authorPatrick Lai <plai@codeaurora.org>
Sun, 1 Jan 2017 06:44:39 +0000 (22:44 -0800)
committerSasha Levin <alexander.levin@verizon.com>
Sun, 10 Sep 2017 20:35:58 +0000 (16:35 -0400)
[ Upstream commit 9f169b9f52a4afccdab7a7d2311b0c53a78a1e6b ]

When multiple front-ends are using the same back-end, putting state of a
front-end to STOP state upon receiving pause command will result in backend
stream getting released by DPCM framework unintentionally. In order to
avoid backend to be released when another active front-end stream is
present, put the stream state to PAUSED state instead of STOP state.

Signed-off-by: Patrick Lai <plai@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
sound/soc/soc-pcm.c

index 52fe7eb2dea1fb02042b9bcb70c4117b7209ae09..418c214a0c8aec388a60df3bbe040739b63aa3a5 100644 (file)
@@ -1991,9 +1991,11 @@ static int dpcm_fe_dai_do_trigger(struct snd_pcm_substream *substream, int cmd)
                break;
        case SNDRV_PCM_TRIGGER_STOP:
        case SNDRV_PCM_TRIGGER_SUSPEND:
-       case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
                fe->dpcm[stream].state = SND_SOC_DPCM_STATE_STOP;
                break;
+       case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
+               fe->dpcm[stream].state = SND_SOC_DPCM_STATE_PAUSED;
+               break;
        }
 
 out: