Hi Philippe, thanks you for your patch, it looks good. One improvement - could you please add support/advertising for TP?
>+ supported |= SUPPORTED_Autoneg; need: supported |= SUPPORTED_TP; >+ advertising = (self->aq_nic_cfg.is_autoneg) ? > ADVERTISED_Autoneg : 0U; need: advertising |= ADVERTISED_TP; Thank you.