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 | ||