From: mugunthanvnm@ti.com (Mugunthan V N)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH RFCv2 0/6] net: phy: Ethernet PHY powerdown optimization
Date: Thu, 5 Dec 2013 13:27:34 +0530 [thread overview]
Message-ID: <52A031EE.1040301@ti.com> (raw)
In-Reply-To: <1386171888-28190-1-git-send-email-sebastian.hesselbarth@gmail.com>
On Wednesday 04 December 2013 09:14 PM, Sebastian Hesselbarth wrote:
> This is v2 of an RFC sent earlier [1] to reduce power consumption of network
> PHYs with link that are either unused or the corresponding netdev is down.
>
> In contrast to RFCv1, this now integrates phy_suspend/phy_resume transparent
> to the netdev drivers. Also, phy_suspend now only suspends the PHY if WOL is
> disabled. Moreover, the phy state machine calls phy_suspend on entering
> HALTED state.
>
> Again, a branch with RFCv2 applied to v3.13-rc2 can also be found at
> https://github.com/shesselba/linux-dove.git topic/ethphy-power-rfc-v2
>
> [1] http://lwn.net/Articles/574426/
>
> Sebastian Hesselbarth (6):
> net: mv643xx_eth: properly start/stop phy device
> net: phy: marvell: provide genphy suspend/resume
> net: phy: provide phy_resume/phy_suspend helpers
> net: phy: resume/suspend PHYs on attach/detach
> net: phy: suspend unused PHYs on mdio_bus in late_initcall
> net: phy: suspend phydev when going to HALTED
>
> drivers/net/ethernet/marvell/mv643xx_eth.c | 4 +++-
> drivers/net/phy/marvell.c | 22 ++++++++++++++++++++++
> drivers/net/phy/mdio_bus.c | 25 +++++++++++++++++++++++++
> drivers/net/phy/phy.c | 6 +++++-
> drivers/net/phy/phy_device.c | 27 +++++++++++++++++++++++++++
> include/linux/phy.h | 2 ++
> 6 files changed, 84 insertions(+), 2 deletions(-)
>
> ---
> Cc: David Miller <davem@davemloft.net>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Mugunthan V N <mugunthanvnm@ti.com>
> Cc: netdev at vger.kernel.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-kernel at vger.kernel.org
Apart form Sergei's comment the patch series looks good to me.
Acked-by: Mugunthan V N <mugunthanvnm@ti.com>
Regards
Mugunthan V N
WARNING: multiple messages have this Message-ID (diff)
From: Mugunthan V N <mugunthanvnm@ti.com>
To: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Cc: David Miller <davem@davemloft.net>,
Florian Fainelli <f.fainelli@gmail.com>, <netdev@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH RFCv2 0/6] net: phy: Ethernet PHY powerdown optimization
Date: Thu, 5 Dec 2013 13:27:34 +0530 [thread overview]
Message-ID: <52A031EE.1040301@ti.com> (raw)
In-Reply-To: <1386171888-28190-1-git-send-email-sebastian.hesselbarth@gmail.com>
On Wednesday 04 December 2013 09:14 PM, Sebastian Hesselbarth wrote:
> This is v2 of an RFC sent earlier [1] to reduce power consumption of network
> PHYs with link that are either unused or the corresponding netdev is down.
>
> In contrast to RFCv1, this now integrates phy_suspend/phy_resume transparent
> to the netdev drivers. Also, phy_suspend now only suspends the PHY if WOL is
> disabled. Moreover, the phy state machine calls phy_suspend on entering
> HALTED state.
>
> Again, a branch with RFCv2 applied to v3.13-rc2 can also be found at
> https://github.com/shesselba/linux-dove.git topic/ethphy-power-rfc-v2
>
> [1] http://lwn.net/Articles/574426/
>
> Sebastian Hesselbarth (6):
> net: mv643xx_eth: properly start/stop phy device
> net: phy: marvell: provide genphy suspend/resume
> net: phy: provide phy_resume/phy_suspend helpers
> net: phy: resume/suspend PHYs on attach/detach
> net: phy: suspend unused PHYs on mdio_bus in late_initcall
> net: phy: suspend phydev when going to HALTED
>
> drivers/net/ethernet/marvell/mv643xx_eth.c | 4 +++-
> drivers/net/phy/marvell.c | 22 ++++++++++++++++++++++
> drivers/net/phy/mdio_bus.c | 25 +++++++++++++++++++++++++
> drivers/net/phy/phy.c | 6 +++++-
> drivers/net/phy/phy_device.c | 27 +++++++++++++++++++++++++++
> include/linux/phy.h | 2 ++
> 6 files changed, 84 insertions(+), 2 deletions(-)
>
> ---
> Cc: David Miller <davem@davemloft.net>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Mugunthan V N <mugunthanvnm@ti.com>
> Cc: netdev@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-kernel@vger.kernel.org
Apart form Sergei's comment the patch series looks good to me.
Acked-by: Mugunthan V N <mugunthanvnm@ti.com>
Regards
Mugunthan V N
next prev parent reply other threads:[~2013-12-05 7:57 UTC|newest]
Thread overview: 116+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-20 20:21 [PATCH RFC v1 0/7] net: phy: Ethernet PHY powerdown optimization Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` [PATCH RFC v1 1/7] net: phy: marvell: provide genphy suspend/resume Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` [PATCH RFC v1 2/7] net: phy: provide phy_resume/phy_suspend helpers Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:36 ` Florian Fainelli
2013-11-20 20:36 ` Florian Fainelli
2013-11-20 20:21 ` [PATCH RFC v1 3/7] net: phy: resume/suspend PHYs on attach/detach Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` [PATCH RFC v1 4/7] net: phy: suspend unused PHYs on mdio_bus in late_initcall Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:58 ` Florian Fainelli
2013-11-20 20:58 ` Florian Fainelli
2013-11-20 21:05 ` Sebastian Hesselbarth
2013-11-20 21:05 ` Sebastian Hesselbarth
2013-11-20 22:36 ` Florian Fainelli
2013-11-20 22:36 ` Florian Fainelli
2013-11-20 20:21 ` [PATCH RFC v1 5/7] net: mv643xx_eth: resume/suspend PHY on port start/stop Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` [PATCH RFC v1 6/7] net: mvneta: " Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:21 ` [PATCH RFC v1 7/7] net: cpsw: " Sebastian Hesselbarth
2013-11-20 20:21 ` Sebastian Hesselbarth
2013-11-20 20:48 ` Florian Fainelli
2013-11-20 20:48 ` Florian Fainelli
2013-11-20 20:57 ` Sebastian Hesselbarth
2013-11-20 20:57 ` Sebastian Hesselbarth
2013-11-21 8:35 ` Mugunthan V N
2013-11-21 8:35 ` Mugunthan V N
2013-11-21 8:35 ` Mugunthan V N
2013-11-21 8:41 ` Sebastian Hesselbarth
2013-11-21 8:41 ` Sebastian Hesselbarth
2013-11-20 20:36 ` [PATCH RFC v1 0/7] net: phy: Ethernet PHY powerdown optimization David Miller
2013-11-20 20:36 ` David Miller
2013-11-20 20:54 ` Sebastian Hesselbarth
2013-11-20 20:54 ` Sebastian Hesselbarth
2013-11-20 21:10 ` Florian Fainelli
2013-11-20 21:10 ` Florian Fainelli
2013-11-20 21:20 ` Sebastian Hesselbarth
2013-11-20 21:20 ` Sebastian Hesselbarth
2013-11-20 22:15 ` David Miller
2013-11-20 22:15 ` David Miller
2013-12-04 15:44 ` [PATCH RFCv2 0/6] " Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-05 7:57 ` Mugunthan V N [this message]
2013-12-05 7:57 ` Mugunthan V N
2013-12-06 0:16 ` Florian Fainelli
2013-12-06 0:16 ` Florian Fainelli
2013-12-08 14:40 ` [PATCH v1 " Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-08 14:40 ` [PATCH v1 1/6] net: mv643xx_eth: properly start/stop phy device Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-11 2:52 ` David Miller
2013-12-11 2:52 ` David Miller
2013-12-11 2:56 ` David Miller
2013-12-11 2:56 ` David Miller
2013-12-11 2:56 ` David Miller
2013-12-11 7:06 ` Sebastian Hesselbarth
2013-12-11 7:06 ` Sebastian Hesselbarth
2013-12-11 7:06 ` Sebastian Hesselbarth
2013-12-11 17:28 ` David Miller
2013-12-11 17:28 ` David Miller
2013-12-11 17:28 ` David Miller
2013-12-08 14:40 ` [PATCH v1 2/6] net: phy: marvell: provide genphy suspend/resume Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-08 14:40 ` [PATCH v1 3/6] net: phy: provide phy_resume/phy_suspend helpers Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-08 14:40 ` [PATCH v1 4/6] net: phy: resume/suspend PHYs on attach/detach Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-08 14:40 ` [PATCH v1 5/6] net: phy: suspend unused PHYs on mdio_bus in late_initcall Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-08 14:40 ` [PATCH v1 6/6] net: phy: suspend phydev when going to HALTED Sebastian Hesselbarth
2013-12-08 14:40 ` Sebastian Hesselbarth
2013-12-13 9:20 ` [PATCH v2 0/5] net: phy: Ethernet PHY powerdown optimization Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-17 19:43 ` David Miller
2013-12-17 19:43 ` David Miller
2014-02-04 19:38 ` Sebastian Hesselbarth
2014-02-04 19:38 ` Sebastian Hesselbarth
2014-02-04 22:51 ` Florian Fainelli
2014-02-04 22:51 ` Florian Fainelli
2014-02-06 16:57 ` Ezequiel Garcia
2014-02-06 16:57 ` Ezequiel Garcia
2013-12-13 9:20 ` [PATCH v2 1/5] net: mv643xx_eth: properly start/stop phy device Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-13 9:20 ` [PATCH v2 2/5] net: phy: marvell: provide genphy suspend/resume Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-13 9:20 ` [PATCH v2 3/5] net: phy: provide phy_resume/phy_suspend helpers Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-13 9:20 ` [PATCH v2 4/5] net: phy: resume/suspend PHYs on attach/detach Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-13 9:20 ` [PATCH v2 5/5] net: phy: suspend phydev when going to HALTED Sebastian Hesselbarth
2013-12-13 9:20 ` Sebastian Hesselbarth
2013-12-04 15:44 ` [PATCH RFCv2 1/6] net: mv643xx_eth: properly start/stop phy device Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 15:44 ` [PATCH RFCv2 2/6] net: phy: marvell: provide genphy suspend/resume Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 15:44 ` [PATCH RFCv2 3/6] net: phy: provide phy_resume/phy_suspend helpers Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-06 0:08 ` Florian Fainelli
2013-12-06 0:08 ` Florian Fainelli
2013-12-04 15:44 ` [PATCH RFCv2 4/6] net: phy: resume/suspend PHYs on attach/detach Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 15:44 ` [PATCH RFCv2 5/6] net: phy: suspend unused PHYs on mdio_bus in late_initcall Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
2013-12-04 21:05 ` Sergei Shtylyov
2013-12-04 21:05 ` Sergei Shtylyov
2013-12-06 0:02 ` Florian Fainelli
2013-12-06 0:02 ` Florian Fainelli
2013-12-04 15:44 ` [PATCH RFCv2 6/6] net: phy: suspend phydev when going to HALTED Sebastian Hesselbarth
2013-12-04 15:44 ` Sebastian Hesselbarth
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52A031EE.1040301@ti.com \
--to=mugunthanvnm@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.