net: phy: dp83848: Reorganize code for readability and safety
authorAndrew F. Davis <afd@ti.com>
Sun, 7 Feb 2016 17:47:19 +0000 (11:47 -0600)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Feb 2016 16:52:02 +0000 (11:52 -0500)
Reorganize code by moving the desired interrupt mask definition
out of function. Also rearrange the enable/disable interrupt function
to prevent accidental over-writing of values in registers.

Signed-off-by: Andrew F. Davis <afd@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/dp83848.c

index d4686d5f6e038cdba0aa28b106ebe8263c36ae27..20d3b9dae44437dd0b5b564e93140a85de7c9402 100644 (file)
 #define DP83848_MISR_ED_INT_EN         BIT(6) /* Energy detect */
 #define DP83848_MISR_LQM_INT_EN                BIT(7) /* Link Quality Monitor */
 
+#define DP83848_INT_EN_MASK            \
+       (DP83848_MISR_ANC_INT_EN |      \
+        DP83848_MISR_DUP_INT_EN |      \
+        DP83848_MISR_SPD_INT_EN |      \
+        DP83848_MISR_LINK_INT_EN)
+
 static int dp83848_ack_interrupt(struct phy_device *phydev)
 {
        int err = phy_read(phydev, DP83848_MISR);
@@ -46,23 +52,24 @@ static int dp83848_ack_interrupt(struct phy_device *phydev)
 
 static int dp83848_config_intr(struct phy_device *phydev)
 {
-       int err;
+       int control, ret;
+
+       control = phy_read(phydev, DP83848_MICR);
+       if (control < 0)
+               return control;
 
        if (phydev->interrupts == PHY_INTERRUPT_ENABLED) {
-               err = phy_write(phydev, DP83848_MICR,
-                               DP83848_MICR_INT_OE |
-                               DP83848_MICR_INTEN);
-               if (err < 0)
-                       return err;
-
-               return phy_write(phydev, DP83848_MISR,
-                                DP83848_MISR_ANC_INT_EN |
-                                DP83848_MISR_DUP_INT_EN |
-                                DP83848_MISR_SPD_INT_EN |
-                                DP83848_MISR_LINK_INT_EN);
+               control |= DP83848_MICR_INT_OE;
+               control |= DP83848_MICR_INTEN;
+
+               ret = phy_write(phydev, DP83848_MISR, DP83848_INT_EN_MASK);
+               if (ret < 0)
+                       return ret;
+       } else {
+               control &= ~DP83848_MICR_INTEN;
        }
 
-       return phy_write(phydev, DP83848_MICR, 0x0);
+       return phy_write(phydev, DP83848_MICR, control);
 }
 
 static struct mdio_device_id __maybe_unused dp83848_tbl[] = {