net: phy: marvell: #defines for copper and fibre pages
authorAndrew Lunn <andrew@lunn.ch>
Thu, 25 May 2017 19:42:05 +0000 (21:42 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 May 2017 18:44:49 +0000 (14:44 -0400)
Replace magic numbers for PHY pages with symbolic names.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/marvell.c

index 88cd97b..bb06702 100644 (file)
@@ -650,7 +650,7 @@ static int m88e1116r_config_init(struct phy_device *phydev)
 
        mdelay(500);
 
-       err = marvell_set_page(phydev, 0);
+       err = marvell_set_page(phydev, MII_M1111_COPPER);
        if (err < 0)
                return err;
 
@@ -662,7 +662,7 @@ static int m88e1116r_config_init(struct phy_device *phydev)
        if (err < 0)
                return err;
 
-       err = marvell_set_page(phydev, 2);
+       err = marvell_set_page(phydev, MII_88E1121_PHY_MSCR_PAGE);
        if (err < 0)
                return err;
        temp = phy_read(phydev, MII_M1116R_CONTROL_REG_MAC);
@@ -671,7 +671,7 @@ static int m88e1116r_config_init(struct phy_device *phydev)
        err = phy_write(phydev, MII_M1116R_CONTROL_REG_MAC, temp);
        if (err < 0)
                return err;
-       err = marvell_set_page(phydev, 0);
+       err = marvell_set_page(phydev, MII_M1111_COPPER);
        if (err < 0)
                return err;
 
@@ -892,7 +892,7 @@ static int m88e1510_config_init(struct phy_device *phydev)
                        return err;
 
                /* Reset page selection */
-               err = marvell_set_page(phydev, 0);
+               err = marvell_set_page(phydev, MII_M1111_COPPER);
                if (err < 0)
                        return err;
        }
@@ -922,7 +922,7 @@ static int m88e1118_config_init(struct phy_device *phydev)
        int err;
 
        /* Change address */
-       err = marvell_set_page(phydev, 2);
+       err = marvell_set_page(phydev, MII_88E1121_PHY_MSCR_PAGE);
        if (err < 0)
                return err;
 
@@ -932,7 +932,7 @@ static int m88e1118_config_init(struct phy_device *phydev)
                return err;
 
        /* Change address */
-       err = marvell_set_page(phydev, 3);
+       err = marvell_set_page(phydev, MII_88E1318S_PHY_LED_PAGE);
        if (err < 0)
                return err;
 
@@ -949,7 +949,7 @@ static int m88e1118_config_init(struct phy_device *phydev)
                return err;
 
        /* Reset address */
-       err = marvell_set_page(phydev, 0);
+       err = marvell_set_page(phydev, MII_M1111_COPPER);
        if (err < 0)
                return err;
 
@@ -961,7 +961,7 @@ static int m88e1149_config_init(struct phy_device *phydev)
        int err;
 
        /* Change address */
-       err = marvell_set_page(phydev, 2);
+       err = marvell_set_page(phydev, MII_88E1121_PHY_MSCR_PAGE);
        if (err < 0)
                return err;
 
@@ -975,7 +975,7 @@ static int m88e1149_config_init(struct phy_device *phydev)
                return err;
 
        /* Reset address */
-       err = marvell_set_page(phydev, 0);
+       err = marvell_set_page(phydev, MII_M1111_COPPER);
        if (err < 0)
                return err;
 
@@ -1409,7 +1409,7 @@ static void m88e1318_get_wol(struct phy_device *phydev,
            MII_88E1318S_PHY_WOL_CTRL_MAGIC_PACKET_MATCH_ENABLE)
                wol->wolopts |= WAKE_MAGIC;
 
-       if (marvell_set_page(phydev, 0x00) < 0)
+       if (marvell_set_page(phydev, MII_M1111_COPPER) < 0)
                return;
 }
 
@@ -1422,7 +1422,7 @@ static int m88e1318_set_wol(struct phy_device *phydev,
 
        if (wol->wolopts & WAKE_MAGIC) {
                /* Explicitly switch to page 0x00, just to be sure */
-               err = marvell_set_page(phydev, 0x00);
+               err = marvell_set_page(phydev, MII_M1111_COPPER);
                if (err < 0)
                        return err;