]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
net: macb: fix phy interrupt parsing
authorAlexandre Belloni <alexandre.belloni@free-electrons.com>
Wed, 26 Apr 2017 10:06:28 +0000 (12:06 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 1 May 2017 02:21:49 +0000 (22:21 -0400)
Since 83a77e9ec415, the phydev irq is explicitly set to PHY_POLL when
there is no pdata. It doesn't work on DT enabled platforms because the
phydev irq is already set by libphy before.

Fixes: 83a77e9ec415 ("net: macb: Added PCI wrapper for Platform Driver.")
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cadence/macb.c

index 5cbd1e7a926aa2f1eaf3a2f73aa75fe231b219f2..91f7492623d3f1d614afeb7f525ecfb491e5965e 100644 (file)
@@ -432,15 +432,17 @@ static int macb_mii_probe(struct net_device *dev)
        }
 
        pdata = dev_get_platdata(&bp->pdev->dev);
-       if (pdata && gpio_is_valid(pdata->phy_irq_pin)) {
-               ret = devm_gpio_request(&bp->pdev->dev, pdata->phy_irq_pin,
-                                       "phy int");
-               if (!ret) {
-                       phy_irq = gpio_to_irq(pdata->phy_irq_pin);
-                       phydev->irq = (phy_irq < 0) ? PHY_POLL : phy_irq;
+       if (pdata) {
+               if (gpio_is_valid(pdata->phy_irq_pin)) {
+                       ret = devm_gpio_request(&bp->pdev->dev,
+                                               pdata->phy_irq_pin, "phy int");
+                       if (!ret) {
+                               phy_irq = gpio_to_irq(pdata->phy_irq_pin);
+                               phydev->irq = (phy_irq < 0) ? PHY_POLL : phy_irq;
+                       }
+               } else {
+                       phydev->irq = PHY_POLL;
                }
-       } else {
-               phydev->irq = PHY_POLL;
        }
 
        /* attach the mac to the phy */