diff mbox series

[net-next,1/1] net: phy: broadcom: Add genphy_suspend and genphy_resume for BCM5464

Message ID 20190608135356.29898-1-olteanv@gmail.com
State Accepted
Delegated to: David Miller
Headers show
Series [net-next,1/1] net: phy: broadcom: Add genphy_suspend and genphy_resume for BCM5464 | expand

Commit Message

Vladimir Oltean June 8, 2019, 1:53 p.m. UTC
This puts the quad PHY ports in power-down mode when the PHY transitions
to the PHY_HALTED state.  It is likely that all the other PHYs support
the BMCR_PDOWN bit, but I only have the BCM5464R to test.

Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
---
 drivers/net/phy/broadcom.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Florian Fainelli June 10, 2019, 1:32 a.m. UTC | #1
On 6/8/2019 6:53 AM, Vladimir Oltean wrote:
> This puts the quad PHY ports in power-down mode when the PHY transitions
> to the PHY_HALTED state.  It is likely that all the other PHYs support
> the BMCR_PDOWN bit, but I only have the BCM5464R to test.

Yes, they should all support that low power mode, but like you, only
have a limited subset to work with.

> 
> Signed-off-by: Vladimir Oltean <olteanv@gmail.com>

Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
David Miller June 10, 2019, 3:05 a.m. UTC | #2
From: Vladimir Oltean <olteanv@gmail.com>
Date: Sat,  8 Jun 2019 16:53:56 +0300

> This puts the quad PHY ports in power-down mode when the PHY transitions
> to the PHY_HALTED state.  It is likely that all the other PHYs support
> the BMCR_PDOWN bit, but I only have the BCM5464R to test.
> 
> Signed-off-by: Vladimir Oltean <olteanv@gmail.com>

Applied.
diff mbox series

Patch

diff --git a/drivers/net/phy/broadcom.c b/drivers/net/phy/broadcom.c
index 67fa05d67523..937d0059e8ac 100644
--- a/drivers/net/phy/broadcom.c
+++ b/drivers/net/phy/broadcom.c
@@ -663,6 +663,8 @@  static struct phy_driver broadcom_drivers[] = {
 	.config_init	= bcm54xx_config_init,
 	.ack_interrupt	= bcm_phy_ack_intr,
 	.config_intr	= bcm_phy_config_intr,
+	.suspend	= genphy_suspend,
+	.resume		= genphy_resume,
 }, {
 	.phy_id		= PHY_ID_BCM5481,
 	.phy_id_mask	= 0xfffffff0,