diff options
| author | Russell King (Oracle) <[email protected]> | 2025-01-08 16:48:50 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2025-01-11 01:51:02 +0000 |
| commit | 27af08164247dfbe4371ee485366b851232e80e5 (patch) | |
| tree | 63a91a3bda22d734015105002621439dd2ca6a46 /net/tls/tls_main.c | |
| parent | net: stmmac: move setup of eee_ctrl_timer to stmmac_dvr_probe() (diff) | |
| download | kernel-27af08164247dfbe4371ee485366b851232e80e5.tar.gz kernel-27af08164247dfbe4371ee485366b851232e80e5.zip | |
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 <[email protected]>
Tested-by: Choong Yong Liang <[email protected]>
Signed-off-by: Russell King (Oracle) <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/tls/tls_main.c')
0 files changed, 0 insertions, 0 deletions