Commit 3b0720ba authored by Nathan Rossi's avatar Nathan Rossi Committed by David S. Miller

net: dsa: mv88e6xxx: Update mv88e6393x serdes errata

In early erratas this issue only covered port 0 when changing from
[x]MII (rev A 3.6). In subsequent errata versions this errata changed to
cover the additional "Hardware reset in CPU managed mode" condition, and
removed the note specifying that it only applied to port 0.

In designs where the device is configured with CPU managed mode
(CPU_MGD), on reset all SERDES ports (p0, p9, p10) have a stuck power
down bit and require this initial power up procedure. As such apply this
errata to all three SERDES ports of the mv88e6393x.
Signed-off-by: default avatarNathan Rossi <nathan.rossi@digi.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 446e7f21
...@@ -1277,15 +1277,16 @@ static int mv88e6393x_serdes_port_errata(struct mv88e6xxx_chip *chip, int lane) ...@@ -1277,15 +1277,16 @@ static int mv88e6393x_serdes_port_errata(struct mv88e6xxx_chip *chip, int lane)
int err; int err;
/* mv88e6393x family errata 4.6: /* mv88e6393x family errata 4.6:
* Cannot clear PwrDn bit on SERDES on port 0 if device is configured * Cannot clear PwrDn bit on SERDES if device is configured CPU_MGD
* CPU_MGD mode or P0_mode is configured for [x]MII. * mode or P0_mode is configured for [x]MII.
* Workaround: Set Port0 SERDES register 4.F002 bit 5=0 and bit 15=1. * Workaround: Set SERDES register 4.F002 bit 5=0 and bit 15=1.
* *
* It seems that after this workaround the SERDES is automatically * It seems that after this workaround the SERDES is automatically
* powered up (the bit is cleared), so power it down. * powered up (the bit is cleared), so power it down.
*/ */
if (lane == MV88E6393X_PORT0_LANE) { if (lane == MV88E6393X_PORT0_LANE || lane == MV88E6393X_PORT9_LANE ||
err = mv88e6390_serdes_read(chip, MV88E6393X_PORT0_LANE, lane == MV88E6393X_PORT10_LANE) {
err = mv88e6390_serdes_read(chip, lane,
MDIO_MMD_PHYXS, MDIO_MMD_PHYXS,
MV88E6393X_SERDES_POC, &reg); MV88E6393X_SERDES_POC, &reg);
if (err) if (err)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment