Re: [PATCH net-next 6/6] ethtool: clarify the ethtool FEC interface

2021-03-29 Thread Jakub Kicinski
On Mon, 29 Mar 2021 12:56:30 +0100 Edward Cree wrote: > On 25/03/2021 01:12, Jakub Kicinski wrote: > > Drivers should reject mixing %ETHTOOL_FEC_AUTO_BIT with other > > + * FEC modes, because it's unclear whether in this case other modes > > constrain > > + * AUTO or are independent choices. >

Re: [PATCH net-next 6/6] ethtool: clarify the ethtool FEC interface

2021-03-29 Thread Andrew Lunn
On Mon, Mar 29, 2021 at 12:56:30PM +0100, Edward Cree wrote: > On 25/03/2021 01:12, Jakub Kicinski wrote: > > Drivers should reject mixing %ETHTOOL_FEC_AUTO_BIT with other > > + * FEC modes, because it's unclear whether in this case other modes > > constrain > > + * AUTO or are independent choices

Re: [PATCH net-next 6/6] ethtool: clarify the ethtool FEC interface

2021-03-29 Thread Edward Cree
On 25/03/2021 01:12, Jakub Kicinski wrote: > Drivers should reject mixing %ETHTOOL_FEC_AUTO_BIT with other > + * FEC modes, because it's unclear whether in this case other modes constrain > + * AUTO or are independent choices. Does this mean you want me to spin a patch to sfc to reject this? Curre

[PATCH net-next 6/6] ethtool: clarify the ethtool FEC interface

2021-03-24 Thread Jakub Kicinski
The definition of the FEC driver interface is quite unclear. Improve the documentation. This is based on current driver and user space code, as well as the discussions about the interface: RFC v1 (24 Oct 2016): https://lore.kernel.org/netdev/1477363849-36517-1-git-send-email-vi...@cumulusnetwork