Skip to content

Commit a6fe31d

Browse files
atenartdavem330
authored andcommitted
net: mvpp2: 2500baseX support
This patch adds the 2500Base-X PHY mode support in the Marvell PPv2 driver. 2500Base-X is quite close to 1000Base-X and SGMII modes and uses nearly the same code path. Signed-off-by: Antoine Tenart <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent d97c9f4 commit a6fe31d

File tree

1 file changed

+39
-12
lines changed
  • drivers/net/ethernet/marvell

1 file changed

+39
-12
lines changed

drivers/net/ethernet/marvell/mvpp2.c

Lines changed: 39 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4871,6 +4871,7 @@ static int mvpp22_gop_init(struct mvpp2_port *port)
48714871
break;
48724872
case PHY_INTERFACE_MODE_SGMII:
48734873
case PHY_INTERFACE_MODE_1000BASEX:
4874+
case PHY_INTERFACE_MODE_2500BASEX:
48744875
mvpp22_gop_init_sgmii(port);
48754876
break;
48764877
case PHY_INTERFACE_MODE_10GKR:
@@ -4909,7 +4910,8 @@ static void mvpp22_gop_unmask_irq(struct mvpp2_port *port)
49094910

49104911
if (phy_interface_mode_is_rgmii(port->phy_interface) ||
49114912
port->phy_interface == PHY_INTERFACE_MODE_SGMII ||
4912-
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX) {
4913+
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX ||
4914+
port->phy_interface == PHY_INTERFACE_MODE_2500BASEX) {
49134915
/* Enable the GMAC link status irq for this port */
49144916
val = readl(port->base + MVPP22_GMAC_INT_SUM_MASK);
49154917
val |= MVPP22_GMAC_INT_SUM_MASK_LINK_STAT;
@@ -4940,7 +4942,8 @@ static void mvpp22_gop_mask_irq(struct mvpp2_port *port)
49404942

49414943
if (phy_interface_mode_is_rgmii(port->phy_interface) ||
49424944
port->phy_interface == PHY_INTERFACE_MODE_SGMII ||
4943-
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX) {
4945+
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX ||
4946+
port->phy_interface == PHY_INTERFACE_MODE_2500BASEX) {
49444947
val = readl(port->base + MVPP22_GMAC_INT_SUM_MASK);
49454948
val &= ~MVPP22_GMAC_INT_SUM_MASK_LINK_STAT;
49464949
writel(val, port->base + MVPP22_GMAC_INT_SUM_MASK);
@@ -4953,7 +4956,8 @@ static void mvpp22_gop_setup_irq(struct mvpp2_port *port)
49534956

49544957
if (phy_interface_mode_is_rgmii(port->phy_interface) ||
49554958
port->phy_interface == PHY_INTERFACE_MODE_SGMII ||
4956-
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX) {
4959+
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX ||
4960+
port->phy_interface == PHY_INTERFACE_MODE_2500BASEX) {
49574961
val = readl(port->base + MVPP22_GMAC_INT_MASK);
49584962
val |= MVPP22_GMAC_INT_MASK_LINK_STAT;
49594963
writel(val, port->base + MVPP22_GMAC_INT_MASK);
@@ -4968,6 +4972,16 @@ static void mvpp22_gop_setup_irq(struct mvpp2_port *port)
49684972
mvpp22_gop_unmask_irq(port);
49694973
}
49704974

4975+
/* Sets the PHY mode of the COMPHY (which configures the serdes lanes).
4976+
*
4977+
* The PHY mode used by the PPv2 driver comes from the network subsystem, while
4978+
* the one given to the COMPHY comes from the generic PHY subsystem. Hence they
4979+
* differ.
4980+
*
4981+
* The COMPHY configures the serdes lanes regardless of the actual use of the
4982+
* lanes by the physical layer. This is why configurations like
4983+
* "PPv2 (2500BaseX) - COMPHY (2500SGMII)" are valid.
4984+
*/
49714985
static int mvpp22_comphy_init(struct mvpp2_port *port)
49724986
{
49734987
enum phy_mode mode;
@@ -4981,6 +4995,9 @@ static int mvpp22_comphy_init(struct mvpp2_port *port)
49814995
case PHY_INTERFACE_MODE_1000BASEX:
49824996
mode = PHY_MODE_SGMII;
49834997
break;
4998+
case PHY_INTERFACE_MODE_2500BASEX:
4999+
mode = PHY_MODE_2500SGMII;
5000+
break;
49845001
case PHY_INTERFACE_MODE_10GKR:
49855002
mode = PHY_MODE_10GKR;
49865003
break;
@@ -5062,7 +5079,8 @@ static void mvpp2_port_loopback_set(struct mvpp2_port *port,
50625079
val &= ~MVPP2_GMAC_GMII_LB_EN_MASK;
50635080

50645081
if (port->phy_interface == PHY_INTERFACE_MODE_SGMII ||
5065-
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX)
5082+
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX ||
5083+
port->phy_interface == PHY_INTERFACE_MODE_2500BASEX)
50665084
val |= MVPP2_GMAC_PCS_LB_EN_MASK;
50675085
else
50685086
val &= ~MVPP2_GMAC_PCS_LB_EN_MASK;
@@ -6273,7 +6291,8 @@ static irqreturn_t mvpp2_link_status_isr(int irq, void *dev_id)
62736291
}
62746292
} else if (phy_interface_mode_is_rgmii(port->phy_interface) ||
62756293
port->phy_interface == PHY_INTERFACE_MODE_SGMII ||
6276-
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX) {
6294+
port->phy_interface == PHY_INTERFACE_MODE_1000BASEX ||
6295+
port->phy_interface == PHY_INTERFACE_MODE_2500BASEX) {
62776296
val = readl(port->base + MVPP22_GMAC_INT_STAT);
62786297
if (val & MVPP22_GMAC_INT_STAT_LINK) {
62796298
event = true;
@@ -8056,8 +8075,10 @@ static void mvpp2_phylink_validate(struct net_device *dev,
80568075
phylink_set(mask, 10000baseT_Full);
80578076
/* Fall-through */
80588077
case PHY_INTERFACE_MODE_1000BASEX:
8078+
case PHY_INTERFACE_MODE_2500BASEX:
80598079
phylink_set(mask, 1000baseT_Full);
80608080
phylink_set(mask, 1000baseX_Full);
8081+
phylink_set(mask, 2500baseX_Full);
80618082
}
80628083

80638084
bitmap_and(supported, supported, mask, __ETHTOOL_LINK_MODE_MASK_NBITS);
@@ -8100,6 +8121,9 @@ static void mvpp2_gmac_link_state(struct mvpp2_port *port,
81008121
case PHY_INTERFACE_MODE_1000BASEX:
81018122
state->speed = SPEED_1000;
81028123
break;
8124+
case PHY_INTERFACE_MODE_2500BASEX:
8125+
state->speed = SPEED_2500;
8126+
break;
81038127
default:
81048128
if (val & MVPP2_GMAC_STATUS0_GMII_SPEED)
81058129
state->speed = SPEED_1000;
@@ -8199,11 +8223,12 @@ static void mvpp2_gmac_config(struct mvpp2_port *port, unsigned int mode,
81998223
ctrl0 &= ~MVPP2_GMAC_PORT_TYPE_MASK;
82008224
ctrl2 &= ~(MVPP2_GMAC_PORT_RESET_MASK | MVPP2_GMAC_PCS_ENABLE_MASK);
82018225

8202-
if (state->interface == PHY_INTERFACE_MODE_1000BASEX) {
8203-
/* 1000BaseX port cannot negotiate speed nor can it negotiate
8204-
* duplex: they are always operating with a fixed speed of
8205-
* 1000Mbps in full duplex, so force 1000 speed and full duplex
8206-
* here.
8226+
if (state->interface == PHY_INTERFACE_MODE_1000BASEX ||
8227+
state->interface == PHY_INTERFACE_MODE_2500BASEX) {
8228+
/* 1000BaseX and 2500BaseX ports cannot negotiate speed nor can
8229+
* they negotiate duplex: they are always operating with a fixed
8230+
* speed of 1000/2500Mbps in full duplex, so force 1000/2500
8231+
* speed and full duplex here.
82078232
*/
82088233
ctrl0 |= MVPP2_GMAC_PORT_TYPE_MASK;
82098234
an |= MVPP2_GMAC_CONFIG_GMII_SPEED |
@@ -8220,7 +8245,8 @@ static void mvpp2_gmac_config(struct mvpp2_port *port, unsigned int mode,
82208245
an |= MVPP2_GMAC_FC_ADV_ASM_EN;
82218246

82228247
if (state->interface == PHY_INTERFACE_MODE_SGMII ||
8223-
state->interface == PHY_INTERFACE_MODE_1000BASEX) {
8248+
state->interface == PHY_INTERFACE_MODE_1000BASEX ||
8249+
state->interface == PHY_INTERFACE_MODE_2500BASEX) {
82248250
an |= MVPP2_GMAC_IN_BAND_AUTONEG;
82258251
ctrl2 |= MVPP2_GMAC_INBAND_AN_MASK | MVPP2_GMAC_PCS_ENABLE_MASK;
82268252

@@ -8286,7 +8312,8 @@ static void mvpp2_mac_config(struct net_device *dev, unsigned int mode,
82868312
mvpp2_xlg_config(port, mode, state);
82878313
else if (phy_interface_mode_is_rgmii(state->interface) ||
82888314
state->interface == PHY_INTERFACE_MODE_SGMII ||
8289-
state->interface == PHY_INTERFACE_MODE_1000BASEX)
8315+
state->interface == PHY_INTERFACE_MODE_1000BASEX ||
8316+
state->interface == PHY_INTERFACE_MODE_2500BASEX)
82908317
mvpp2_gmac_config(port, mode, state);
82918318

82928319
if (port->priv->hw_version == MVPP21 && port->flags & MVPP2_F_LOOPBACK)

0 commit comments

Comments
 (0)