net: dsa: mv88e6xxx: Cache the port cmode
[linux-2.6-block.git] / drivers / net / dsa / mv88e6xxx / serdes.c
index 880b2cf0a53019027299cb4ae5356100b94fc651..064d0bb8fe02806dbdc008b0ffce0123ec71a612 100644 (file)
@@ -35,6 +35,22 @@ static int mv88e6352_serdes_write(struct mv88e6xxx_chip *chip, int reg,
                                        reg, val);
 }
 
+static int mv88e6390_serdes_read(struct mv88e6xxx_chip *chip,
+                                int lane, int device, int reg, u16 *val)
+{
+       int reg_c45 = MII_ADDR_C45 | device << 16 | reg;
+
+       return mv88e6xxx_phy_read(chip, lane, reg_c45, val);
+}
+
+static int mv88e6390_serdes_write(struct mv88e6xxx_chip *chip,
+                                 int lane, int device, int reg, u16 val)
+{
+       int reg_c45 = MII_ADDR_C45 | device << 16 | reg;
+
+       return mv88e6xxx_phy_write(chip, lane, reg_c45, val);
+}
+
 static int mv88e6352_serdes_power_set(struct mv88e6xxx_chip *chip, bool on)
 {
        u16 val, new_val;
@@ -57,14 +73,7 @@ static int mv88e6352_serdes_power_set(struct mv88e6xxx_chip *chip, bool on)
 
 static bool mv88e6352_port_has_serdes(struct mv88e6xxx_chip *chip, int port)
 {
-       u8 cmode;
-       int err;
-
-       err = mv88e6xxx_port_get_cmode(chip, port, &cmode);
-       if (err) {
-               dev_err(chip->dev, "failed to read cmode\n");
-               return false;
-       }
+       u8 cmode = chip->ports[port].cmode;
 
        if ((cmode == MV88E6XXX_PORT_STS_CMODE_100BASE_X) ||
            (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X) ||
@@ -174,16 +183,121 @@ int mv88e6352_serdes_get_stats(struct mv88e6xxx_chip *chip, int port,
        return ARRAY_SIZE(mv88e6352_serdes_hw_stats);
 }
 
+/* Return the SERDES lane address a port is using. Only Ports 9 and 10
+ * have SERDES lanes. Returns -ENODEV if a port does not have a lane.
+ */
+static int mv88e6390_serdes_get_lane(struct mv88e6xxx_chip *chip, int port)
+{
+       u8 cmode = chip->ports[port].cmode;
+
+       switch (port) {
+       case 9:
+               if (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode == MV88E6XXX_PORT_STS_CMODE_2500BASEX)
+                       return MV88E6390_PORT9_LANE0;
+               return -ENODEV;
+       case 10:
+               if (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode == MV88E6XXX_PORT_STS_CMODE_2500BASEX)
+                       return MV88E6390_PORT10_LANE0;
+               return -ENODEV;
+       default:
+               return -ENODEV;
+       }
+}
+
+/* Return the SERDES lane address a port is using. Ports 9 and 10 can
+ * use multiple lanes. If so, return the first lane the port uses.
+ * Returns -ENODEV if a port does not have a lane.
+ */
+static int mv88e6390x_serdes_get_lane(struct mv88e6xxx_chip *chip, int port)
+{
+       u8 cmode_port9, cmode_port10, cmode_port;
+
+       cmode_port9 = chip->ports[9].cmode;
+       cmode_port10 = chip->ports[10].cmode;
+       cmode_port = chip->ports[port].cmode;
+
+       switch (port) {
+       case 2:
+               if (cmode_port9 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_2500BASEX)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT9_LANE1;
+               return -ENODEV;
+       case 3:
+               if (cmode_port9 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT9_LANE2;
+               return -ENODEV;
+       case 4:
+               if (cmode_port9 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT9_LANE3;
+               return -ENODEV;
+       case 5:
+               if (cmode_port10 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_2500BASEX)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT10_LANE1;
+               return -ENODEV;
+       case 6:
+               if (cmode_port10 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT10_LANE2;
+               return -ENODEV;
+       case 7:
+               if (cmode_port10 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       if (cmode_port == MV88E6XXX_PORT_STS_CMODE_1000BASE_X)
+                               return MV88E6390_PORT10_LANE3;
+               return -ENODEV;
+       case 9:
+               if (cmode_port9 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_XAUI ||
+                   cmode_port9 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       return MV88E6390_PORT9_LANE0;
+               return -ENODEV;
+       case 10:
+               if (cmode_port10 == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_SGMII ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_2500BASEX ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_XAUI ||
+                   cmode_port10 == MV88E6XXX_PORT_STS_CMODE_RXAUI)
+                       return MV88E6390_PORT10_LANE0;
+               return -ENODEV;
+       default:
+               return -ENODEV;
+       }
+}
+
 /* Set the power on/off for 10GBASE-R and 10GBASE-X4/X2 */
-static int mv88e6390_serdes_10g(struct mv88e6xxx_chip *chip, int addr, bool on)
+static int mv88e6390_serdes_power_10g(struct mv88e6xxx_chip *chip, int lane,
+                                     bool on)
 {
        u16 val, new_val;
-       int reg_c45;
        int err;
 
-       reg_c45 = MII_ADDR_C45 | MV88E6390_SERDES_DEVICE |
-               MV88E6390_PCS_CONTROL_1;
-       err = mv88e6xxx_phy_read(chip, addr, reg_c45, &val);
+       err = mv88e6390_serdes_read(chip, lane, MDIO_MMD_PHYXS,
+                                   MV88E6390_PCS_CONTROL_1, &val);
+
        if (err)
                return err;
 
@@ -195,22 +309,21 @@ static int mv88e6390_serdes_10g(struct mv88e6xxx_chip *chip, int addr, bool on)
                new_val = val | MV88E6390_PCS_CONTROL_1_PDOWN;
 
        if (val != new_val)
-               err = mv88e6xxx_phy_write(chip, addr, reg_c45, new_val);
+               err = mv88e6390_serdes_write(chip, lane, MDIO_MMD_PHYXS,
+                                            MV88E6390_PCS_CONTROL_1, new_val);
 
        return err;
 }
 
-/* Set the power on/off for 10GBASE-R and 10GBASE-X4/X2 */
-static int mv88e6390_serdes_sgmii(struct mv88e6xxx_chip *chip, int addr,
-                                 bool on)
+/* Set the power on/off for SGMII and 1000Base-X */
+static int mv88e6390_serdes_power_sgmii(struct mv88e6xxx_chip *chip, int lane,
+                                       bool on)
 {
        u16 val, new_val;
-       int reg_c45;
        int err;
 
-       reg_c45 = MII_ADDR_C45 | MV88E6390_SERDES_DEVICE |
-               MV88E6390_SGMII_CONTROL;
-       err = mv88e6xxx_phy_read(chip, addr, reg_c45, &val);
+       err = mv88e6390_serdes_read(chip, lane, MDIO_MMD_PHYXS,
+                                   MV88E6390_SGMII_CONTROL, &val);
        if (err)
                return err;
 
@@ -222,106 +335,65 @@ static int mv88e6390_serdes_sgmii(struct mv88e6xxx_chip *chip, int addr,
                new_val = val | MV88E6390_SGMII_CONTROL_PDOWN;
 
        if (val != new_val)
-               err = mv88e6xxx_phy_write(chip, addr, reg_c45, new_val);
+               err = mv88e6390_serdes_write(chip, lane, MDIO_MMD_PHYXS,
+                                            MV88E6390_SGMII_CONTROL, new_val);
 
        return err;
 }
 
-static int mv88e6390_serdes_lower(struct mv88e6xxx_chip *chip, u8 cmode,
-                                 int port_donor, int lane, bool rxaui, bool on)
+static int mv88e6390_serdes_power_lane(struct mv88e6xxx_chip *chip, int port,
+                                      int lane, bool on)
 {
-       int err;
-       u8 cmode_donor;
-
-       err = mv88e6xxx_port_get_cmode(chip, port_donor, &cmode_donor);
-       if (err)
-               return err;
+       u8 cmode = chip->ports[port].cmode;
 
-       switch (cmode_donor) {
-       case MV88E6XXX_PORT_STS_CMODE_RXAUI:
-               if (!rxaui)
-                       break;
-               /* Fall through */
-       case MV88E6XXX_PORT_STS_CMODE_1000BASE_X:
-       case MV88E6XXX_PORT_STS_CMODE_SGMII:
-       case MV88E6XXX_PORT_STS_CMODE_2500BASEX:
-               if (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
-                   cmode == MV88E6XXX_PORT_STS_CMODE_SGMII)
-                       return  mv88e6390_serdes_sgmii(chip, lane, on);
-       }
-       return 0;
-}
-
-static int mv88e6390_serdes_port9(struct mv88e6xxx_chip *chip, u8 cmode,
-                                 bool on)
-{
        switch (cmode) {
-       case MV88E6XXX_PORT_STS_CMODE_1000BASE_X:
        case MV88E6XXX_PORT_STS_CMODE_SGMII:
-               return mv88e6390_serdes_sgmii(chip, MV88E6390_PORT9_LANE0, on);
+       case MV88E6XXX_PORT_STS_CMODE_1000BASE_X:
+       case MV88E6XXX_PORT_STS_CMODE_2500BASEX:
+               return mv88e6390_serdes_power_sgmii(chip, lane, on);
        case MV88E6XXX_PORT_STS_CMODE_XAUI:
        case MV88E6XXX_PORT_STS_CMODE_RXAUI:
-       case MV88E6XXX_PORT_STS_CMODE_2500BASEX:
-               return mv88e6390_serdes_10g(chip, MV88E6390_PORT9_LANE0, on);
+               return mv88e6390_serdes_power_10g(chip, lane, on);
        }
 
        return 0;
 }
 
-static int mv88e6390_serdes_port10(struct mv88e6xxx_chip *chip, u8 cmode,
-                                  bool on)
+int mv88e6390_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on)
 {
-       switch (cmode) {
-       case MV88E6XXX_PORT_STS_CMODE_SGMII:
-               return mv88e6390_serdes_sgmii(chip, MV88E6390_PORT10_LANE0, on);
-       case MV88E6XXX_PORT_STS_CMODE_XAUI:
-       case MV88E6XXX_PORT_STS_CMODE_RXAUI:
-       case MV88E6XXX_PORT_STS_CMODE_1000BASE_X:
-       case MV88E6XXX_PORT_STS_CMODE_2500BASEX:
-               return mv88e6390_serdes_10g(chip, MV88E6390_PORT10_LANE0, on);
+       int lane;
+
+       lane = mv88e6390_serdes_get_lane(chip, port);
+       if (lane == -ENODEV)
+               return 0;
+
+       if (lane < 0)
+               return lane;
+
+       switch (port) {
+       case 9 ... 10:
+               return mv88e6390_serdes_power_lane(chip, port, lane, on);
        }
 
        return 0;
 }
 
-int mv88e6390_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on)
+int mv88e6390x_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on)
 {
-       u8 cmode;
-       int err;
+       int lane;
 
-       err = mv88e6xxx_port_get_cmode(chip, port, &cmode);
-       if (err)
-               return err;
+       lane = mv88e6390x_serdes_get_lane(chip, port);
+       if (lane == -ENODEV)
+               return 0;
+
+       if (lane < 0)
+               return lane;
 
        switch (port) {
-       case 2:
-               return mv88e6390_serdes_lower(chip, cmode, 9,
-                                             MV88E6390_PORT9_LANE1,
-                                             false, on);
-       case 3:
-               return mv88e6390_serdes_lower(chip, cmode, 9,
-                                             MV88E6390_PORT9_LANE2,
-                                             true, on);
-       case 4:
-               return mv88e6390_serdes_lower(chip, cmode, 9,
-                                             MV88E6390_PORT9_LANE3,
-                                             true, on);
-       case 5:
-               return mv88e6390_serdes_lower(chip, cmode, 10,
-                                             MV88E6390_PORT10_LANE1,
-                                             false, on);
-       case 6:
-               return mv88e6390_serdes_lower(chip, cmode, 10,
-                                             MV88E6390_PORT10_LANE2,
-                                             true, on);
-       case 7:
-               return mv88e6390_serdes_lower(chip, cmode, 10,
-                                             MV88E6390_PORT10_LANE3,
-                                             true, on);
-       case 9:
-               return mv88e6390_serdes_port9(chip, cmode, on);
-       case 10:
-               return mv88e6390_serdes_port10(chip, cmode, on);
+       case 2 ... 4:
+       case 5 ... 7:
+       case 9 ... 10:
+               return mv88e6390_serdes_power_lane(chip, port, lane, on);
        }
 
        return 0;
@@ -329,20 +401,16 @@ int mv88e6390_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on)
 
 int mv88e6341_serdes_power(struct mv88e6xxx_chip *chip, int port, bool on)
 {
-       int err;
-       u8 cmode;
+       u8 cmode = chip->ports[port].cmode;
 
        if (port != 5)
                return 0;
 
-       err = mv88e6xxx_port_get_cmode(chip, port, &cmode);
-       if (err)
-               return err;
-
        if (cmode == MV88E6XXX_PORT_STS_CMODE_1000BASE_X ||
            cmode == MV88E6XXX_PORT_STS_CMODE_SGMII ||
            cmode == MV88E6XXX_PORT_STS_CMODE_2500BASEX)
-               return mv88e6390_serdes_sgmii(chip, MV88E6341_ADDR_SERDES, on);
+               return mv88e6390_serdes_power_sgmii(chip, MV88E6341_ADDR_SERDES,
+                                                   on);
 
        return 0;
 }