ASoC: qcom: q6apm-lpass-dai: close graphs before opening a new one
authorSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Thu, 23 Mar 2023 16:44:00 +0000 (16:44 +0000)
committerMark Brown <broonie@kernel.org>
Thu, 6 Apr 2023 13:45:11 +0000 (14:45 +0100)
On multiple prepare calls, its possible that the playback graphs are
not unloaded from the DSP, which can have some wierd side-effects,
one of them is that the data not consumed without any errors.

Fixes: c2ac3aec474d("ASoC: qcom: q6apm-lpass-dai: unprepare stream if its already prepared")
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Link: https://lore.kernel.org/r/20230323164403.6654-2-srinivas.kandagatla@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/qcom/qdsp6/q6apm-lpass-dais.c

index 23d23bc..420e8aa 100644 (file)
@@ -130,6 +130,9 @@ static int q6apm_lpass_dai_prepare(struct snd_pcm_substream *substream, struct s
        if (dai_data->is_port_started[dai->id]) {
                q6apm_graph_stop(dai_data->graph[dai->id]);
                dai_data->is_port_started[dai->id] = false;
+
+               if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK)
+                       q6apm_graph_close(dai_data->graph[dai->id]);
        }
 
        /**