net: phy: improve phy_driver callback handle_interrupt

did_interrupt() clears the interrupt, therefore handle_interrupt() can
not check which event triggered the interrupt. To overcome this
constraint and allow more flexibility for customer interrupt handlers,
let's decouple handle_interrupt() from parts of the phylib interrupt
handling. Custom interrupt handlers now have to implement the
did_interrupt() functionality in handle_interrupt() if needed.

Fortunately we have just one custom interrupt handler so far (in the
mscc PHY driver), convert it to the changed API.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/phy/mscc/mscc_main.c b/drivers/net/phy/mscc/mscc_main.c
index cb4d65f81095f94c48ae54ba210a4d498489ee96..4727aba8e051259b8d2c6d9fed2e3f7675b9aa75 100644 (file)
--- a/drivers/net/phy/mscc/mscc_main.c
+++ b/drivers/net/phy/mscc/mscc_main.c
@@ -1429,11 +1429,18 @@ err:
        return ret;
 }
 
-static int vsc8584_handle_interrupt(struct phy_device *phydev)
+static irqreturn_t vsc8584_handle_interrupt(struct phy_device *phydev)
 {
+       int irq_status;
+
+       irq_status = phy_read(phydev, MII_VSC85XX_INT_STATUS);
+       if (irq_status < 0 || !(irq_status & MII_VSC85XX_INT_MASK_MASK))
+               return IRQ_NONE;
+
        vsc8584_handle_macsec_interrupt(phydev);
        phy_mac_interrupt(phydev);
-       return 0;
+
+       return IRQ_HANDLED;
 }
 
 static int vsc85xx_config_init(struct phy_device *phydev)

diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index 355bfdef48d2b1f6c276e51bab37d3ee995a213e..d71212a418f3f747d2f0d425ce571e6d225bc374 100644 (file)
--- a/drivers/net/phy/phy.c
+++ b/drivers/net/phy/phy.c
@@ -715,26 +715,24 @@ static int phy_disable_interrupts(struct phy_device *phydev)
 static irqreturn_t phy_interrupt(int irq, void *phy_dat)
 {
        struct phy_device *phydev = phy_dat;
+       struct phy_driver *drv = phydev->drv;
 
-       if (phydev->drv->did_interrupt && !phydev->drv->did_interrupt(phydev))
+       if (drv->handle_interrupt)
+               return drv->handle_interrupt(phydev);
+
+       if (drv->did_interrupt && !drv->did_interrupt(phydev))
                return IRQ_NONE;
 
-       if (phydev->drv->handle_interrupt) {
-               if (phydev->drv->handle_interrupt(phydev))
-                       goto phy_err;
-       } else {
-               /* reschedule state queue work to run as soon as possible */
-               phy_trigger_machine(phydev);
-       }
+       /* reschedule state queue work to run as soon as possible */
+       phy_trigger_machine(phydev);
 
        /* did_interrupt() may have cleared the interrupt already */
-       if (!phydev->drv->did_interrupt && phy_clear_interrupt(phydev))
-               goto phy_err;
-       return IRQ_HANDLED;
+       if (!drv->did_interrupt && phy_clear_interrupt(phydev)) {
+               phy_error(phydev);
+               return IRQ_NONE;
+       }
 
-phy_err:
-       phy_error(phydev);
-       return IRQ_NONE;
+       return IRQ_HANDLED;
 }
 
 /**

diff --git a/include/linux/phy.h b/include/linux/phy.h
index 6b872aed8ba62c8e6e6728d567f419de5b0a689b..cb5a2182ba6d6621461459a1867eb83e60ac87b8 100644 (file)
--- a/include/linux/phy.h
+++ b/include/linux/phy.h
@@ -23,6 +23,7 @@
 #include <linux/workqueue.h>
 #include <linux/mod_devicetable.h>
 #include <linux/u64_stats_sync.h>
+#include <linux/irqreturn.h>
 
 #include <linux/atomic.h>
 
@@ -568,7 +569,7 @@ struct phy_driver {
        int (*did_interrupt)(struct phy_device *phydev);
 
        /* Override default interrupt handling */
-       int (*handle_interrupt)(struct phy_device *phydev);
+       irqreturn_t (*handle_interrupt)(struct phy_device *phydev);
 
        /* Clears up any memory if needed */
        void (*remove)(struct phy_device *phydev);