[PATCH net-next v3 09/13] net: phylink: Use phy_caps_lookup for fixed-link configuration

Maxime Chevallier maxime.chevallier at bootlin.com
Fri Feb 28 08:49:45 PST 2025


On Fri, 28 Feb 2025 16:20:54 +0000
"Russell King (Oracle)" <linux at armlinux.org.uk> wrote:

> On Fri, Feb 28, 2025 at 03:55:34PM +0100, Maxime Chevallier wrote:
> >  	adv = pl->link_config.advertising;
> >  	linkmode_zero(adv);
> > -	linkmode_set_bit(s->bit, adv);
> > +	linkmode_and(adv, pl->supported, c->linkmodes);  
> 
> There is no need for linkmode_zero() with linkmode_and() immediately
> after. linkmode_and() writes to its entire destination.
> 

Ah true indeed, I'll address that.

Thanks Russell,

Maxime



More information about the linux-arm-kernel mailing list