2019-05-23 08:56:57

by Christophe Leroy

Subject: [PATCH net-next v2] net: phy: lxt: Add suspend/resume support to LXT971 and LXT973.

All LXT PHYs implement the standard "power down" bit 11 of
BMCR, so this patch adds support using the generic
genphy_{suspend,resume} functions added by
commit 0f0ca340e57b ("phy: power management support").

LXT970 is left aside because all registers get cleared upon
"power down" exit.

Reviewed-by: Heiner Kallweit <[email protected]>
Signed-off-by: Christophe Leroy <[email protected]>
---
I'd be happy if you could also merge that into 4.19

v2: revised commit log without the Fixes: tag.

drivers/net/phy/lxt.c | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/drivers/net/phy/lxt.c b/drivers/net/phy/lxt.c
index 314486288119..356bd6472f49 100644
--- a/drivers/net/phy/lxt.c
+++ b/drivers/net/phy/lxt.c
@@ -262,6 +262,8 @@ static struct phy_driver lxt97x_driver[] = {
/* PHY_BASIC_FEATURES */
.ack_interrupt = lxt971_ack_interrupt,
.config_intr = lxt971_config_intr,
+ .suspend = genphy_suspend,
+ .resume = genphy_resume,
}, {
.phy_id = 0x00137a10,
.name = "LXT973-A2",
@@ -271,6 +273,8 @@ static struct phy_driver lxt97x_driver[] = {
.probe = lxt973_probe,
.config_aneg = lxt973_config_aneg,
.read_status = lxt973a2_read_status,
+ .suspend = genphy_suspend,
+ .resume = genphy_resume,
}, {
.phy_id = 0x00137a10,
.name = "LXT973",
@@ -279,6 +283,8 @@ static struct phy_driver lxt97x_driver[] = {
.flags = 0,
.probe = lxt973_probe,
.config_aneg = lxt973_config_aneg,
+ .suspend = genphy_suspend,
+ .resume = genphy_resume,
} };

module_phy_driver(lxt97x_driver);
--
2.13.3

2019-05-23 14:38:39

by Florian Fainelli

Subject: Re: [PATCH net-next v2] net: phy: lxt: Add suspend/resume support to LXT971 and LXT973.

On 5/23/2019 1:55 AM, Christophe Leroy wrote:
> All LXT PHYs implement the standard "power down" bit 11 of
> BMCR, so this patch adds support using the generic
> genphy_{suspend,resume} functions added by
> commit 0f0ca340e57b ("phy: power management support").
>
> LXT970 is left aside because all registers get cleared upon
> "power down" exit.
>
> Reviewed-by: Heiner Kallweit <[email protected]>
> Signed-off-by: Christophe Leroy <[email protected]>

Reviewed-by: Florian Fainelli <[email protected]>
--
Florian

2019-05-23 16:21:52

by David Miller

Subject: Re: [PATCH net-next v2] net: phy: lxt: Add suspend/resume support to LXT971 and LXT973.

From: Christophe Leroy <[email protected]>
Date: Thu, 23 May 2019 08:55:32 +0000 (UTC)

> All LXT PHYs implement the standard "power down" bit 11 of
> BMCR, so this patch adds support using the generic
> genphy_{suspend,resume} functions added by
> commit 0f0ca340e57b ("phy: power management support").
>
> LXT970 is left aside because all registers get cleared upon
> "power down" exit.
>
> Reviewed-by: Heiner Kallweit <[email protected]>
> Signed-off-by: Christophe Leroy <[email protected]>
> ---
> I'd be happy if you could also merge that into 4.19
>
> v2: revised commit log without the Fixes: tag.

Applied, thanks.