diff options
| author | David Daney <ddaney@caviumnetworks.com> | 2010-11-19 06:58:52 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-11-22 11:34:22 -0500 |
| commit | 27d916d680e7b324087a75d080f215e7c34a4e8f (patch) | |
| tree | fd8d18ea66aff66afd600f83bb3d2217002d2b32 | |
| parent | 84cf7029b63ca3eef823438a29ceb5ba139b883a (diff) | |
phylib: Use common page register definition for Marvell PHYs.
The definition of the Marvell PHY page register is not specific to
88E1121, so rename the macro to MII_MARVELL_PHY_PAGE, and use it
throughout.
Suggested-by: Cyril Chemparathy <cyril@ti.com>
Signed-off-by: David Daney <ddaney@caviumnetworks.com>
Cc: Cyril Chemparathy <cyril@ti.com>
Cc: Arnaud Patard <arnaud.patard@rtp-net.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/phy/marvell.c | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c index f0bd1a1aba3a..3600b8b21bf2 100644 --- a/drivers/net/phy/marvell.c +++ b/drivers/net/phy/marvell.c | |||
| @@ -35,6 +35,8 @@ | |||
| 35 | #include <asm/irq.h> | 35 | #include <asm/irq.h> |
| 36 | #include <asm/uaccess.h> | 36 | #include <asm/uaccess.h> |
| 37 | 37 | ||
| 38 | #define MII_MARVELL_PHY_PAGE 22 | ||
| 39 | |||
| 38 | #define MII_M1011_IEVENT 0x13 | 40 | #define MII_M1011_IEVENT 0x13 |
| 39 | #define MII_M1011_IEVENT_CLEAR 0x0000 | 41 | #define MII_M1011_IEVENT_CLEAR 0x0000 |
| 40 | 42 | ||
| @@ -80,7 +82,6 @@ | |||
| 80 | #define MII_88E1121_PHY_LED_CTRL 16 | 82 | #define MII_88E1121_PHY_LED_CTRL 16 |
| 81 | #define MII_88E1121_PHY_LED_PAGE 3 | 83 | #define MII_88E1121_PHY_LED_PAGE 3 |
| 82 | #define MII_88E1121_PHY_LED_DEF 0x0030 | 84 | #define MII_88E1121_PHY_LED_DEF 0x0030 |
| 83 | #define MII_88E1121_PHY_PAGE 22 | ||
| 84 | 85 | ||
| 85 | #define MII_M1011_PHY_STATUS 0x11 | 86 | #define MII_M1011_PHY_STATUS 0x11 |
| 86 | #define MII_M1011_PHY_STATUS_1000 0x8000 | 87 | #define MII_M1011_PHY_STATUS_1000 0x8000 |
| @@ -190,9 +191,9 @@ static int m88e1121_config_aneg(struct phy_device *phydev) | |||
| 190 | { | 191 | { |
| 191 | int err, oldpage, mscr; | 192 | int err, oldpage, mscr; |
| 192 | 193 | ||
| 193 | oldpage = phy_read(phydev, MII_88E1121_PHY_PAGE); | 194 | oldpage = phy_read(phydev, MII_MARVELL_PHY_PAGE); |
| 194 | 195 | ||
| 195 | err = phy_write(phydev, MII_88E1121_PHY_PAGE, | 196 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, |
| 196 | MII_88E1121_PHY_MSCR_PAGE); | 197 | MII_88E1121_PHY_MSCR_PAGE); |
| 197 | if (err < 0) | 198 | if (err < 0) |
| 198 | return err; | 199 | return err; |
| @@ -218,7 +219,7 @@ static int m88e1121_config_aneg(struct phy_device *phydev) | |||
| 218 | return err; | 219 | return err; |
| 219 | } | 220 | } |
| 220 | 221 | ||
| 221 | phy_write(phydev, MII_88E1121_PHY_PAGE, oldpage); | 222 | phy_write(phydev, MII_MARVELL_PHY_PAGE, oldpage); |
| 222 | 223 | ||
| 223 | err = phy_write(phydev, MII_BMCR, BMCR_RESET); | 224 | err = phy_write(phydev, MII_BMCR, BMCR_RESET); |
| 224 | if (err < 0) | 225 | if (err < 0) |
| @@ -229,11 +230,11 @@ static int m88e1121_config_aneg(struct phy_device *phydev) | |||
| 229 | if (err < 0) | 230 | if (err < 0) |
| 230 | return err; | 231 | return err; |
| 231 | 232 | ||
| 232 | oldpage = phy_read(phydev, MII_88E1121_PHY_PAGE); | 233 | oldpage = phy_read(phydev, MII_MARVELL_PHY_PAGE); |
| 233 | 234 | ||
| 234 | phy_write(phydev, MII_88E1121_PHY_PAGE, MII_88E1121_PHY_LED_PAGE); | 235 | phy_write(phydev, MII_MARVELL_PHY_PAGE, MII_88E1121_PHY_LED_PAGE); |
| 235 | phy_write(phydev, MII_88E1121_PHY_LED_CTRL, MII_88E1121_PHY_LED_DEF); | 236 | phy_write(phydev, MII_88E1121_PHY_LED_CTRL, MII_88E1121_PHY_LED_DEF); |
| 236 | phy_write(phydev, MII_88E1121_PHY_PAGE, oldpage); | 237 | phy_write(phydev, MII_MARVELL_PHY_PAGE, oldpage); |
| 237 | 238 | ||
| 238 | err = genphy_config_aneg(phydev); | 239 | err = genphy_config_aneg(phydev); |
| 239 | 240 | ||
| @@ -244,9 +245,9 @@ static int m88e1318_config_aneg(struct phy_device *phydev) | |||
| 244 | { | 245 | { |
| 245 | int err, oldpage, mscr; | 246 | int err, oldpage, mscr; |
| 246 | 247 | ||
| 247 | oldpage = phy_read(phydev, MII_88E1121_PHY_PAGE); | 248 | oldpage = phy_read(phydev, MII_MARVELL_PHY_PAGE); |
| 248 | 249 | ||
| 249 | err = phy_write(phydev, MII_88E1121_PHY_PAGE, | 250 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, |
| 250 | MII_88E1121_PHY_MSCR_PAGE); | 251 | MII_88E1121_PHY_MSCR_PAGE); |
| 251 | if (err < 0) | 252 | if (err < 0) |
| 252 | return err; | 253 | return err; |
| @@ -258,7 +259,7 @@ static int m88e1318_config_aneg(struct phy_device *phydev) | |||
| 258 | if (err < 0) | 259 | if (err < 0) |
| 259 | return err; | 260 | return err; |
| 260 | 261 | ||
| 261 | err = phy_write(phydev, MII_88E1121_PHY_PAGE, oldpage); | 262 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, oldpage); |
| 262 | if (err < 0) | 263 | if (err < 0) |
| 263 | return err; | 264 | return err; |
| 264 | 265 | ||
| @@ -398,7 +399,7 @@ static int m88e1118_config_init(struct phy_device *phydev) | |||
| 398 | int err; | 399 | int err; |
| 399 | 400 | ||
| 400 | /* Change address */ | 401 | /* Change address */ |
| 401 | err = phy_write(phydev, 0x16, 0x0002); | 402 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, 0x0002); |
| 402 | if (err < 0) | 403 | if (err < 0) |
| 403 | return err; | 404 | return err; |
| 404 | 405 | ||
| @@ -408,7 +409,7 @@ static int m88e1118_config_init(struct phy_device *phydev) | |||
| 408 | return err; | 409 | return err; |
| 409 | 410 | ||
| 410 | /* Change address */ | 411 | /* Change address */ |
| 411 | err = phy_write(phydev, 0x16, 0x0003); | 412 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, 0x0003); |
| 412 | if (err < 0) | 413 | if (err < 0) |
| 413 | return err; | 414 | return err; |
| 414 | 415 | ||
| @@ -421,7 +422,7 @@ static int m88e1118_config_init(struct phy_device *phydev) | |||
| 421 | return err; | 422 | return err; |
| 422 | 423 | ||
| 423 | /* Reset address */ | 424 | /* Reset address */ |
| 424 | err = phy_write(phydev, 0x16, 0x0); | 425 | err = phy_write(phydev, MII_MARVELL_PHY_PAGE, 0x0); |
| 425 | if (err < 0) | 426 | if (err < 0) |
| 426 | return err; | 427 | return err; |
| 427 | 428 | ||
