Re: [PATCH v4 2/4] ARM: dts: Add SROMc to Exynos 5410

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Pavel,

On Friday 30 October 2015 12:11 PM, Pavel Fedin wrote:
  Hello!

-----Original Message-----
From: linux-samsung-soc-owner@xxxxxxxxxxxxxxx [mailto:linux-samsung-soc-owner@xxxxxxxxxxxxxxx]
On Behalf Of Pankaj Dubey
Sent: Thursday, October 29, 2015 8:28 PM
To: Pavel Fedin
Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc;
linux-kernel@xxxxxxxxxxxxxxx; Rob Herring; Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala;
Kukjin Kim; Krzysztof Kozlowski
Subject: Re: [PATCH v4 2/4] ARM: dts: Add SROMc to Exynos 5410

Hi Pavel,

On 29 October 2015 at 18:12, Pavel Fedin <p.fedin@xxxxxxxxxxx> wrote:
This machine uses own SoC device tree file, add missing part.

Signed-off-by: Pavel Fedin <p.fedin@xxxxxxxxxxx>
---
  arch/arm/boot/dts/exynos5410.dtsi | 15 +++++++++++++++
  1 file changed, 15 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5410.dtsi b/arch/arm/boot/dts/exynos5410.dtsi
index 4603356..da6a8fa0e 100644
--- a/arch/arm/boot/dts/exynos5410.dtsi
+++ b/arch/arm/boot/dts/exynos5410.dtsi
@@ -101,6 +101,15 @@
                         reg = <0x10000000 0x100>;
                 };

+               sromc: sromc@12250000 {
+                       #address-cells = <1>;
+                       #size-cells = <1>;
+                       ranges;
+

We do not need to specify these three properties as they are already
present in parent node "soc".

  We do, otherwise dtc complains about defaults of #address-cells = 2 and #size-cells=1, and without empty "ranges" subnode's resources are not correctly translated.


First of all this patch will not give this error. So this part should not be a part of this patch. You should be getting above error after applying v4 4/4 "ARM: dts: Add Ethernet chip to SMDK5410". So if its failing for ethernet subnode, you can move this address-cells and size-cells in dts file just above ethernet node as shown below:

index 311e7be..d69981d 100644
--- a/arch/arm/boot/dts/exynos5410-smdk5410.dts
+++ b/arch/arm/boot/dts/exynos5410-smdk5410.dts
@@ -95,6 +95,9 @@
 };

 &sromc {
+       #address-cells = <1>;
+       #size-cells = <1>;
+       ranges;
        pinctrl-names = "default";
        pinctrl-0 = <&srom_ctl>, <&srom_ebi>;

And another question still remains open, we can't just like that change "smsc,lan9115" binding by adding samsung specific properties. If you want to do and there is no other way, you need to update DT binding of "smsc,lan9115" and get it reviewed. Probably you can check suggestions from Krzysztof, where he pointed out some hint on how other places this is getting used.


Thanks,
Pankaj Dubey

+                       compatible = "samsung,exynos-srom";
+                       reg = <0x12250000 0x14>;
+               };
+
                 pmu_system_controller: system-controller@10040000 {
                         compatible = "samsung,exynos5410-pmu", "syscon";
                         reg = <0x10040000 0x5000>;
@@ -133,6 +142,12 @@
                                                 <10 &gic 0 130 0>,
                                                 <11 &gic 0 131 0>;
                         };
+
+                       arch_timer {
+                               compatible = "arm,armv7-timer";
+                               clock-frequency = <24000000>;
+                       };
+

This change should not be part of this patch.

  Ooops, thank you very much, this should not have been here at all. This is a leftover from my experiments, i was tracing DT parsing code and added it just for test, to see why timer gets probed as a subnode.
  Just forgot to remove it afterwards and it slipped into the patch.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux