Unverified Commit f30a4c31 authored Jan 24, 2018 by Kuninori Morimoto Committed by Mark Brown Jan 24, 2018

ASoC: soc-pcm: don't call flush_delayed_work() many times in soc_pcm_private_free()



commit f523aceb ("ASoC: add Component level pcm_new/pcm_free v2")
added component level pcm_new/pcm_free, but flush_delayed_work()
on soc_pcm_private_free() is called in for_each_rtdcom() loop.
It doesn't need to be called many times.
This patch moves it out of loop.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

parent 4fbd8d19

sound/soc/soc-pcm.c

+2 −3

Original line number	Diff line number	Diff line
		@@ -2831,10 +2831,9 @@ static void soc_pcm_private_free(struct snd_pcm *pcm)
		struct snd_soc_rtdcom_list *rtdcom;
		struct snd_soc_component *component;

		for_each_rtdcom(rtd, rtdcom) {
		/* need to sync the delayed work before releasing resources */

		flush_delayed_work(&rtd->delayed_work);
		for_each_rtdcom(rtd, rtdcom) {
		component = rtdcom->component;

		if (component->pcm_free)

Admin message