Commit aaaebc4d authored by Dan Carpenter's avatar Dan Carpenter Committed by Jiri Slaby

ASoC: imx-wm8962: Add a missing error check

commit 474ff0ae upstream.

My static checker complains that:

	sound/soc/fsl/imx-wm8962.c:196 imx_wm8962_probe() warn:
	we tested 'ret' before and it was 'false'

The intent was that we use "ret" to check imx_audmux_v2_configure_port().

Fixes: 8de2ae2a ('ASoC: fsl: add imx-wm8962 machine driver')
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Otherwise, Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Signed-off-by: default avatarJiri Slaby <jslaby@suse.cz>
parent b9eee50f
...@@ -192,7 +192,7 @@ static int imx_wm8962_probe(struct platform_device *pdev) ...@@ -192,7 +192,7 @@ static int imx_wm8962_probe(struct platform_device *pdev)
dev_err(&pdev->dev, "audmux internal port setup failed\n"); dev_err(&pdev->dev, "audmux internal port setup failed\n");
return ret; return ret;
} }
imx_audmux_v2_configure_port(ext_port, ret = imx_audmux_v2_configure_port(ext_port,
IMX_AUDMUX_V2_PTCR_SYN, IMX_AUDMUX_V2_PTCR_SYN,
IMX_AUDMUX_V2_PDCR_RXDSEL(int_port)); IMX_AUDMUX_V2_PDCR_RXDSEL(int_port));
if (ret) { if (ret) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment