net: bcmsysport: Indicate MAC is in charge of PHY PM
authorFlorian Fainelli <f.fainelli@gmail.com>
Tue, 25 Oct 2022 23:42:01 +0000 (16:42 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 3 Nov 2022 14:59:19 +0000 (23:59 +0900)
[ Upstream commit 9f172134dde7e4f5bf4b9139f23a1e741ec1c36e ]

Avoid the PHY library call unnecessarily into the suspend/resume
functions by setting phydev->mac_managed_pm to true. The SYSTEMPORT
driver essentially does exactly what mdio_bus_phy_resume() does by
calling phy_resume().

Fixes: fba863b81604 ("net: phy: make PHY PM ops a no-op if MAC driver manages PHY PM")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Link: https://lore.kernel.org/r/20221025234201.2549360-1-f.fainelli@gmail.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bcmsysport.c

index ae541a9..4c7f828 100644 (file)
@@ -1991,6 +1991,9 @@ static int bcm_sysport_open(struct net_device *dev)
                goto out_clk_disable;
        }
 
+       /* Indicate that the MAC is responsible for PHY PM */
+       phydev->mac_managed_pm = true;
+
        /* Reset house keeping link status */
        priv->old_duplex = -1;
        priv->old_link = -1;