Thread (1 message) 1 message, 1 author, 2018-02-06

Re: [PATCH] ARM: dts: exynos: Use pmu label in exynos4412

From: Krzysztof Kozlowski <hidden>
Date: 2018-02-06 13:25:23
Also in: linux-samsung-soc

On Tue, Feb 6, 2018 at 12:25 PM, Maciej Purski [off-list ref] 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 <redacted>
---
 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

quoted hunk
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help