+ 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]