net: stmmac: simplify stmmac_init_phy()

If we fail to attach a PHY, there is no point trying to configure WoL
settings. Exit the function after printing the "cannot attach to PHY"
error, and remove the now unnecessary code indentation for configuring
the LPI timer in phylink. Since we know that "ret" must be zero at this
point, change the final return to use a constant rather than "ret".

Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Reviewed-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
Link: https://patch.msgid.link/E1v11A8-0000000774M-3pmH@rmk-PC.armlinux.org.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 4844d563e2913dbeec8d8918786583e1a7b8bd80..be064f240895cd1e9642571c59eea3c49487e040 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -1115,6 +1115,7 @@ static int stmmac_init_phy(struct net_device *dev)
        int mode = priv->plat->phy_interface;
        struct fwnode_handle *phy_fwnode;
        struct fwnode_handle *fwnode;
+       struct ethtool_keee eee;
        int ret;
 
        if (!phylink_expects_phy(priv->phylink))
@@ -1160,19 +1161,17 @@ static int stmmac_init_phy(struct net_device *dev)
        if (ret) {
                netdev_err(priv->dev, "cannot attach to PHY (error: %pe)\n",
                           ERR_PTR(ret));
-       } else {
-               struct ethtool_keee eee;
+               return ret;
+       }
 
-               /* Configure phylib's copy of the LPI timer. Normally,
-                * phylink_config.lpi_timer_default would do this, but there is
-                * a chance that userspace could change the eee_timer setting
-                * via sysfs before the first open. Thus, preserve existing
-                * behaviour.
-                */
-               if (!phylink_ethtool_get_eee(priv->phylink, &eee)) {
-                       eee.tx_lpi_timer = priv->tx_lpi_timer;
-                       phylink_ethtool_set_eee(priv->phylink, &eee);
-               }
+       /* Configure phylib's copy of the LPI timer. Normally,
+        * phylink_config.lpi_timer_default would do this, but there is a
+        * chance that userspace could change the eee_timer setting via sysfs
+        * before the first open. Thus, preserve existing behaviour.
+        */
+       if (!phylink_ethtool_get_eee(priv->phylink, &eee)) {
+               eee.tx_lpi_timer = priv->tx_lpi_timer;
+               phylink_ethtool_set_eee(priv->phylink, &eee);
        }
 
        if (!priv->plat->pmt) {
@@ -1183,7 +1182,7 @@ static int stmmac_init_phy(struct net_device *dev)
                device_set_wakeup_enable(priv->device, !!wol.wolopts);
        }
 
-       return ret;
+       return 0;
 }
 
 static int stmmac_phy_setup(struct stmmac_priv *priv)