| Message ID | E1m2y9G-0005vm-Vm@rmk-PC.armlinux.org.uk |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [RFC,net-next] net: dpaa2-mac: add support for more ethtool 10G link modes | expand |
On Mon, Jul 12, 2021 at 04:47:10PM +0100, Russell King wrote: > Phylink documentation says: > * Note that the PHY may be able to transform from one connection > * technology to another, so, eg, don't clear 1000BaseX just > * because the MAC is unable to BaseX mode. This is more about > * clearing unsupported speeds and duplex settings. The port modes > * should not be cleared; phylink_set_port_modes() will help with this. > > So add the missing 10G modes. > > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> Acked-by: Ioana Ciornei <ioana.ciornei@nxp.com> > --- > net-next is currently closed, but I'd like to collect acks for this > patch. Thanks. > > drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c > index ae6d382d8735..543c1f202420 100644 > --- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c > +++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c > @@ -140,6 +140,11 @@ static void dpaa2_mac_validate(struct phylink_config *config, > case PHY_INTERFACE_MODE_10GBASER: > case PHY_INTERFACE_MODE_USXGMII: > phylink_set(mask, 10000baseT_Full); > + phylink_set(mask, 10000baseCR_Full); > + phylink_set(mask, 10000baseSR_Full); > + phylink_set(mask, 10000baseLR_Full); > + phylink_set(mask, 10000baseLRM_Full); > + phylink_set(mask, 10000baseER_Full); > if (state->interface == PHY_INTERFACE_MODE_10GBASER) > break; > phylink_set(mask, 5000baseT_Full); > -- > 2.20.1 >
diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c index ae6d382d8735..543c1f202420 100644 --- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c +++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c @@ -140,6 +140,11 @@ static void dpaa2_mac_validate(struct phylink_config *config, case PHY_INTERFACE_MODE_10GBASER: case PHY_INTERFACE_MODE_USXGMII: phylink_set(mask, 10000baseT_Full); + phylink_set(mask, 10000baseCR_Full); + phylink_set(mask, 10000baseSR_Full); + phylink_set(mask, 10000baseLR_Full); + phylink_set(mask, 10000baseLRM_Full); + phylink_set(mask, 10000baseER_Full); if (state->interface == PHY_INTERFACE_MODE_10GBASER) break; phylink_set(mask, 5000baseT_Full);
Phylink documentation says: * Note that the PHY may be able to transform from one connection * technology to another, so, eg, don't clear 1000BaseX just * because the MAC is unable to BaseX mode. This is more about * clearing unsupported speeds and duplex settings. The port modes * should not be cleared; phylink_set_port_modes() will help with this. So add the missing 10G modes. Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> --- net-next is currently closed, but I'd like to collect acks for this patch. Thanks. drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c | 5 +++++ 1 file changed, 5 insertions(+)