spi: atmel: fix return value check in atmel_spi_probe()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Mon, 21 Oct 2013 03:12:02 +0000 (11:12 +0800)
committerMark Brown <broonie@linaro.org>
Wed, 23 Oct 2013 09:38:46 +0000 (10:38 +0100)
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR().

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/spi/spi-atmel.c

index 118a938776bd9eec7a79372a915c3498f7610103..273db0beb2b88ffeb588307ce08b2cb40120c790 100644 (file)
@@ -1547,8 +1547,10 @@ static int atmel_spi_probe(struct platform_device *pdev)
 
        as->pdev = pdev;
        as->regs = devm_ioremap_resource(&pdev->dev, regs);
-       if (!as->regs)
+       if (IS_ERR(as->regs)) {
+               ret = PTR_ERR(as->regs);
                goto out_free_buffer;
+       }
        as->phybase = regs->start;
        as->irq = irq;
        as->clk = clk;