powerpc/4xx: Fix return value check in hsta_msi_probe()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Sun, 20 Jul 2014 07:20:59 +0000 (15:20 +0800)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 29 Dec 2014 04:45:43 +0000 (15:45 +1100)
In case of error, the function ioremap() returns NULL
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: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/sysdev/ppc4xx_hsta_msi.c

index ed9970ff8d94abed1f4c76d4069b789248af3c6d..f366d2d4c0790653fd3a7eec3fcca86e864e2451 100644 (file)
@@ -145,7 +145,7 @@ static int hsta_msi_probe(struct platform_device *pdev)
        ppc4xx_hsta_msi.address = mem->start;
        ppc4xx_hsta_msi.data = ioremap(mem->start, resource_size(mem));
        ppc4xx_hsta_msi.irq_count = irq_count;
-       if (IS_ERR(ppc4xx_hsta_msi.data)) {
+       if (!ppc4xx_hsta_msi.data) {
                dev_err(dev, "Unable to map memory\n");
                return -ENOMEM;
        }