Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755588AbbGPBjW (ORCPT ); Wed, 15 Jul 2015 21:39:22 -0400 Received: from mailout2.w1.samsung.com ([210.118.77.12]:26561 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752957AbbGPBiz (ORCPT ); Wed, 15 Jul 2015 21:38:55 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-6c-55a70b2c64bd Message-id: <55A70B2B.1060100@samsung.com> Date: Thu, 16 Jul 2015 10:38:51 +0900 From: Krzysztof Kozlowski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-version: 1.0 To: Bartlomiej Zolnierkiewicz , Thomas Abraham , Sylwester Nawrocki , Michael Turquette , Kukjin Kim , Kukjin Kim , Viresh Kumar Cc: Tomasz Figa , Lukasz Majewski , Heiko Stuebner , Chanwoo Choi , Kevin Hilman , Javier Martinez Canillas , linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Doug Anderson , Andreas Faerber Subject: Re: [PATCH v3 2/4] ARM: dts: Exynos5250: add CPU OPP and regulator supply property References: <1435756238-845-1-git-send-email-b.zolnierkie@samsung.com> <1435756238-845-3-git-send-email-b.zolnierkie@samsung.com> <55A70A3E.5020600@samsung.com> In-reply-to: <55A70A3E.5020600@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrJIsWRmVeSWpSXmKPExsVy+t/xK7o63MtDDSav47Zo3lRssXHGelaL 61+es1qcXXaQzeL/o9esFtd+z2CzeP3C0KJ3wVU2i/7Hr5ktvh5ewWjx5uFmRotNj6+xWnzs ucdqcXnXHDaLz71HGC1mnN/HZHHxlKvF4TftrBYdyxgtVu36w2ix8auHg6jH+xut7B6zGy6y ePyd3crssXPWXXaPTas62TzuXNvD5rF5Sb1H35ZVjB7br81j9th8utrj8ya5AO4oLpuU1JzM stQifbsErox70x4yFazTqGg/0MTewLhPoYuRg0NCwERi3znPLkZOIFNM4sK99WxdjFwcQgJL GSUmL3vIDuE8ZZR4e/wWM0gVr4CWxLmVbUwgNouAqsT7760sIDabgLHE5uVL2EBsUYEIibeX TzJB1AtK/Jh8jwVkkIjAeiaJ3ou3wFYwCzxklvh28Q4rSJWwQKzEtUfzGCHWLWGUmN39Bayd U0BbovH3DHaQW5kF9CTuX9QCCTMLyEtsXvOWeQKjwCwkS2YhVM1CUrWAkXkVo2hqaXJBcVJ6 rqFecWJucWleul5yfu4mRkiEftnBuPiY1SFGAQ5GJR5ejt/LQoVYE8uKK3MPMUpwMCuJ8Hq9 BArxpiRWVqUW5ccXleakFh9ilOZgURLnnbvrfYiQQHpiSWp2ampBahFMlomDU6qBsebAex1R jpsTHiu465jLSqc2sN+2bVxb0u/PVcjNeMDz/LGXUT5cmlfUul9HZfCs1fdZH7bkEWvUvG/+ mTuVBbSafDq5TzjubZZgkvz/VqrrVmbFyZkz1v3xrDB2XXz78sY4Z84Ey8To/+bCbmJhJfcK r9T7rlx9vvQeV3Tqr9PlXD+2zNqgxFKckWioxVxUnAgA9g7bg8wCAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5253 Lines: 163 On 16.07.2015 10:34, Krzysztof Kozlowski wrote: > On 01.07.2015 22:10, Bartlomiej Zolnierkiewicz wrote: >> From: Thomas Abraham >> >> For Exynos5250 platforms, add CPU operating points and CPU >> regulator supply properties for migrating from Exynos specific >> cpufreq driver to using generic cpufreq driver. >> >> Changes by Bartlomiej: >> - split Exynos5250 support from the original patch >> - added CPU regulator supply property for Google Spring board >> - put the new cpu nodes in alphabetical order >> >> Cc: Kukjin Kim >> Cc: Doug Anderson >> Cc: Javier Martinez Canillas >> Cc: Andreas Faerber >> Signed-off-by: Thomas Abraham >> Signed-off-by: Bartlomiej Zolnierkiewicz >> --- >> arch/arm/boot/dts/exynos5250-arndale.dts | 4 ++++ >> arch/arm/boot/dts/exynos5250-smdk5250.dts | 4 ++++ >> arch/arm/boot/dts/exynos5250-snow.dts | 4 ++++ >> arch/arm/boot/dts/exynos5250-spring.dts | 4 ++++ >> arch/arm/boot/dts/exynos5250.dtsi | 22 ++++++++++++++++++++++ >> 5 files changed, 38 insertions(+) >> >> diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts >> index 7e728a1..db3f65f 100644 >> --- a/arch/arm/boot/dts/exynos5250-arndale.dts >> +++ b/arch/arm/boot/dts/exynos5250-arndale.dts >> @@ -117,6 +117,10 @@ >> }; >> }; >> >> +&cpu0 { >> + cpu0-supply = <&buck2_reg>; >> +}; >> + >> &dp { >> status = "okay"; >> samsung,color-space = <0>; >> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts >> index 4fe186d..15aea76 100644 >> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts >> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts >> @@ -74,6 +74,10 @@ >> }; >> }; >> >> +&cpu0 { >> + cpu0-supply = <&buck2_reg>; >> +}; >> + >> &dp { >> samsung,color-space = <0>; >> samsung,dynamic-range = <0>; >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> index b7f4122..a4133df 100644 >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> @@ -235,6 +235,10 @@ >> }; >> }; >> >> +&cpu0 { >> + cpu0-supply = <&buck2_reg>; >> +}; >> + >> &dp { >> status = "okay"; >> pinctrl-names = "default"; >> diff --git a/arch/arm/boot/dts/exynos5250-spring.dts b/arch/arm/boot/dts/exynos5250-spring.dts >> index d03f9b8..c1edd6d 100644 >> --- a/arch/arm/boot/dts/exynos5250-spring.dts >> +++ b/arch/arm/boot/dts/exynos5250-spring.dts >> @@ -65,6 +65,10 @@ >> }; >> }; >> >> +&cpu0 { >> + cpu0-supply = <&buck2_reg>; >> +}; >> + >> &dp { >> status = "okay"; >> pinctrl-names = "default"; >> diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi >> index bf9bee6..0c7ef12 100644 >> --- a/arch/arm/boot/dts/exynos5250.dtsi >> +++ b/arch/arm/boot/dts/exynos5250.dtsi >> @@ -63,6 +63,28 @@ >> compatible = "arm,cortex-a15"; >> reg = <0>; >> clock-frequency = <1700000000>; >> + clocks = <&clock CLK_ARM_CLK>; >> + clock-names = "cpu"; >> + clock-latency = <140000>; >> + >> + operating-points = < >> + 1700000 1300000 >> + 1600000 1250000 >> + 1500000 1225000 >> + 1400000 1200000 >> + 1300000 1150000 >> + 1200000 1125000 >> + 1100000 1100000 >> + 1000000 1075000 >> + 900000 1050000 >> + 800000 1025000 >> + 700000 1012500 >> + 600000 1000000 >> + 500000 975000 >> + 400000 950000 >> + 300000 937500 >> + 200000 925000 >> + >; >> cooling-min-level = <15>; >> cooling-max-level = <9>; >> #cooling-cells = <2>; /* min followed by max */ >> > > > I see you have all necessary acks so this can go through samsung-soc. I > started applying everything. Patch 4/4 does not apply cleanly but this > was easy to solve (I think it is better to rebase on Linus tree not on > next in such case; subsystems cannot apply easily something depending on > next). > > Unfortunately this patch (2/4) has build errors: > > DTC arch/arm/boot/dts/exynos5250-arndale.dtb > Error: ../arch/arm/boot/dts/exynos5250.dtsi:65.21-22 syntax error > FATAL ERROR: Unable to parse input tree > make[2]: *** [arch/arm/boot/dts/exynos5250-arndale.dtb] Error 1 > make[1]: *** [dtbs] Error 2 > make[1]: *** Waiting for unfinished jobs.... > make[1]: Leaving directory `/home/kozik/dev/linux/krzk/out' > make: *** [sub-make] Error 2 > Fail: Make error > > I tried to apply this patch on my dt-for-next branch: > https://github.com/krzk/linux/commits/dt-for-next > > I hold back other patches until this is clarified. Stupid me, it should not be split into separate branches as it depends on clock id from headers. Everything is fine, I applied patchset with respective tags (Javier's reviewed/tested, Sylwester's and Viresh's acks). I'll send later to Kukjin for v4.3 unless he picks it also. Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/