ASoC: fsl_spdif: fix return value check in fsl_spdif_probe()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Fri, 25 Oct 2013 09:29:25 +0000 (17:29 +0800)
committerMark Brown <broonie@linaro.org>
Sat, 26 Oct 2013 20:32:29 +0000 (21:32 +0100)
In case of error, the function platform_get_resource() returns NULL
pointer not ERR_PTR(). The IS_ERR() test in the return value check
should be replaced with NULL test.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Mark Brown <broonie@linaro.org>
sound/soc/fsl/fsl_spdif.c

index c0fea02114e1e9b10c8f6b9afcfb61b1b5d901bc..e1bf5ef31bdd9fda6e6c166d7234edcedbd64e3c 100644 (file)
@@ -1107,9 +1107,9 @@ static int fsl_spdif_probe(struct platform_device *pdev)
 
        /* Get the addresses and IRQ */
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (IS_ERR(res)) {
+       if (!res) {
                dev_err(&pdev->dev, "could not determine device resources\n");
-               return PTR_ERR(res);
+               return -ENXIO;
        }
 
        regs = devm_ioremap_resource(&pdev->dev, res);