net: stmmac: convert to phylink managed EEE support

Convert stmmac to use phylink managed EEE support rather than delving
into phylib:

1. Move the stmmac_eee_init() calls out of mac_link_down() and
   mac_link_up() methods into the new mac_{enable,disable}_lpi()
   methods. We leave the calls to stmmac_set_eee_pls() in place as
   these change bits which tell the EEE hardware when the link came
   up or down, and is used for a separate hardware timer. However,
   symmetrically conditionalise this with priv->dma_cap.eee.

2. Update the current LPI timer each time LPI is enabled - which we
   need for software-timed LPI.

3. With phylink managed EEE, phylink manages the receive clock stop
   configuration via phylink_config.eee_rx_clk_stop_enable. Set this
   appropriately which makes the call to phy_eee_rx_clock_stop()
   redundant.

4. From what I can work out, all supported interfaces support LPI
   signalling on stmmac (there's no restriction implemented.) It
   also appears to support LPI at all full duplex speeds at or over
   100M. Set these capabilities.

5. The default timer appears to be derived from a module parameter.
   Set this the same, although we keep code that reconfigures the
   timer in stmmac_init_phy().

6. Remove the direct call to phy_support_eee(), which phylink will do
   on the drivers behalf if phylink_config.eee_enabled_default is set.

Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Link: https://patch.msgid.link/E1tYAEG-0014QH-9O@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 49b4114..d99ad77 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -988,8 +988,8 @@ static void stmmac_mac_link_down(struct phylink_config *config,
        struct stmmac_priv *priv = netdev_priv(to_net_dev(config->dev));
 
        stmmac_mac_set(priv, priv->ioaddr, false);
-       stmmac_eee_init(priv, false);
-       stmmac_set_eee_pls(priv, priv->hw, false);
+       if (priv->dma_cap.eee)
+               stmmac_set_eee_pls(priv, priv->hw, false);
 
        if (stmmac_fpe_supported(priv))
                stmmac_fpe_link_state_handle(priv, false);
@@ -1096,13 +1096,8 @@ static void stmmac_mac_link_up(struct phylink_config *config,
                writel(ctrl, priv->ioaddr + MAC_CTRL_REG);
 
        stmmac_mac_set(priv, priv->ioaddr, true);
-       if (phy && priv->dma_cap.eee) {
-               phy_eee_rx_clock_stop(phy, !(priv->plat->flags &
-                                            STMMAC_FLAG_RX_CLK_RUNS_IN_LPI));
-               priv->tx_lpi_timer = phy->eee_cfg.tx_lpi_timer;
-               stmmac_eee_init(priv, phy->enable_tx_lpi);
+       if (priv->dma_cap.eee)
                stmmac_set_eee_pls(priv, priv->hw, true);
-       }
 
        if (stmmac_fpe_supported(priv))
                stmmac_fpe_link_state_handle(priv, true);
@@ -1111,12 +1106,32 @@ static void stmmac_mac_link_up(struct phylink_config *config,
                stmmac_hwtstamp_correct_latency(priv, priv);
 }
 
+static void stmmac_mac_disable_tx_lpi(struct phylink_config *config)
+{
+       struct stmmac_priv *priv = netdev_priv(to_net_dev(config->dev));
+
+       stmmac_eee_init(priv, false);
+}
+
+static int stmmac_mac_enable_tx_lpi(struct phylink_config *config, u32 timer,
+                                   bool tx_clk_stop)
+{
+       struct stmmac_priv *priv = netdev_priv(to_net_dev(config->dev));
+
+       priv->tx_lpi_timer = timer;
+       stmmac_eee_init(priv, true);
+
+       return 0;
+}
+
 static const struct phylink_mac_ops stmmac_phylink_mac_ops = {
        .mac_get_caps = stmmac_mac_get_caps,
        .mac_select_pcs = stmmac_mac_select_pcs,
        .mac_config = stmmac_mac_config,
        .mac_link_down = stmmac_mac_link_down,
        .mac_link_up = stmmac_mac_link_up,
+       .mac_disable_tx_lpi = stmmac_mac_disable_tx_lpi,
+       .mac_enable_tx_lpi = stmmac_mac_enable_tx_lpi,
 };
 
 /**
@@ -1189,9 +1204,6 @@ static int stmmac_init_phy(struct net_device *dev)
                        return -ENODEV;
                }
 
-               if (priv->dma_cap.eee)
-                       phy_support_eee(phydev);
-
                ret = phylink_connect_phy(priv->phylink, phydev);
        } else {
                fwnode_handle_put(phy_fwnode);
@@ -1201,7 +1213,12 @@ static int stmmac_init_phy(struct net_device *dev)
        if (ret == 0) {
                struct ethtool_keee eee;
 
-               /* Configure phylib's copy of the LPI timer */
+               /* 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);
@@ -1234,6 +1251,9 @@ static int stmmac_phy_setup(struct stmmac_priv *priv)
        /* Stmmac always requires an RX clock for hardware initialization */
        priv->phylink_config.mac_requires_rxc = true;
 
+       if (!(priv->plat->flags & STMMAC_FLAG_RX_CLK_RUNS_IN_LPI))
+               priv->phylink_config.eee_rx_clk_stop_enable = true;
+
        mdio_bus_data = priv->plat->mdio_bus_data;
        if (mdio_bus_data)
                priv->phylink_config.default_an_inband =
@@ -1255,6 +1275,19 @@ static int stmmac_phy_setup(struct stmmac_priv *priv)
                                 priv->phylink_config.supported_interfaces,
                                 pcs->supported_interfaces);
 
+       if (priv->dma_cap.eee) {
+               /* Assume all supported interfaces also support LPI */
+               memcpy(priv->phylink_config.lpi_interfaces,
+                      priv->phylink_config.supported_interfaces,
+                      sizeof(priv->phylink_config.lpi_interfaces));
+
+               /* All full duplex speeds above 100Mbps are supported */
+               priv->phylink_config.lpi_capabilities = ~(MAC_1000FD - 1) |
+                                                       MAC_100FD;
+               priv->phylink_config.lpi_timer_default = eee_timer * 1000;
+               priv->phylink_config.eee_enabled_default = true;
+       }
+
        fwnode = priv->plat->port_node;
        if (!fwnode)
                fwnode = dev_fwnode(priv->device);