netdev: pasemi: fix return value check in pasemi_mac_phy_init()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Wed, 26 Sep 2012 19:51:58 +0000 (19:51 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 27 Sep 2012 17:20:50 +0000 (13:20 -0400)
In case of error, the function of_phy_connect() returns NULL
pointer not ERR_PTR(). The IS_ERR() test in the return value
check should be replaced with NULL test.

dpatch engine is used to auto generate this patch.
(https://github.com/weiyj/dpatch)

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/pasemi/pasemi_mac.c

index e559dfa..6fa74d5 100644 (file)
@@ -1101,9 +1101,9 @@ static int pasemi_mac_phy_init(struct net_device *dev)
        phydev = of_phy_connect(dev, phy_dn, &pasemi_adjust_link, 0,
                                PHY_INTERFACE_MODE_SGMII);
 
-       if (IS_ERR(phydev)) {
+       if (!phydev) {
                printk(KERN_ERR "%s: Could not attach to phy\n", dev->name);
-               return PTR_ERR(phydev);
+               return -ENODEV;
        }
 
        mac->phydev = phydev;