]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
net: ethernet: ti: cpsw: disable PHY EEE advertisement
authorHeiner Kallweit <hkallweit1@gmail.com>
Mon, 16 Dec 2024 21:32:25 +0000 (22:32 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 20 Dec 2024 03:07:48 +0000 (19:07 -0800)
commitc9f5a5dabbf5ab53a6392d7c782d373d2c892e21
tree418d622a3cbc1b5bc69fefb747e22d132aeb6aa8
parentb55498ff14bd14860d48dc8d2a0b6889b218c408
net: ethernet: ti: cpsw: disable PHY EEE advertisement

It seems the cpsw MAC doesn't support EEE. See e.g. the commit message of
ce2899428ec0 ("ARM: dts: am335x-baltos: disable EEE for Atheros 8035 PHY").
There are cases where this causes issues if the PHY's on both sides have
negotiated EEE. As a workaround EEE modes of the PHY are marked broken
in DT, effectively disabling EEE advertisement.
Improve this by using new function phy_disable_eee() in the MAC driver.
This properly disables EEE advertisement, and allows to remove the
eee-broken-xxx properties from DT. As EEE is disabled anyway, we can
remove also the set_eee ethtool op.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/d08a798e-8565-422c-b2ed-121794db077f@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/ti/cpsw.c
drivers/net/ethernet/ti/cpsw_ethtool.c
drivers/net/ethernet/ti/cpsw_new.c
drivers/net/ethernet/ti/cpsw_priv.h