lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 4 Aug 2019 22:37:08 +0300 From: Vladimir Oltean <olteanv@...il.com> To: Hubert Feurstein <h.feurstein@...il.com> Cc: netdev <netdev@...r.kernel.org>, lkml <linux-kernel@...r.kernel.org>, Andrew Lunn <andrew@...n.ch>, Vivien Didelot <vivien.didelot@...il.com>, Florian Fainelli <f.fainelli@...il.com>, "David S. Miller" <davem@...emloft.net>, Ioana Ciornei <ioana.ciornei@....com> Subject: Re: [PATCH net] net: dsa: mv88e6xxx: drop adjust_link to enabled phylink On Wed, 31 Jul 2019 at 18:43, Hubert Feurstein <h.feurstein@...il.com> wrote: > > We have to drop the adjust_link callback in order to finally migrate to > phylink. > > Otherwise we get the following warning during startup: > "mv88e6xxx 2188000.ethernet-1:10: Using legacy PHYLIB callbacks. Please > migrate to PHYLINK!" > > The warning is generated in the function dsa_port_link_register_of in > dsa/port.c: > > int dsa_port_link_register_of(struct dsa_port *dp) > { > struct dsa_switch *ds = dp->ds; > > if (!ds->ops->adjust_link) > return dsa_port_phylink_register(dp); > > dev_warn(ds->dev, > "Using legacy PHYLIB callbacks. Please migrate to PHYLINK!\n"); > [...] > } > > Signed-off-by: Hubert Feurstein <h.feurstein@...il.com> > --- Reviewed-by: Vladimir Oltean <olteanv@...il.com> > drivers/net/dsa/mv88e6xxx/chip.c | 26 -------------------------- > 1 file changed, 26 deletions(-) > > diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c > index 366f70bfe055..37e8babd035f 100644 > --- a/drivers/net/dsa/mv88e6xxx/chip.c > +++ b/drivers/net/dsa/mv88e6xxx/chip.c > @@ -27,7 +27,6 @@ > #include <linux/platform_data/mv88e6xxx.h> > #include <linux/netdevice.h> > #include <linux/gpio/consumer.h> > -#include <linux/phy.h> > #include <linux/phylink.h> > #include <net/dsa.h> > > @@ -482,30 +481,6 @@ static int mv88e6xxx_phy_is_internal(struct dsa_switch *ds, int port) > return port < chip->info->num_internal_phys; > } > > -/* We expect the switch to perform auto negotiation if there is a real > - * phy. However, in the case of a fixed link phy, we force the port > - * settings from the fixed link settings. > - */ > -static void mv88e6xxx_adjust_link(struct dsa_switch *ds, int port, > - struct phy_device *phydev) > -{ > - struct mv88e6xxx_chip *chip = ds->priv; > - int err; > - > - if (!phy_is_pseudo_fixed_link(phydev) && > - mv88e6xxx_phy_is_internal(ds, port)) > - return; > - > - mv88e6xxx_reg_lock(chip); > - err = mv88e6xxx_port_setup_mac(chip, port, phydev->link, phydev->speed, > - phydev->duplex, phydev->pause, > - phydev->interface); > - mv88e6xxx_reg_unlock(chip); > - > - if (err && err != -EOPNOTSUPP) > - dev_err(ds->dev, "p%d: failed to configure MAC\n", port); > -} > - > static void mv88e6065_phylink_validate(struct mv88e6xxx_chip *chip, int port, > unsigned long *mask, > struct phylink_link_state *state) > @@ -4755,7 +4730,6 @@ static int mv88e6xxx_port_egress_floods(struct dsa_switch *ds, int port, > static const struct dsa_switch_ops mv88e6xxx_switch_ops = { > .get_tag_protocol = mv88e6xxx_get_tag_protocol, > .setup = mv88e6xxx_setup, > - .adjust_link = mv88e6xxx_adjust_link, > .phylink_validate = mv88e6xxx_validate, > .phylink_mac_link_state = mv88e6xxx_link_state, > .phylink_mac_config = mv88e6xxx_mac_config, > -- > 2.22.0 >
Powered by blists - more mailing lists