Message ID | E1nNbgx-00Akj1-Sn@rmk-PC.armlinux.org.uk (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | net: dsa: ocelot: phylink updates | expand |
On Fri, Feb 25, 2022 at 02:35:31PM +0000, Russell King (Oracle) wrote: > The ocelot DSA driver does not make use of the speed, duplex, pause or > advertisement in its phylink_mac_config() implementation, so it can be > marked as a non-legacy driver. > > Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> > --- Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com> Tested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
On Fri, Feb 25, 2022 at 03:46:50PM +0000, Vladimir Oltean wrote: > On Fri, Feb 25, 2022 at 02:35:31PM +0000, Russell King (Oracle) wrote: > > The ocelot DSA driver does not make use of the speed, duplex, pause or > > advertisement in its phylink_mac_config() implementation, so it can be > > marked as a non-legacy driver. > > > > Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> > > --- > > Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com> > Tested-by: Vladimir Oltean <vladimir.oltean@nxp.com> Hi Vladimir, Shall I assume the tested-by applies to patch 2 and 3 as well? Thanks.
On Fri, Feb 25, 2022 at 03:59:42PM +0000, Russell King (Oracle) wrote: > On Fri, Feb 25, 2022 at 03:46:50PM +0000, Vladimir Oltean wrote: > > On Fri, Feb 25, 2022 at 02:35:31PM +0000, Russell King (Oracle) wrote: > > > The ocelot DSA driver does not make use of the speed, duplex, pause or > > > advertisement in its phylink_mac_config() implementation, so it can be > > > marked as a non-legacy driver. > > > > > > Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> > > > --- > > > > Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com> > > Tested-by: Vladimir Oltean <vladimir.oltean@nxp.com> > > Hi Vladimir, > > Shall I assume the tested-by applies to patch 2 and 3 as well? > > Thanks. Yes.
diff --git a/drivers/net/dsa/ocelot/felix.c b/drivers/net/dsa/ocelot/felix.c index 20ac74ee322d..f517bf902812 100644 --- a/drivers/net/dsa/ocelot/felix.c +++ b/drivers/net/dsa/ocelot/felix.c @@ -784,6 +784,12 @@ static void felix_phylink_get_caps(struct dsa_switch *ds, int port, struct ocelot *ocelot = ds->priv; struct felix *felix = ocelot_to_felix(ocelot); + /* This driver does not make use of the speed, duplex, pause or the + * advertisement in its mac_config, so it is safe to mark this driver + * as non-legacy. + */ + config->legacy_pre_march2020 = false; + if (felix->info->phylink_get_caps) felix->info->phylink_get_caps(ocelot, port, config); }
The ocelot DSA driver does not make use of the speed, duplex, pause or advertisement in its phylink_mac_config() implementation, so it can be marked as a non-legacy driver. Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> --- drivers/net/dsa/ocelot/felix.c | 6 ++++++ 1 file changed, 6 insertions(+)