Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752502AbcDKEB1 (ORCPT ); Mon, 11 Apr 2016 00:01:27 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:60101 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752387AbcDKEBW convert rfc822-to-8bit (ORCPT ); Mon, 11 Apr 2016 00:01:22 -0400 X-AuditID: cbfee68e-f79d96d0000012b1-9f-570b218fb1dd MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 Content-transfer-encoding: 8BIT Message-id: <570B218E.5000908@samsung.com> Date: Mon, 11 Apr 2016 13:01:18 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: myungjoo.ham@samsung.com, k.kozlowski@samsung.com, kgene@kernel.org Cc: kyungmin.park@samsung.com, s.nawrocki@samsung.com, tomasz.figa@gmail.com, rjw@rjwysocki.net, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux@arm.linux.org.uk, linux.amoon@gmail.com, m.reichl@fivetechno.de, tjakobi@math.uni-bielefeld.de, inki.dae@samsung.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: Re: [PATCH v9 00/20] PM / devferq: Add generic exynos bus frequency driver and new passive governor References: <1460347078-15175-1-git-send-email-cw00.choi@samsung.com> In-reply-to: <1460347078-15175-1-git-send-email-cw00.choi@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrEKsWRmVeSWpSXmKPExsWyRsSkQLdfkTvc4OgKMYv5R86xWvS/Wchq ce7VSkaLSfcnsFi8fmFo0f/4NbPF2aY37BabHl9jtbi8aw6bxefeI4wWM87vY7JYt/EWu8Xt y7wWL4/8YLRYev0ik8XtxhVsFhOmr2WxOHP6EqtF694j7BaH37SzWrSt/sBqsWrXH0YHMY81 89YwerQ097B5XO7rZfK4dafeY+esu+weK5d/YfPYtKqTzWPzknqPf8fYPbZcbWfx6NuyitHj 8ya5AJ4oLpuU1JzMstQifbsEroyuRztYC35lVeydZ9HA+CO4i5GTQ0LAROLtmxYmCFtM4sK9 9WxdjFwcQgIrGCVevp8IlOAAK7o1zxMiPotR4ufGPWANvAKCEj8m32MBsZkF1CUmzVvEDGGL Sizqv8IIYWtLLFv4mhmi+QGjxMPl2xkhmrUk/vZsBbNZBFQlZs4+DGazAcX3v7jBBrJYVCBC ovtEJUhYRMBd4uu93WDHMQv8ZJa4f/gvO0hCWCBP4v+CGcwg9UICrhLPPteBhDkF3CQ+rW9n BamXEPjCIfGvu40NYpeAxLfJh1ggHpOV2HSAGeJ5SYmDK26wTGAUn4XktVlIXpuF5LVZSF5b wMiyilE0tSC5oDgpvchIrzgxt7g0L10vOT93EyMw6Zz+96xvB+PNA9aHGAU4GJV4eB2ucYUL sSaWFVfmHmI0BbpoIrOUaHI+MLXllcQbGpsZWZiamBobmVuaKYnzJkj9DBYSSE8sSc1OTS1I LYovKs1JLT7EyMTBKdXAaGrFs+NmxcxkS48FRUW3HjzLv7ueh9V42eGjJz89FL7w81qX95+U OWbeOls7hSrDX31/8H9e86OAkCPiYYWSHxw6madmferuZ3gVa3zHevW3I8xbk7J/mAb3rtjA udrP13c/b/uJ+3f+fjqyjXey/trss1tk57JWrt7s/vzG19DQq5NnTcjUWaXEUpyRaKjFXFSc CAC8uxHkNQMAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrGKsWRmVeSWpSXmKPExsVy+t9jQd1+Re5wg/OTeS3mHznHatH/ZiGr xblXKxktJt2fwGLx+oWhRf/j18wWZ5vesFtsenyN1eLyrjlsFp97jzBazDi/j8li3cZb7Ba3 L/NavDzyg9Fi6fWLTBa3G1ewWUyYvpbF4szpS6wWrXuPsFscftPOatG2+gOrxapdfxgdxDzW zFvD6NHS3MPmcbmvl8nj1p16j52z7rJ7rFz+hc1j06pONo/NS+o9/h1j99hytZ3Fo2/LKkaP z5vkAniiGhhtMlITU1KLFFLzkvNTMvPSbZW8g+Od403NDAx1DS0tzJUU8hJzU22VXHwCdN0y c4AeVlIoS8wpBQoFJBYXK+nbYZoQGuKmawHTGKHrGxIE12NkgAYS1jBm/Fm3nblgUVbFjo2T mRoYFwR3MXJwSAiYSNya59nFyAlkiklcuLeerYuRi0NIYBajxM+Ne5hAErwCghI/Jt9jAaln FpCXOHIpGyTMLKAuMWneImaI+geMEg+Xb2eEqNeS+NuzFcxmEVCVmDn7MJjNBhTf/+IGG8gc UYEIie4TlSBhEQF3ia/3doPtZRb4ySxx//BfdpCEsECexP8FM5hB6oUEXCWefa4DCXMKuEl8 Wt/OOoER6EiE62YhXDcLyXULGJlXMUqkFiQXFCel5xrlpZbrFSfmFpfmpesl5+duYgSnqWfS OxgP73I/xCjAwajEw/viMle4EGtiWXFl7iFGCQ5mJRHefAXucCHelMTKqtSi/Pii0pzU4kOM pkDvTWSWEk3OB6bQvJJ4Q2MTMyNLI3NDCyNjcyVx3sf/14UJCaQnlqRmp6YWpBbB9DFxcEo1 MB42+HT10HpvsSXBHx5+2nfb7qZM2AGpY1t5g+qEt8uvLN+e1f7sZRHT1mkabvduTFJ7Us/F ovvp8zt3x5VHD2/WFLZq05HR7tTWZ78e8lf3haVxC/9PU+7fd3td9BY4M0ueNbI+bfrg16zH Vyt8bgfMtJqVGXTtw3XDBZ9cH5t8mKudtvGVc7USS3FGoqEWc1FxIgBGIACoaQMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 13299 Lines: 301 Dear Myungjoo and Krzysztof, This patch-set already get the acked-by or signed-off-by tag from Exynos SoC and DEVFREQ maintainer except for patch10. If there is no comment, could you pick this patch-set? Best Regards, Chanwoo Choi On 2016년 04월 11일 12:57, Chanwoo Choi wrote: > This patch-set includes the two features as following. The generic exynos bus > frequency driver is able to support almost Exynos SoCs for bus frequency > scaling. And the new passive governor is able to make the dependency on > between devices for frequency/voltage scaling. I had posted the patch-set[1] > with the similiar concept. This is is revised version for exynos bus frequency. > - Generic exynos bus frequency driver > - New passive governor of DEVFREQ framework > [1] https://lkml.org/lkml/2015/1/7/872 > : [PATCHv3 0/8] devfreq: Add generic exynos memory-bus frequency driver > > Changes from v8: > (https://lkml.org/lkml/2016/4/8/14) > - Fix possible recursive locking issue by using mutex_lock_nested() > > Changes from v7: > (https://lkml.org/lkml/2016/3/31/308) > - Use IS_ERR() instead of IS_ERR_OR_NULL() macro > - Add signed-off-by/acked-by tag of DEVFREQ maintainer > - Fix the comment in governor_passive.c > > Changes from v6: > (https://lkml.org/lkml/2016/3/27/187) > - Patch1/2 add signed-off-by tag from DEVFREQ Maintainer (Myungjoo Ham) > - Drop patch5 (PM / devfreq: Add governer type with unique number) > - Add missing description about the transition_notifier_list in struct devfreq > - Remove the passive governor code from devfreq.c core driver. Instead, passive > governor uses the DEVFREQ_GOV_START/STOP signal to register/unregister the > DEVFREQ_TRANSITION_NOTIFIER between the parent devfreq device and passive > devfreq device. > - Add the author (myungjoo.ham@samsung.com) of passive governor > - Add 'struct devfreq_passive_data' to pass the devfreq instance of parent. > > Changes from v5: > (https://lkml.org/lkml/2016/3/24/5) > - Rebase the patch-set on Linux v4.6-rc1 > - Add Tested-by tag from Markus Reichl > - Add Tested-by tag from Anand Moon > > Changes from v4: > (https://lkml.org/lkml/2015/12/14/43) > - Add new DEVFREQ_TRANSITION_NOTIFIER notifier. The passive > devfreq device recevie the changed frequency of parent > devfreq device through DEVFREQ_TRANSITION_NOTIFIER. > - Add governor type to identify thme using the defined constant > - Modify the passive governor using the DEVFREQ_TRANSITION_NOTIFIER notifier. > - Fix the RCU locking probrlm (Reported-by: Tobias Jakobi) > - Fix the debugfs error during the kernel booting (Reported-by: Tobias Jakobi) > - The Device Tree patches have got Reviewed-by tag from Exynos SoC Maintainer > (Krzysztof Kozlowski ) > > Changes from v3: > (https://lkml.org/lkml/2015/12/11/75) > - Add the reviewed-by tag from Krzysztof Kozlowski (patch2/3/13/14/15/16/17) > - Fix typo of the description on patch14 > - Modify the subject and description of patch17 > - Reorder the 'bus_xxx' device tree node alphabetically in > both exynos3250-rinato/monk.dts and exynos4412-trats/odroidu3 > > Changes from v2: > (https://lkml.org/lkml/2015/12/8/869) > - Fix typo on documentation > - Modify the more appropriate sentence on patch description > - Add the detailed description about both parent and passive bus device > - Modify the DMC frequency for Exynos4x12 DMC bus (200MHz -> 267MHz) > - Modify the voltage of 200MHz was included in Exynos3250 DMC bus (800mV -> 825mV) > - Rename OPP nodes as 'opp@' > - Delete the duplicate 'opp-microvolt' property of passive devfreq device > - Reorder the 'bus_xxx' device tree node alphabetically in exynos3250-rinato/monk.dts > - Reorder the 'bus_xxx' device tree node alphabetically in exynos4412-trats/odroidu3 > - Add new exynos4412-ppmu-common.dtsi to remove the duplicate PPMU dt node > on rinato/monk/trats2/odroid-u3 board > - Add the log message if bus device is registered to devfreq framework successfully > - Add the reviewed-by tag from Krzysztof Kozlowski > - Add the tested-by tag from Anand Moon on Odroid U3 > - Add 'SAMSUNG BUS FREQUENCY DRIVER' entry to MAINTAINERS > > Changes from v1: > (https://lkml.org/lkml/2015/11/26/260) > - Check whether the instance of regulator is NULL or not > when executing regulator_disable() because of only parent > devfreq device has the regulator instance. After fixing it, > the wake-up from suspend state is well working. (patch1) > - Fix bug which checks 'bus-clk' instead of 'bus->regulator' > after calling devm_clk_get() (on patch1) > - Update the documentation to remove the description about > DEVFREQ-EVENT subsystem (on patch2) > - Add the full name of DMC (Dynamic Memory Controller) (on patch2) > - Modify the detailed correlation of buses for Exynos3250 > on documentation (patch2) > - Add the MFC bus node for Exynos3250 (on patch11, patch12) > - Fix the duplicate frequency of bus_display on Exynos4x12.dtsi > - Add the PPMU node for exynos4412-odroidu3 > - Add the support of bus frequency for exynos4412-odroidu3 > > > [Description] > Detailed descirption for patch-set: > 1. Add generic exynos bus frequency driver > : This patch-set adds the generic exynos bus frequency driver for AXI bus > of sub-blocks in exynos SoC. The Samsung Exynos SoC have the common > architecture for bus between DRAM and sub-blocks in SoC. > > There are the different buses according to Exynos SoC because Exynos SoC > has the differnt sub-blocks and bus speed. In spite of this difference > among Exynos SoCs, this driver is able to support almost Exynos SoC by adding > unique data of each bus in the devicetree file. > > In devicetree, each bus node has a bus clock, regulator, operation-point > and devfreq-event devices which measure the utilization of each bus block. > > For example, > - The bus of DMC block in exynos3250.dtsi are listed below: > > bus_dmc: bus_dmc { > compatible = "samsung,exynos-bus"; > clocks = <&cmu_dmc CLK_DIV_DMC>; > clock-names = "bus"; > operating-points-v2 = <&bus_dmc_opp_table>; > status = "disabled"; > }; > > bus_dmc_opp_table: opp_table1 { > compatible = "operating-points-v2"; > opp-shared; > > opp@50000000 { > opp-hz = /bits/ 64 <50000000>; > opp-microvolt = <800000>; > }; > opp@100000000 { > opp-hz = /bits/ 64 <100000000>; > opp-microvolt = <800000>; > }; > opp@134000000 { > opp-hz = /bits/ 64 <134000000>; > opp-microvolt = <800000>; > }; > opp@200000000 { > opp-hz = /bits/ 64 <200000000>; > opp-microvolt = <825000>; > }; > opp@400000000 { > opp-hz = /bits/ 64 <400000000>; > opp-microvolt = <875000>; > }; > }; > > - Usage case to handle the frequency and voltage of bus on runtime > in exynos3250-rinato.dts are listed below: > > &bus_dmc { > devfreq-events = <&ppmu_dmc0_3>, <&ppmu_dmc1_3>; > vdd-supply = <&buck1_reg>; /* VDD_MIF */ > status = "okay"; > }; > > 2. Add new passive governor of DEVFREQ framework > : This patch-set add the new passive governor for DEVFREQ framework. > The existing governors (ondemand, performance and so on) are used for DVFS > (Dynamic Voltage and Frequency Scaling) drivers. The existing governors > are independently used for specific device driver which don't give the > influence to other device drviers and also don't receive the effect from > other device drivers. > > The passive governor depends on operation of parent driver with existing > governors(ondemand, performance and so on) extremely and is not able to > decide the new frequency by oneself. According to the decided new frequency > of parent driver with governor, the passive governor uses it to decide > the appropriate frequency for own device driver. The passive governor > must need the following information from device tree: > > For exameple, > There are one more bus device drivers in Exynos3250 which need to > change their source clock according to their utilization on runtime. > But, they share the same power line (e.g., regulator). So, LEFTBUS bus > driver is operated as parent with ondemand governor and then the rest > device driver with passive governor. > > The buses of Internal block in exynos3250.dtsi are listed below: > When LEFTBUS bus driver (parent) changes the bus frequency with > ondemand governor on runtime, the rest bus devices which sharing > the same power line (VDD_INT) will change the each bus frequency > according to the decision of LEFTBUS bus driver (parent). > > - INT (Internal) block > : VDD_INT |--- LEFTBUS > |--- PERIL > |--- MFC > |--- G3D > |--- RIGHTBUS > |--- FSYS > |--- LCD0 > |--- PERIR > |--- ISP > |--- CAM > > - The buss of INT block in exynos3250.dtsi are listed below: > bus_leftbus: bus_leftbus { > compatible = "samsung,exynos-bus"; > clocks = <&cmu CLK_DIV_GDL>; > clock-names = "bus"; > operating-points-v2 = <&bus_leftbus_opp_table>; > status = "disabled"; > }; > > bus_rightbus: bus_rightbus { > compatible = "samsung,exynos-bus"; > clocks = <&cmu CLK_DIV_GDR>; > clock-names = "bus"; > operating-points-v2 = <&bus_leftbus_opp_table>; > status = "disabled"; > }; > > (Omit the rest bus dt node) > > - Usage case to handle the frequency and voltage of bus on runtime > in exynos3250-rinato.dts are listed below: > /* Parent bus device of VDD_INT */ > &bus_leftbus { > devfreq-events = <&ppmu_leftbus_3>, <&ppmu_rightbus_3>; > vdd-supply = <&buck3_reg>; > status = "okay"; > }; > > /* Passive bus device depend on LEFTBUS bus. */ > &bus_rightbus { > devfreq = <&bus_leftbus>; /* 'devfreq' property indicates > the phandle of parent device. */ > status = "okay"; > }; > > (Omit the rest bus dt node) > > > Chanwoo Choi (20): > PM / devfreq: exynos: Add generic exynos bus frequency driver > PM / devfreq: exynos: Add documentation for generic exynos bus frequency driver > PM / devfreq: Add devfreq_get_devfreq_by_phandle() > PM / devfreq: Add new DEVFREQ_TRANSITION_NOTIFIER notifier > PM / devfreq: Add new passive governor > PM / devfreq: exynos: Add support of bus frequency of sub-blocks using passive governor > PM / devfreq: exynos: Update documentation for bus devices using passive governor > PM / devfreq: exynos: Add the detailed correlation between sub-blocks and power line > PM / devfreq: exynos: Remove unused exynos4/5 busfreq driver > MAINTAINERS: Add samsung bus frequency driver entry > ARM: dts: Add DMC bus node for Exynos3250 > ARM: dts: Add DMC bus frequency for exynos3250-rinato/monk > ARM: dts: Add bus nodes using VDD_INT for Exynos3250 > ARM: dts: Add bus nodes using VDD_MIF for Exynos4x12 > ARM: dts: Add bus nodes using VDD_INT for Exynos4x12 > ARM: dts: Add bus nodes using VDD_MIF for Exynos4210 > ARM: dts: Add exynos4412-ppmu-common dtsi to delete duplicate PPMU nodes > ARM: dts: Add support of bus frequency using VDD_INT for exynos3250-rinato > ARM: dts: Expand the voltage range of buck1/3 regulator for exynos4412-odroidu3 > ARM: dts: Add support of bus frequency for exynos4412-trats/odroidu3 > .../devicetree/bindings/devfreq/exynos-bus.txt | 390 ++++++++ > MAINTAINERS | 9 + > arch/arm/boot/dts/exynos3250-monk.dts | 47 +- > arch/arm/boot/dts/exynos3250-rinato.dts | 88 +- > arch/arm/boot/dts/exynos3250.dtsi | 181 ++++ > arch/arm/boot/dts/exynos4210.dtsi | 159 +++ > arch/arm/boot/dts/exynos4412-odroid-common.dtsi | 56 +- > arch/arm/boot/dts/exynos4412-ppmu-common.dtsi | 50 + > arch/arm/boot/dts/exynos4412-trats2.dts | 88 +- > arch/arm/boot/dts/exynos4x12.dtsi | 174 ++++ > drivers/devfreq/Kconfig | 36 +- > drivers/devfreq/Makefile | 4 +- > drivers/devfreq/devfreq.c | 207 +++- > drivers/devfreq/exynos-bus.c | 570 +++++++++++ > drivers/devfreq/exynos/Makefile | 3 - > drivers/devfreq/exynos/exynos4_bus.c | 1055 -------------------- > drivers/devfreq/exynos/exynos4_bus.h | 110 -- > drivers/devfreq/exynos/exynos5_bus.c | 431 -------- > drivers/devfreq/exynos/exynos_ppmu.c | 119 --- > drivers/devfreq/exynos/exynos_ppmu.h | 86 -- > drivers/devfreq/governor_passive.c | 207 ++++ > include/linux/devfreq.h | 99 ++ > 22 files changed, 2221 insertions(+), 1948 deletions(-) > create mode 100644 Documentation/devicetree/bindings/devfreq/exynos-bus.txt > create mode 100644 arch/arm/boot/dts/exynos4412-ppmu-common.dtsi > create mode 100644 drivers/devfreq/exynos-bus.c > delete mode 100644 drivers/devfreq/exynos/Makefile > delete mode 100644 drivers/devfreq/exynos/exynos4_bus.c > delete mode 100644 drivers/devfreq/exynos/exynos4_bus.h > delete mode 100644 drivers/devfreq/exynos/exynos5_bus.c > delete mode 100644 drivers/devfreq/exynos/exynos_ppmu.c > delete mode 100644 drivers/devfreq/exynos/exynos_ppmu.h > create mode 100644 drivers/devfreq/governor_passive.c >