net: genet: enable link energy detect powerdown for external PHYs
authorJonathan Bell <jonathan@raspberrypi.org>
Tue, 14 May 2019 16:17:59 +0000 (17:17 +0100)
committerpopcornmix <popcornmix@gmail.com>
Wed, 27 Jan 2021 19:12:59 +0000 (19:12 +0000)
There are several warts surrounding bcmgenet_mii_probe() as this
function is called from ndo_open, but it's calling registration-type
functions. The probe should be called at probe time and refactored
such that the PHY device data can be extracted to limit the scope
of this flag to Broadcom PHYs.

For now, pass this flag in as it puts our attached PHY into a low-power
state when disconnected.

Signed-off-by: Jonathan Bell <jonathan@raspberrypi.org>
drivers/net/ethernet/broadcom/genet/bcmmii.c

index 6fb6c35..2e516f4 100644 (file)
@@ -293,6 +293,8 @@ int bcmgenet_mii_probe(struct net_device *dev)
        /* Communicate the integrated PHY revision */
        if (priv->internal_phy)
                phy_flags = priv->gphy_rev;
+       else
+               phy_flags = PHY_BRCM_AUTO_PWRDWN_ENABLE;
 
        /* Initialize link state variables that bcmgenet_mii_setup() uses */
        priv->old_link = -1;