On Fri, May 08, 2026 at 05:47:02AM +0800, KhaiWenTan wrote: > From: Faizal Rahim <[email protected]> > > This series adds support for forcing 10/100 Mb/s link speed via ethtool > when autonegotiation is disabled on the igc driver. > > Changes in v5: > - add removal justification to include copper context in commit > description for igc: remove unused autoneg_failed field (Paul) > - check that cmd->base.duplex is either DUPLEX_HALF or DUPLEX_FULL > in igc_ethtool_set_link_ksettings() (Simon) > - dynamically override hw->fc.current_mode to igc_fc_none during > link configuration instead of mutating requested_mode (Simon) > > Changes in v4: > - Validate that autoneg is AUTONEG_ENABLE or AUTONEG_DISABLE early > in igc_ethtool_set_link_ksettings() to avoid passing unexpected > values to igc_handle_autoneg_disabled(). (Simon Horman) > > Changes in v3: > - Modify condition from "if (duplex == DUPLEX_HALF)" to > "if (duplex != DUPLEX_FULL)". (Simon Horman) > > Changes in v2: > - When forcing half-duplex, set hw->fc.requested_mode = igc_fc_none, > since half-duplex cannot support flow control per IEEE 802.3. > (Simon Horman) > - Split the original single patch into three patches for clarity: > patches 1 and 2 are preparatory cleanups; patch 3 carries the > functional change.
Thanks for your persistence. For the series: Reviewed-by: Simon Horman <[email protected]>
