diff options
author | Russell King (Oracle) <rmk+kernel@armlinux.org.uk> | 2025-01-08 16:48:50 +0000 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2025-01-10 17:51:02 -0800 |
commit | 27af08164247dfbe4371ee485366b851232e80e5 (patch) | |
tree | 63a91a3bda22d734015105002621439dd2ca6a46 /tools/perf/scripts/python | |
parent | 84f2776e391943ccae074f5232e54799f36ce73b (diff) |
net: stmmac: remove unnecessary EEE handling in stmmac_release()
phylink_stop() will cause phylink to call the mac_link_down() operation
before phylink_stop() returns. As mac_link_down() will call
stmmac_eee_init(false), this will set both priv->eee_active and
priv->eee_enabled to be false, deleting the eee_ctrl_timer if
priv->eee_enabled was previously set.
As stmmac_release() calls phylink_stop() before checking whether
priv->eee_enabled is true, this is a condition that can never be
satisfied, and thus the code within this if() block will never be
executed. Remove it.
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Tested-by: Choong Yong Liang <yong.liang.choong@linux.intel.com>
Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1tVZEg-0002LE-HH@rmk-PC.armlinux.org.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions