From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [PATCH v2 00/19] PM / devferq: Add generic exynos bus frequency driver and new passive governor Date: Thu, 10 Dec 2015 13:20:21 +0900 Message-ID: <5668FD85.5090509@samsung.com> References: <1449634091-1842-1-git-send-email-cw00.choi@samsung.com> <5668C382.9030101@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mailout1.samsung.com ([203.254.224.24]:33754 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752392AbbLJEUZ convert rfc822-to-8bit (ORCPT ); Wed, 9 Dec 2015 23:20:25 -0500 In-reply-to: Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Anand Moon Cc: myungjoo.ham@samsung.com, =?UTF-8?B?S3J6eXN6dG9mIEtvesWCb3dza2k=?= , Kukjin Kim , Kyungmin Park , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Tobias Jakobi , Linux Kernel , Linux PM list , "linux-samsung-soc@vger.kernel.org" , devicetree Hi Anand, On 2015=EB=85=84 12=EC=9B=94 10=EC=9D=BC 13:14, Anand Moon wrote: > Hi Chanwoo Choi, >=20 > On 10 December 2015 at 05:42, Chanwoo Choi wr= ote: >> Hi Anand, >> >> First of all, thanks for trying to test this series. >> >> On 2015=EB=85=84 12=EC=9B=94 10=EC=9D=BC 04:05, Anand Moon wrote: >>> Hi Chanwoo Choi, >>> >>> On 9 December 2015 at 09:37, Chanwoo Choi w= rote: >>>> This patch-set includes the two features as following. The generic= exynos bus >>>> frequency driver is able to support almost Exynos SoCs for bus fre= quency >>>> scaling. And the new passive governor is able to make the dependen= cy on >>>> between devices for frequency/voltage scaling. I had posted the pa= tch-set[1] >>>> with the similiar concept. This is is revised version for exynos b= us frequency. >>>> - Generic exynos bus frequency driver >>>> - New passive governor of DEVFREQ framework >>>> >>>> Depends on: >>>> - This patch-set is based on devfreq.git[2]. >>>> [1] https://lkml.org/lkml/2015/1/7/872 >>>> : [PATCHv3 0/8] devfreq: Add generic exynos memory-bus frequenc= y driver >>>> [2] https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/ = (branch: for-rafael) >>>> >>>> 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 >>>> >>>> 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 commo= n >>>> architecture for bus between DRAM and sub-blocks in SoC. >>>> >>>> There are the different buses according to Exynos SoC because Exy= nos SoC >>>> has the differnt sub-blocks and bus speed. In spite of this differ= ence >>>> among Exynos SoCs, this driver is able to support almost Exynos So= C by adding >>>> unique data of each bus in the devicetree file. >>>> >>>> In devicetree, each bus node has a bus clock, regulator, operatio= n-point >>>> and devfreq-event devices which measure the utilization of each bu= s block. >>>> >>>> For example, >>>> - The bus of DMC block in exynos3250.dtsi are listed below: >>>> >>>> bus_dmc: bus_dmc { >>>> compatible =3D "samsung,exynos-bus"; >>>> clocks =3D <&cmu_dmc CLK_DIV_DMC>; >>>> clock-names =3D "bus"; >>>> operating-points-v2 =3D <&bus_dmc_opp_table>; >>>> status =3D "disabled"; >>>> }; >>>> >>>> bus_dmc_opp_table: opp_table0 { >>>> compatible =3D "operating-points-v2"; >>>> opp-shared; >>>> >>>> opp00 { >>>> opp-hz =3D /bits/ 64 <50000000>; >>>> opp-microvolt =3D <800000>; >>>> }; >>>> opp01 { >>>> opp-hz =3D /bits/ 64 <100000000>; >>>> opp-microvolt =3D <800000>; >>>> }; >>>> opp02 { >>>> opp-hz =3D /bits/ 64 <134000000>; >>>> opp-microvolt =3D <800000>; >>>> }; >>>> opp03 { >>>> opp-hz =3D /bits/ 64 <200000000>; >>>> opp-microvolt =3D <800000>; >>>> }; >>>> opp04 { >>>> opp-hz =3D /bits/ 64 <400000000>; >>>> opp-microvolt =3D <875000>; >>>> }; >>>> }; >>>> >>>> - Usage case to handle the frequency and voltage of bus on runtime >>>> in exynos3250-rinato.dts are listed below: >>>> >>>> &bus_dmc { >>>> devfreq-events =3D <&ppmu_dmc0_3>, <&ppmu_dmc1_3>; >>>> vdd-supply =3D <&buck1_reg>; /* VDD_MIF */ >>>> status =3D "okay"; >>>> }; >>>> >>>> 2. Add new passive governor of DEVFREQ framework (patch5-patch7) >>>> : This patch-set add the new passive governor for DEVFREQ framewor= k. >>>> The existing governors (ondemand, performance and so on) are used = for DVFS >>>> (Dynamic Voltage and Frequency Scaling) drivers. The existing gove= rnors >>>> are independently used for specific device driver which don't give= the >>>> influence to other device drviers and also don't receive the effec= t from >>>> other device drivers. >>>> >>>> The passive governor depends on operation of parent driver with e= xisting >>>> governors(ondemand, performance and so on) extremely and is not ab= le to >>>> decide the new frequency by oneself. According to the decided new = frequency >>>> of parent driver with governor, the passive governor uses it to de= cide >>>> the appropriate frequency for own device driver. The passive gover= nor >>>> 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 runtim= e. >>>> But, they share the same power line (e.g., regulator). So, LEFTBUS= bus >>>> driver is operated as parent with ondemand governor and then the r= est >>>> 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 =3D "samsung,exynos-bus"; >>>> clocks =3D <&cmu CLK_DIV_GDL>; >>>> clock-names =3D "bus"; >>>> operating-points-v2 =3D <&bus_leftbus_opp_table>; >>>> status =3D "disabled"; >>>> }; >>>> >>>> bus_rightbus: bus_rightbus { >>>> compatible =3D "samsung,exynos-bus"; >>>> clocks =3D <&cmu CLK_DIV_GDR>; >>>> clock-names =3D "bus"; >>>> operating-points-v2 =3D <&bus_leftbus_opp_table>; >>>> status =3D "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 =3D <&ppmu_leftbus_3>, <&ppmu_right= bus_3>; >>>> vdd-supply =3D <&buck3_reg>; >>>> status =3D "okay"; >>>> }; >>>> >>>> /* Passive bus device depend on LEFTBUS bus. */ >>>> &bus_rightbus { >>>> devfreq =3D <&bus_leftbus>; /* 'devfreq' property = indicates >>>> the phandle of parent= device. */ >>>> status =3D "okay"; >>>> }; >>>> >>>> (Omit the rest bus dt node) >>>> >>>> Chanwoo Choi (19): >>>> PM / devfreq: exynos: Add generic exynos bus frequency driver >>>> PM / devfreq: exynos: Add documentation for generic exynos bus f= requency driver >>>> ARM: dts: Add DMC bus node for Exynos3250 >>>> ARM: dts: Add DMC bus frequency for exynos3250-rinato/monk >>>> PM / devfreq: Add new passive governor >>>> PM / devfreq: Add devfreq_get_devfreq_by_phandle() >>>> PM / devfreq: Show the related information according to governor= type >>>> 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-b= locks and power line >>>> PM / devfreq: exynos: Remove unused exynos4/5 busfreq driver >>>> 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 PPMU node for exynos4412-odroidu3 >>>> ARM: dts: Add support of bus frequency using VDD_INT for exynos3= 250-rinato >>>> ARM: dts: Expand the voltage range of buck1/3 regulator for exyn= os4412-odroidu3 >>>> ARM: dts: Add support of bus frequency for exynos4412-trats/odro= idu3 >>>> >>>> .../devicetree/bindings/devfreq/exynos-bus.txt | 383 +++++++ >>>> arch/arm/boot/dts/exynos3250-monk.dts | 6 + >>>> arch/arm/boot/dts/exynos3250-rinato.dts | 47 + >>>> arch/arm/boot/dts/exynos3250.dtsi | 194 ++++ >>>> arch/arm/boot/dts/exynos4210.dtsi | 172 ++++ >>>> arch/arm/boot/dts/exynos4412-odroid-common.dtsi | 93 +- >>>> arch/arm/boot/dts/exynos4412-trats2.dts | 47 + >>>> arch/arm/boot/dts/exynos4x12.dtsi | 184 ++++ >>>> drivers/devfreq/Kconfig | 37 +- >>>> drivers/devfreq/Makefile | 2 + >>>> drivers/devfreq/devfreq.c | 120 ++- >>>> drivers/devfreq/exynos/Makefile | 3 +- >>>> drivers/devfreq/exynos/exynos-bus.c | 549 +++++++= +++ >>>> 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.h | 7 + >>>> drivers/devfreq/governor_passive.c | 109 ++ >>>> drivers/devfreq/governor_performance.c | 1 + >>>> drivers/devfreq/governor_powersave.c | 1 + >>>> drivers/devfreq/governor_simpleondemand.c | 1 + >>>> drivers/devfreq/governor_userspace.c | 1 + >>>> include/linux/devfreq.h | 28 + >>>> 25 files changed, 1958 insertions(+), 1828 deletions(-) >>>> create mode 100644 Documentation/devicetree/bindings/devfreq/exyn= os-bus.txt >>>> create mode 100644 drivers/devfreq/exynos/exynos-bus.c >>>> 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 >>>> >>>> -- >>>> 1.9.1 >>>> >>> >>> I could not get this series to work with my Odroid U3. >>> >>> [ 4.602768] input: gpio_keys as /devices/platform/gpio_keys/inpu= t/input0 >>> [ 4.605527] devfreq bus_leftbus: Couldn't update frequency >>> transition information. >>> [ 4.607319] devfreq bus_dmc: Couldn't update frequency transitio= n >>> information. >>> [ 4.625096] usb 1-3: New USB device found, idVendor=3D0424, idPr= oduct=3D3503 >> >> This log indicates the problem of 'trats_stat' sysfs entry of devfre= q framework >> during kernel booting. But, this log don't affect the behavior of bu= s frequency >> on Odroid-U3. >> >> After completing kernel and platform booting, you can check the oper= ation >> of Bus frequency with follwoing sysfs entry: >> >> root@localhost:~# ls -al /sys/class/devfreq >> total 0 >> drwxr-xr-x 2 root root 0 Dec 31 17:00 . >> drwxr-xr-x 44 root root 0 Dec 31 17:00 .. >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_acp -> ../../devices/plat= form/bus_acp/devfreq/bus_acp >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_c2c -> ../../devices/plat= form/bus_c2c/devfreq/bus_c2c >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_display -> ../../devices/= platform/bus_display/devfreq/bus_display >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_dmc -> ../../devices/plat= form/bus_dmc/devfreq/bus_dmc >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_fsys -> ../../devices/pla= tform/bus_fsys/devfreq/bus_fsys >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_leftbus -> ../../devices/= platform/bus_leftbus/devfreq/bus_leftbus >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_mfc -> ../../devices/plat= form/bus_mfc/devfreq/bus_mfc >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_peri -> ../../devices/pla= tform/bus_peri/devfreq/bus_peri >> lrwxrwxrwx 1 root root 0 Dec 31 17:00 bus_rightbus -> ../../devices= /platform/bus_rightbus/devfreq/bus_rightbus >> >> root@localhost:~# cat /sys/class/devfreq/bus_leftbus/trans_stat >> From : To >> : 100000000 134000000 160000000 200000000 time(ms) >> 100000000: 0 0 0 24 2020 >> 134000000: 0 0 0 10 1190 >> 160000000: 0 0 0 0 0 >> * 200000000: 24 10 0 0 118160 >> Total transition : 68 >> >> Also, when changing the frequency of each bus, exynos-bus.c show >> the debug message. If you need more detailed information, >> you could check this log message. >> >> Regards, >> Chanwoo Choi >=20 > Thanks for you input. It seem to be working I have see the counter in= crement. > Sorry for the noise. No problem. >=20 > root@odroidu3:~# cat /sys/class/devfreq/bus_dmc/trans_stat > From : To > : 100000000 134000000 160000000 200000000 400000000 time= (ms) > 100000000: 0 0 0 0 2 = 560 > 134000000: 0 0 0 0 1 = 190 > 160000000: 0 0 0 0 0 = 0 > 200000000: 0 0 0 0 2 = 310 > * 400000000: 2 1 0 2 0 332= 400 > Total transition : 10 > root@odroidu3:~# cat /sys/class/devfreq/bus_rightbus/trans_stat > Not Supported. > root@odroidu3:~# >=20 > Tested on Odroid U3 > Tested-by: Anand Moon Thanks for your test. Best Regards, Chanwoo Choi