On 22/09/2023 19:38, Prasad Koya wrote:
This reverts commit 9ac3fc2f42e5ffa1e927dcbffb71b15fa81459e2.After the command "ethtool -s enps0 speed 100 duplex full autoneg on", i.e., advertise only 100Mbps speed to the peer, "ethtool enps0" shows advertised speeds as 100Mbps and 2500Mbps. Same behavior is seen when changing the speed to 10Mbps or 1000Mbps. This applies to I225/226 parts, which only supports copper mode. Reverting to original till the ambiguity is resolved. Fixes: 9ac3fc2f42e5 ("igc: set TP bit in 'supported' and 'advertising' fields of ethtool_link_ksettings") Signed-off-by: Prasad Koya <[email protected]>
Acked-by: Sasha Neftin <[email protected]>
--- drivers/net/ethernet/intel/igc/igc_ethtool.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/ethernet/intel/igc/igc_ethtool.c b/drivers/net/ethernet/intel/igc/igc_ethtool.c index 93bce729be76..0e2cb00622d1 100644 --- a/drivers/net/ethernet/intel/igc/igc_ethtool.c +++ b/drivers/net/ethernet/intel/igc/igc_ethtool.c @@ -1708,8 +1708,6 @@ static int igc_ethtool_get_link_ksettings(struct net_device *netdev, /* twisted pair */ cmd->base.port = PORT_TP; cmd->base.phy_address = hw->phy.addr; - ethtool_link_ksettings_add_link_mode(cmd, supported, TP); - ethtool_link_ksettings_add_link_mode(cmd, advertising, TP);/* advertising link modes */if (hw->phy.autoneg_advertised & ADVERTISE_10_HALF)
_______________________________________________ Intel-wired-lan mailing list [email protected] https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
