Re: [PATCH net-next] net: ethtool: add support for forward error correction modes

On Sat, Feb 11, 2017 at 5:33 AM, Mintz, Yuval [off-list ref] wrote:

quoted

+/**
+ * struct ethtool_fecparam - Ethernet forward error correction(fec)
+parameters
+ * @cmd: Command number = %ETHTOOL_GFECPARAM or
%ETHTOOL_SFECPARAM
+ * @active_fec: FEC mode which is active on porte
+ * @fec: Bitmask of supported/configured FEC modes
+ * @rsvd: Reserved for future extensions. i.e FEC bypass feature.
+ *
+ * Drivers should reject a non-zero setting of @autoneg when
+ * autoneogotiation is disabled (or not supported) for the link.
+ *
+ */

Why request this from drivers?
Can't you reject such configuration requests in the ethtool-core?

Missed to remove this comment as FEC configuration is completely
decoupled from auto negotiation
and setting of auto negotiation is not required along with FEC
paramters anymore.
Thanks for catching it and will remove the comment in v2 patch if
there are any comments