summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMateusz Palczewski <mateusz.palczewski@intel.com>2021-03-10 11:12:54 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-07-20 16:15:52 +0200
commitac2abd4eec77810b28a77ede662cca7e10dd73cd (patch)
tree9e533e3451714516d9a5b342629fe8297f01a550
parentfdb69fdf4fe8ffcaad0ba0ad3608526ba798f6dd (diff)
downloadlinux-stable-ac2abd4eec77810b28a77ede662cca7e10dd73cd.tar.gz
linux-stable-ac2abd4eec77810b28a77ede662cca7e10dd73cd.tar.bz2
linux-stable-ac2abd4eec77810b28a77ede662cca7e10dd73cd.zip
i40e: Fix autoneg disabling for non-10GBaseT links
[ Upstream commit 9262793e59f0423437166a879a73d056b1fe6f9a ] Disabling autonegotiation was allowed only for 10GBaseT PHY. The condition was changed to check if link media type is BaseT. Fixes: 3ce12ee9d8f9 ("i40e: Fix order of checks when enabling/disabling autoneg in ethtool") Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com> Reviewed-by: Karen Sornek <karen.sornek@intel.com> Signed-off-by: Dawid Lukwinski <dawid.lukwinski@intel.com> Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> Tested-by: Tony Brelinski <tonyx.brelinski@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--drivers/net/ethernet/intel/i40e/i40e_ethtool.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
index 9148d93c5c63..4c7c1998f358 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
@@ -948,8 +948,7 @@ static int i40e_set_link_ksettings(struct net_device *netdev,
if (ethtool_link_ksettings_test_link_mode(&safe_ks,
supported,
Autoneg) &&
- hw->phy.link_info.phy_type !=
- I40E_PHY_TYPE_10GBASE_T) {
+ hw->phy.media_type != I40E_MEDIA_TYPE_BASET) {
netdev_info(netdev, "Autoneg cannot be disabled on this phy\n");
err = -EINVAL;
goto done;