Re: [PATCH net-next 0/2] ice: support FEC automatic disable
From: Gal Pressman <hidden>
Date: 2022-08-24 13:35:57
From: Gal Pressman <hidden>
Date: 2022-08-24 13:35:57
On 23/08/2022 18:04, Jacob Keller wrote:
2) always treat ETHTOOL_FEC_AUTO as "automatic + allow disable" This could work, but it means that behavior will differ depending on the firmware version. Users have no way to know that and might be surprised to find the behavior differ across devices which have different firmware which do or don't support this variation of automatic selection.
Hi Jacob, This is exactly how it's already implemented in mlx5, and I don't really understand how firmware version is related? Is it specific to your device firmware? Maybe you can workaround that in the driver? I feel like we're going the wrong way here having different flags interpretations by different drivers.