net: of_mdio: do not overwrite PHY interrupt configuration
authorFlorian Fainelli <f.fainelli@gmail.com>
Thu, 5 Dec 2013 22:52:12 +0000 (14:52 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 6 Dec 2013 19:57:21 +0000 (14:57 -0500)
If irq_of_parse_and_map fails to find an interrupt line for a given PHY,
we will force the PHY interrupt to be PHY_POLL, completely overriding
the previous value that the MDIO bus may have set for us (e.g:
PHY_IGNORE_INTERRUPT). In case of failure, just restore the previous
value.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/of/of_mdio.c

index f93ebca0fcb7e046f623680f9eee67f83bfc590d..14feffc36964eb30673ece08b59ff7d4dd3b3fc8 100644 (file)
@@ -27,7 +27,7 @@ static int of_mdiobus_register_phy(struct mii_bus *mdio, struct device_node *chi
 {
        struct phy_device *phy;
        bool is_c45;
-       int rc;
+       int rc, prev_irq;
 
        is_c45 = of_device_is_compatible(child,
                                         "ethernet-phy-ieee802.3-c45");
@@ -37,10 +37,11 @@ static int of_mdiobus_register_phy(struct mii_bus *mdio, struct device_node *chi
                return 1;
 
        if (mdio->irq) {
+               prev_irq = mdio->irq[addr];
                mdio->irq[addr] =
                        irq_of_parse_and_map(child, 0);
                if (!mdio->irq[addr])
-                       mdio->irq[addr] = PHY_POLL;
+                       mdio->irq[addr] = prev_irq;
        }
 
        /* Associate the OF node with the device structure so it