ASoC: fsl_aud2htx: Add error handler for pm_runtime_enable

TE-N-ShengjiuWang · broonie · commit b1cd3fd42db7 · 2022-08-25T14:17:31.000+01:00
Call pm_runtime_disable() when error happens in probe() Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> Link: https://lore.kernel.org/r/1661430460-5234-2-git-send-email-shengjiu.wang@nxp.com Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/sound/soc/fsl/fsl_aud2htx.c b/sound/soc/fsl/fsl_aud2htx.c
@@ -241,6 +241,7 @@ static int fsl_aud2htx_probe(struct platform_device *pdev)
 	ret = devm_snd_dmaengine_pcm_register(&pdev->dev, NULL, 0);
 	if (ret) {
 		dev_err(&pdev->dev, "failed to pcm register\n");
+		pm_runtime_disable(&pdev->dev);
 		return ret;
 	}
 
@@ -249,6 +250,7 @@ static int fsl_aud2htx_probe(struct platform_device *pdev)
 					      &fsl_aud2htx_dai, 1);
 	if (ret) {
 		dev_err(&pdev->dev, "failed to register ASoC DAI\n");
+		pm_runtime_disable(&pdev->dev);
 		return ret;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -241,6 +241,7 @@ static int fsl_aud2htx_probe(struct platform_device *pdev)`
`241`	`241`	`ret = devm_snd_dmaengine_pcm_register(&pdev->dev, NULL, 0);`
`242`	`242`	`if (ret) {`
`243`	`243`	`dev_err(&pdev->dev, "failed to pcm register\n");`
	`244`	`+ pm_runtime_disable(&pdev->dev);`
`244`	`245`	`return ret;`
`245`	`246`	`}`
`246`	`247`
`@@ -249,6 +250,7 @@ static int fsl_aud2htx_probe(struct platform_device *pdev)`
`249`	`250`	`&fsl_aud2htx_dai, 1);`
`250`	`251`	`if (ret) {`
`251`	`252`	`dev_err(&pdev->dev, "failed to register ASoC DAI\n");`
	`253`	`+ pm_runtime_disable(&pdev->dev);`
`252`	`254`	`return ret;`
`253`	`255`	`}`
`254`	`256`