Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755436AbbGPBfE (ORCPT ); Wed, 15 Jul 2015 21:35:04 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:27704 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932698AbbGPBe7 (ORCPT ); Wed, 15 Jul 2015 21:34:59 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-78-55a70a3fead0 Message-id: <55A70A3E.5020600@samsung.com> Date: Thu, 16 Jul 2015 10:34:54 +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> In-reply-to: <1435756238-845-3-git-send-email-b.zolnierkie@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrFIsWRmVeSWpSXmKPExsVy+t/xq7r2XMtDDWbus7Bo3lRssXHGelaL 61+es1qcXXaQzeL/o9esFtd+z2CzeP3C0KJ3wVU2i/7Hr5ktvh5ewWjx5uFmRotNj6+xWnzs ucdqcXnXHDaLz71HGC1mnN/HZHHxlKvF4TftrBYdyxgtVu36w2ix8auHg6jH+xut7B6zGy6y ePyd3crssXPWXXaPTas62TzuXNvD5rF5Sb1H35ZVjB7br81j9th8utrj8ya5AO4oLpuU1JzM stQifbsErozLM9YyFuxRqVh36BVLA+Nc2S5GTg4JAROJDQ3rWCBsMYkL99azdTFycQgJLGWU +LxjMpTzlFHi+4pl7CBVvAJaEm+evACzWQRUJU6v+8wGYrMJGEtsXr4EzBYViJB4e/kkE0S9 oMSPyfdYQAaJCKxnkui9eAtsKrPAQ2aJbxfvsIJUCQvESlx7NI8RYl0zo8S7vxPAjuIUcJfY 960dqIMDqENP4v5FLZAws4C8xOY1b5knMArMQrJkFkLVLCRVCxiZVzGKppYmFxQnpeca6hUn 5haX5qXrJefnbmKExOiXHYyLj1kdYhTgYFTi4eX4vSxUiDWxrLgy9xCjBAezkgiv10ugEG9K YmVValF+fFFpTmrxIUZpDhYlcd65u96HCAmkJ5akZqemFqQWwWSZODilGhhTl55e9cZ23sEL m3dlLpmpfEjkQYr27hbLtmu2Uxf/spR/nlwvGrijo0dY8mux//3cB2o3ZitYvL380/b9i44f ys+Fe2Xmn5ro+2bDvB2ra8s6I+VPnriyqWmFw5cl3OI5Mzeum3hkjectZZlmg4ja3T27N82Y U7zlpljCncoYdW1/Z70JRWUrlFiKMxINtZiLihMBSlTDSs0CAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4763 Lines: 155 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. 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/