From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH] ARM: dts: exynos: Use pmu label in exynos4412 Date: Tue, 6 Feb 2018 14:25:23 +0100 Message-ID: References: <1517916352-13614-1-git-send-email-m.purski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <1517916352-13614-1-git-send-email-m.purski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Maciej Purski Cc: linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Kukjin Kim , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Marek Szyprowski , =?UTF-8?B?QmFydMWCb21pZWogxbtvxYJuaWVya2lld2ljeg==?= List-Id: devicetree@vger.kernel.org On Tue, Feb 6, 2018 at 12:25 PM, Maciej Purski wrote: > In order to use soc node, we need to reference soc child nodes by label > in exynos4412. Define pmu label in exynos4.dtsi. Use it in exynos4412 > instead of redefining the node. > > Signed-off-by: Maciej Purski > --- > arch/arm/boot/dts/exynos4.dtsi | 2 +- > arch/arm/boot/dts/exynos4412.dtsi | 7 +++---- > 2 files changed, 4 insertions(+), 5 deletions(-) Looks correct but I am really going to loose the track of your dependencies. You sent this as a separate patch but your v2 base on this. Unless you fix an error here (but then describe the error) this should be probably part of your series. It is okay if series grow with review. Best regards, Krzysztof > diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi > index 6d4775a..da284f3 100644 > --- a/arch/arm/boot/dts/exynos4.dtsi > +++ b/arch/arm/boot/dts/exynos4.dtsi > @@ -162,7 +162,7 @@ > reg = <0x10440000 0x1000>; > }; > > - pmu { > + pmu: pmu { > compatible = "arm,cortex-a9-pmu"; > interrupt-parent = <&combiner>; > interrupts = <2 2>, <3 2>; > diff --git a/arch/arm/boot/dts/exynos4412.dtsi b/arch/arm/boot/dts/exynos4412.dtsi > index a6f3f59..e012cda 100644 > --- a/arch/arm/boot/dts/exynos4412.dtsi > +++ b/arch/arm/boot/dts/exynos4412.dtsi > @@ -546,10 +546,6 @@ > opp-hz = /bits/ 64 <100000000>; > }; > }; > - > - pmu { > - interrupts = <2 2>, <3 2>, <18 2>, <19 2>; > - }; > }; > > &combiner { > @@ -724,6 +720,9 @@ > <&clock CLK_SCLK_HDMI>, <&clock CLK_VP>; > }; > > +&pmu { > + interrupts = <2 2>, <3 2>, <18 2>, <19 2>; > +}; > > &pmu_system_controller { > compatible = "samsung,exynos4412-pmu", "syscon"; > -- > 2.7.4 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html