From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mugunthan V N Subject: Re: [PATCH v2 00/15] drivers: net: cpsw: improve runtime pm Date: Mon, 27 Jun 2016 14:48:11 +0530 Message-ID: <5770EF53.3090500@ti.com> References: <20160624182355.23515-1-grygorii.strashko@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160624182355.23515-1-grygorii.strashko@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Grygorii Strashko , "David S. Miller" , netdev@vger.kernel.org Cc: Sekhar Nori , linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, Ivan Khoronzhuk , Rob Herring , devicetree@vger.kernel.org List-Id: devicetree@vger.kernel.org On Friday 24 June 2016 11:53 PM, Grygorii Strashko wrote: > This series intended to improve runtime PM and allow CPSW to be > RPM suspended when all ethX netdevices are down. > > To achieve above goal it is required to relax runtime PM constraints for > Davinci MDIO which blocks CPSW runtime PM now, because Davinci MDIO is always > powered on during probe and powered off only when it's going to be removed. > - Patches 6-11 implement PM runtime autosuspend for Davinci MDIO, but keep it > disabled by default, because Davinci MDIO is integrated in big set of TI devices > and not all of them verified to work correctly with RPM autosuspend enabled: > expected to work on SoCs where MDIO is defined as part of CPSW in DT > (cpsw.c DRA7/am57x, am437x, am335x) > The CPSW need to be fixed before RPM suspended can be allowed: > - Patches 1-5 ensure that CPSW will not cause L3 errors while it is in RPM > suspended state. > > Davinci MDIO RPM autosuspend can be enabled through sysfs: > echo 100 > /sys/devices/../48484000.ethernet/48485000.mdio/power/autosuspend_delay_ms > > Patches 12 - 15: introduce new compatible string "ti,cpsw-mdio" which is used > then to enable RPM for am335x/am437x/dra7 SoCs. > > Tested on am335x, am437x, am572x and k2g (on k2g with RPM disabled for Davinci MDIO) > These changes should not affect on errata i877 implementation on DRA7. > > Power measurement on am335x GP EVM: > Without this series: 547.60 mW total SoC power > With this series + "ifconfig eth0 down": 477.32 mW Total Soc Power > > Changes in v2: > - CPSW ethtool interface updated to use .begin()/.complete() callbacks > - kbuild failure fixed > - davinci_mdio DT updated with proper description of allowed compatible strings > combinations > > Link on v1: > https://lkml.org/lkml/2016/6/15/362 For the series. Reviewed-by: Mugunthan V N Regards Mugunthan V N > > Grygorii Strashko (15): > drivers: net: cpsw: fix suspend when all ethX devices are down > drivers: net: cpsw: check return code from pm runtime calls > drivers: net: cpsw: remove pm runtime calls from suspend callbacks > drivers: net: cpsw: ethtool: fix accessing to suspended device > drivers: net: cpsw: ndev: fix accessing to suspended device > drivers: net: davinci_mdio: do pm runtime initialization later in probe > drivers: net: davinci_mdio: remove pm runtime calls from suspend callbacks > drivers: net: davinci_mdio: drop suspended and lock fields from mdio_data > drivers: net: davinci_mdio: split reset function on init_clk and enable > drivers: net: davinci_mdio: add pm runtime callbacks > drivers: net: davinci_mdio: implement pm runtime auto mode > net: davinci_mdio: document missed "ti,am4372-mdio" compat string > net: davinci_mdio: introduce "ti,cpsw-mdio" compat string > drivers: net: davinci_mdio: enable pm runtime auto for ti cpsw-mdio > ARM: dts: am335x/am437x/dra7: use new "ti,cpsw-mdio" compat string > > .../devicetree/bindings/net/davinci-mdio.txt | 5 +- > arch/arm/boot/dts/am33xx.dtsi | 2 +- > arch/arm/boot/dts/am4372.dtsi | 2 +- > arch/arm/boot/dts/dra7.dtsi | 2 +- > drivers/net/ethernet/ti/cpsw.c | 79 ++++++++-- > drivers/net/ethernet/ti/davinci_mdio.c | 169 +++++++++++++-------- > 6 files changed, 182 insertions(+), 77 deletions(-) >