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.

Reply via email to