net/phy: micrel: configure intterupts after autoneg workaround
authorZach Brown <zach.brown@ni.com>
Tue, 20 Jun 2017 17:48:11 +0000 (12:48 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 22 Jun 2017 15:05:16 +0000 (11:05 -0400)
The commit ("net/phy: micrel: Add workaround for bad autoneg") fixes an
autoneg failure case by resetting the hardware. This turns off
intterupts. Things will work themselves out if the phy polls, as it will
figure out it's state during a poll. However if the phy uses only
intterupts, the phy will stall, since interrupts are off. This patch
fixes the issue by calling config_intr after resetting the phy.

Fixes: d2fd719bcb0e ("net/phy: micrel: Add workaround for bad autoneg ")
Signed-off-by: Zach Brown <zach.brown@ni.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/micrel.c

index b9252b8d81ffb720272ca5f0b25910c021eb28a3..8b2038844ba96a4a86c5b42aca4fa59fe96ecca2 100644 (file)
@@ -619,6 +619,8 @@ static int ksz9031_read_status(struct phy_device *phydev)
        if ((regval & 0xFF) == 0xFF) {
                phy_init_hw(phydev);
                phydev->link = 0;
+               if (phydev->drv->config_intr && phy_interrupt_is_valid(phydev))
+                       phydev->drv->config_intr(phydev);
        }
 
        return 0;