[alsa-devel] [PATCH v2 1/4] ASoC: imx-sgtl5000: Do not enter the error path on success

Shawn Guo shawn.guo at linaro.org
Thu Apr 25 07:05:57 CEST 2013


On Wed, Apr 24, 2013 at 11:54:43AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam at freescale.com>
> 
> Return on success instead of entering the error path.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>

Acked-by: Shawn Guo <shawn.guo at linaro.org>

> ---
> Changes since v1:
> - Keep of_node_put in the success path.
> 
>  sound/soc/fsl/imx-sgtl5000.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/sound/soc/fsl/imx-sgtl5000.c b/sound/soc/fsl/imx-sgtl5000.c
> index 9584e78..5a6aaa3 100644
> --- a/sound/soc/fsl/imx-sgtl5000.c
> +++ b/sound/soc/fsl/imx-sgtl5000.c
> @@ -174,6 +174,11 @@ static int imx_sgtl5000_probe(struct platform_device *pdev)
>  	}
>  
>  	platform_set_drvdata(pdev, data);
> +	of_node_put(ssi_np);
> +	of_node_put(codec_np);
> +
> +	return 0;
> +
>  clk_fail:
>  	clk_put(data->codec_clk);
>  fail:
> -- 
> 1.7.9.5
> 



More information about the Alsa-devel mailing list