[alsa-devel] [PATCH -next] ASoC: tegra20-ac97: add missing clk_disable_unprepare() on error path
From: Wei Yongjun yongjun_wei@trendmicro.com.cn
Add the missing clk_disable_unprepare() before return from tegra20_ac97_platform_probe() in the error handling case.
Signed-off-by: Wei Yongjun yongjun_wei@trendmicro.com.cn --- sound/soc/tegra/tegra20_ac97.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/sound/soc/tegra/tegra20_ac97.c b/sound/soc/tegra/tegra20_ac97.c index d8b98d7..cf5e1cf 100644 --- a/sound/soc/tegra/tegra20_ac97.c +++ b/sound/soc/tegra/tegra20_ac97.c @@ -393,7 +393,7 @@ static int tegra20_ac97_platform_probe(struct platform_device *pdev) ret = snd_soc_set_ac97_ops(&tegra20_ac97_ops); if (ret) { dev_err(&pdev->dev, "Failed to set AC'97 ops: %d\n", ret); - goto err_asoc_utils_fini; + goto err_clk_disable_unprepare; }
ret = snd_soc_register_component(&pdev->dev, &tegra20_ac97_component, @@ -401,7 +401,7 @@ static int tegra20_ac97_platform_probe(struct platform_device *pdev) if (ret) { dev_err(&pdev->dev, "Could not register DAI: %d\n", ret); ret = -ENOMEM; - goto err_asoc_utils_fini; + goto err_clk_disable_unprepare; }
ret = tegra_pcm_platform_register(&pdev->dev); @@ -417,6 +417,8 @@ static int tegra20_ac97_platform_probe(struct platform_device *pdev)
err_unregister_component: snd_soc_unregister_component(&pdev->dev); +err_clk_disable_unprepare: + clk_disable_unprepare(ac97->clk_ac97); err_asoc_utils_fini: tegra_asoc_utils_fini(&ac97->util_data); err_clk_put:
On 12/13/2013 10:29 PM, Wei Yongjun wrote:
From: Wei Yongjun yongjun_wei@trendmicro.com.cn
Add the missing clk_disable_unprepare() before return from tegra20_ac97_platform_probe() in the error handling case.
Reviewed-by: Stephen Warren swarren@nvidia.com
Mark, I did check that this patch doesn't conflict with any of the ASoC changes I have in the Tegra tree, so there should be no issue when you apply it. The same goes for the previous patch from Wei too, which fixes some return values from this same function.
On Sat, Dec 14, 2013 at 01:29:11PM +0800, Wei Yongjun wrote:
From: Wei Yongjun yongjun_wei@trendmicro.com.cn
Add the missing clk_disable_unprepare() before return from tegra20_ac97_platform_probe() in the error handling case.
Applied, thanks.
participants (3)
-
Mark Brown
-
Stephen Warren
-
Wei Yongjun