> Restart autoneg if autoneg is enabled.

> +     if (!(link_info->autoneg & BNXT_AUTONEG_SPEED))
> +             return -EINVAL;

Looks like a lot of the of implementations of this ethtool
are unconditionally doing the reset.
Perhaps the right thing to do is check for autoneg availability
at the ethtool core itself?

[Not an objection to this patch in any way as there are tens
of implementations doing the same; Simply noting this]

Reply via email to