From: Wei Yongjun Date: Fri, 25 Oct 2013 09:30:24 +0000 (+0800) Subject: powerpc/mv643xx_eth: fix return check in mv64x60_eth_register_shared_pdev() X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=dde8e3ee6b37fa33690604302bebfe7b576c3802;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git powerpc/mv643xx_eth: fix return check in mv64x60_eth_register_shared_pdev() In case of error, the function platform_device_register_simple() returns RR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun Signed-off-by: Scott Wood --- diff --git a/arch/powerpc/sysdev/mv64x60_dev.c b/arch/powerpc/sysdev/mv64x60_dev.c index 4a25c26f0bf4..a3a8fad8537d 100644 --- a/arch/powerpc/sysdev/mv64x60_dev.c +++ b/arch/powerpc/sysdev/mv64x60_dev.c @@ -228,7 +228,7 @@ static struct platform_device * __init mv64x60_eth_register_shared_pdev( if (id == 0) { pdev = platform_device_register_simple("orion-mdio", -1, &r[1], 1); - if (!pdev) + if (IS_ERR(pdev)) return pdev; }