diff options
| author | Russell King (Oracle) <[email protected]> | 2024-12-10 14:18:11 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-12-12 04:29:38 +0000 |
| commit | 66c366392e55ae07e37699eeacca50f01b0bb879 (patch) | |
| tree | 1140cc47881ea823cbf6f58b360ec33310fb83f0 /net/dsa/user.c | |
| parent | net: fec: use phydev->eee_cfg.tx_lpi_timer (diff) | |
| download | kernel-66c366392e55ae07e37699eeacca50f01b0bb879.tar.gz kernel-66c366392e55ae07e37699eeacca50f01b0bb879.zip | |
net: dsa: remove check for dp->pl in EEE methods
When user ports are initialised, a phylink instance is always created,
and so dp->pl will always be non-NULL. The EEE methods are only used
for user ports, so checking for dp->pl to be NULL makes no sense. No
other phylink-calling method implements similar checks in DSA. Remove
this unnecessary check.
Signed-off-by: Russell King (Oracle) <[email protected]>
Reviewed-by: Florian Fainelli <[email protected]>
Reviewed-by: Vladimir Oltean <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/dsa/user.c')
| -rw-r--r-- | net/dsa/user.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/dsa/user.c b/net/dsa/user.c index c736c019e2af..e1a0b153c353 100644 --- a/net/dsa/user.c +++ b/net/dsa/user.c @@ -1230,7 +1230,7 @@ static int dsa_user_set_eee(struct net_device *dev, struct ethtool_keee *e) int ret; /* Port's PHY and MAC both need to be EEE capable */ - if (!dev->phydev || !dp->pl) + if (!dev->phydev) return -ENODEV; if (!ds->ops->set_mac_eee) @@ -1250,7 +1250,7 @@ static int dsa_user_get_eee(struct net_device *dev, struct ethtool_keee *e) int ret; /* Port's PHY and MAC both need to be EEE capable */ - if (!dev->phydev || !dp->pl) + if (!dev->phydev) return -ENODEV; if (!ds->ops->get_mac_eee) |