On 2024-06-17 at 17:08:41, Kamil Horák - 2N (kamilh@xxxxxxxx) wrote: > + > + if (brr_mode) { > + linkmode_set_bit_array(phy_basic_ports_array, > + ARRAY_SIZE(phy_basic_ports_array), > + phydev->supported); > + > + val = phy_read(phydev, MII_BCM54XX_LRESR); > + if (val < 0) > + return val; > + > + linkmode_mod_bit(ETHTOOL_LINK_MODE_Autoneg_BIT, > + phydev->supported, 1); > + linkmode_mod_bit(ETHTOOL_LINK_MODE_100baseT1_Full_BIT, > + phydev->supported, > + val & LRESR_100_1PAIR); > + linkmode_mod_bit(ETHTOOL_LINK_MODE_10baseT1BRR_Full_BIT, > + phydev->supported, > + val & LRESR_10_1PAIR); > + } else { > + return genphy_read_abilities(phydev); > + } > + > + return 0; nit: Could you move this return to "if" statement and get rid of else part ? > +static int bcm5481_config_aneg(struct phy_device *phydev) > +{ > + int ret; > + u8 brr_mode; nit: Reverse xmas-tree. > +static int bcm54811_config_aneg(struct phy_device *phydev) > +{ > + int ret; > + u8 brr_mode; nit: Please apply reverse xmas-tree comment everywhere applicable.