tree 48833a217f4774c575e87c3240852c12a120571f
parent 27f41dfebf226c74691b4b30527459417b6510e8
author Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> 1601515968 +0900
committer Mark Brown <broonie@kernel.org> 1601580498 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFHBAABCgAxFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl92LdITHGJyb29uaWVA
 a2VybmVsLm9yZwAKCRAk1otyXVSH0PtXB/4qboeKq3yFqQFMce3LRoZL7a1rKzC1
 AxtE18S1/WCYGOKAmMvJYjzreImVsr0+czV3GKzJpk4i/wijCiX1693dqU+WeHOt
 z0iRp3cBeJE5lqoN3ldN20fJDmErlRhLRnEiAmN3jMD9D2g6yZZxbgrMmODxdjRs
 cX1/UbeRao3XTuKnjk/AtQVVAB/Y4U6yjuiPhdl8uoCaFYwW1DjzER26NZBodXEv
 D+O0ESuQFG6Fp1Ej2FnnxllrTAejbG3EO6DsK/2/0zbpuaN6e4yKesIjFKSoCZH0
 f4E8+RtEJOdmUGmyLX/wt5HdbrvpR0swOMWE0qkKjtU2hjWIthivUWkX
 =Q9DI
 -----END PGP SIGNATURE-----

ASoC: soc-pcm: ignore un-needed mutex_unlock() case on soc_pcm_open()

commit 140a4532cdb8c ("ASoC: soc-pcm: add soc_pcm_clean() and call it
from soc_pcm_open/close()") switch to call soc_pcm_clean() on
soc_pcm_open() when rollback case.

But, it uses "goto err" (A) *before* mutex_lock() (B) when error of
snd_soc_pcm_component_pm_runtime_get().
The mutex_unlock() (C) is not needed in such case. This patch fix it.

	static int soc_pcm_open(...)
	{
		...
		ret = snd_soc_pcm_component_pm_runtime_get(rtd, substream);
		if (ret < 0)
(A)			goto err;

(B)		mutex_lock_nested(...);
		...
	err:
(C)		mutex_unlock(..);
		...
	}

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87k0waag44.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>