Commit 836b31fe authored Apr 23, 2014 by Kuninori Morimoto Committed by Mark Brown Apr 23, 2014

ASoC: rsnd: call rsnd_dai_pointer_update() from outside of lock



rsnd_soc_dai_trigger() will be called
after rsnd_dai_pointer_update() function
which is using rsnd_lock().
Thus, it should be called from outside of rsnd_lock().
Kernel will be hangup without this patch.
Special thanks to Kataoka-san

Reported-by: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@linaro.org>

parent 79861bbb

sound/soc/sh/rcar/core.c

+2 −3

Original line number	Diff line number	Diff line
		@@ -197,13 +197,12 @@ static void rsnd_dma_complete(void *data)
		* rsnd_dai_pointer_update() will be called twice,
		* ant it will breaks io->byte_pos
		*/

		rsnd_dai_pointer_update(io, io->byte_per_period);

		if (dma->submit_loop)
		rsnd_dma_continue(dma);

		rsnd_unlock(priv, flags);

		rsnd_dai_pointer_update(io, io->byte_per_period);
		}

		static void __rsnd_dma_start(struct rsnd_dma *dma)

Admin message