Skip to content

Commit 25aa058

Browse files
morimotobroonie
authored andcommitted
ASoC: renesas: msiof: tidyup DMAC stop timing
Current DMAC is stopped before HW stop, but it might be cause of sync error. Stop HW first. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Tested-by: Yusuke Goda <yusuke.goda.sx@renesas.com> Link: https://patch.msgid.link/878qi3yuu0.wl-kuninori.morimoto.gx@renesas.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent ab77fa5 commit 25aa058

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

sound/soc/renesas/rcar/msiof.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -238,16 +238,16 @@ static int msiof_hw_stop(struct snd_soc_component *component,
238238
val = SIIER_RDREQE | SIIER_RDMAE | SISTR_ERR_RX;
239239
msiof_update(priv, SIIER, val, 0);
240240

241-
/* Stop DMAC */
242-
snd_dmaengine_pcm_trigger(substream, cmd);
243-
244241
/* SICTR */
245242
if (is_play)
246243
val = SICTR_TXE;
247244
else
248245
val = SICTR_RXE;
249246
msiof_update_and_wait(priv, SICTR, val, 0, 0);
250247

248+
/* Stop DMAC */
249+
snd_dmaengine_pcm_trigger(substream, cmd);
250+
251251
/* indicate error status if exist */
252252
if (priv->err_syc[substream->stream] ||
253253
priv->err_ovf[substream->stream] ||

0 commit comments

Comments
 (0)