Unverified Commit cab45ab9 authored by Srinivas Kandagatla's avatar Srinivas Kandagatla Committed by Mark Brown
Browse files

ASoC: qcom: q6apm-dai: reset queue ptr on trigger stop 



Reset queue pointer on SNDRV_PCM_TRIGGER_STOP event to be inline
with resetting appl_ptr. Without this we will end up with a queue_ptr
out of sync and driver could try to send data that is not ready yet.

Fix this by resetting the queue_ptr.

Fixes: 3d4a4411 ("ASoC: q6apm-dai: schedule all available frames to avoid dsp under-runs")
Cc: Stable@vger.kernel.org
Signed-off-by: default avatarSrinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>
Link: https://patch.msgid.link/20260402081118.348071-6-srinivas.kandagatla@oss.qualcomm.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 69acc488

sound/soc/qcom/qdsp6/q6apm-dai.c

+1 −0
Original line number Diff line number Diff line
@@ -317,6 +317,7 @@ static int q6apm_dai_trigger(struct snd_soc_component *component, 
	case SNDRV_PCM_TRIGGER_STOP:

		/* TODO support be handled via SoftPause Module */

		prtd->state = Q6APM_STREAM_STOPPED;

		prtd->queue_ptr = 0;

		break;

	case SNDRV_PCM_TRIGGER_SUSPEND:

	case SNDRV_PCM_TRIGGER_PAUSE_PUSH:

sound/soc/qcom/qdsp6/q6apm.c

+2 −0
Original line number Diff line number Diff line
@@ -215,6 +215,8 @@ int q6apm_map_memory_regions(struct q6apm_graph *graph, unsigned int dir, phys_a 


	mutex_lock(&graph->lock);



	data->dsp_buf = 0;



	if (data->buf) {

		mutex_unlock(&graph->lock);

		return 0;