Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1474197 > unrolled thread

[PATCH v2 0/7] ARM: dts: exynos: Remove skeleton.dtsi usage and fix memory node DTC warnings

Started byJavier Martinez Canillas <javier@osg.samsung.com>
First post2016-09-01 11:10 +0200
Last post2016-09-01 18:40 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v2 0/7] ARM: dts: exynos: Remove skeleton.dtsi usage and fix memory node DTC warnings Javier Martinez Canillas <javier@osg.samsung.com> - 2016-09-01 11:10 +0200
    [PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS Javier Martinez Canillas <javier@osg.samsung.com> - 2016-09-01 11:10 +0200
      Re: [PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory  nodes in Exynos4 DTS Krzysztof Kozlowski <krzk@kernel.org> - 2016-09-01 18:40 +0200

#1474197 — [PATCH v2 0/7] ARM: dts: exynos: Remove skeleton.dtsi usage and fix memory node DTC warnings

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-09-01 11:10 +0200
Subject[PATCH v2 0/7] ARM: dts: exynos: Remove skeleton.dtsi usage and fix memory node DTC warnings
Message-ID<scwat-7KC-5@gated-at.bofh.it>
Hello Krzysztof,

This series removes the usage of the skeleton.dtsi in all the Exynos dts,
which allows to get rid of the DTC warnings about a mismatch between the
memory nodes' unit names and reg properties. This is a v2 of the series,
v1 can be found at https://lkml.org/lkml/2016/8/31/395.

Patches are pretty trivial and shouldn't cause functional changes AFAIK,
but only the Exynos5 changes have been tested. The others patches were
just built tested.

Best regards,
Javier

Changes in v2:
- Add a fixme to specify that the correct memory values should be set.
  (Bartlomiej Zolnierkiewicz)
- Fix commit message (Krzysztof Kozlowski).

Javier Martinez Canillas (7):
  ARM: dts: exynos: Add missing memory node for Exynos5440 boards
  ARM: dts: exynos: Remove skeleton.dtsi usage for Exynos3
  ARM: dts: exynos: Remove skeleton.dtsi usage for Exynos4
  ARM: dts: exynos: Remove skeleton.dtsi usage for Exynos5
  ARM: dts: exynos: Add missing unit name to memory nodes in Exynos3 DTS
  ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS
  ARM: dts: exynos: Add missing unit name to memory nodes in Exynos5 DTS

 arch/arm/boot/dts/exynos3250-artik5.dtsi           | 3 ++-
 arch/arm/boot/dts/exynos3250-monk.dts              | 3 ++-
 arch/arm/boot/dts/exynos3250-rinato.dts            | 3 ++-
 arch/arm/boot/dts/exynos3250.dtsi                  | 3 ++-
 arch/arm/boot/dts/exynos4.dtsi                     | 3 ++-
 arch/arm/boot/dts/exynos4210-origen.dts            | 3 ++-
 arch/arm/boot/dts/exynos4210-smdkv310.dts          | 3 ++-
 arch/arm/boot/dts/exynos4210-trats.dts             | 3 ++-
 arch/arm/boot/dts/exynos4210-universal_c210.dts    | 3 ++-
 arch/arm/boot/dts/exynos4412-odroidu3.dts          | 3 ++-
 arch/arm/boot/dts/exynos4412-odroidx.dts           | 3 ++-
 arch/arm/boot/dts/exynos4412-odroidx2.dts          | 3 ++-
 arch/arm/boot/dts/exynos4412-origen.dts            | 3 ++-
 arch/arm/boot/dts/exynos4412-smdk4412.dts          | 3 ++-
 arch/arm/boot/dts/exynos4412-tiny4412.dts          | 3 ++-
 arch/arm/boot/dts/exynos4412-trats2.dts            | 3 ++-
 arch/arm/boot/dts/exynos4415.dtsi                  | 3 ++-
 arch/arm/boot/dts/exynos5.dtsi                     | 3 ++-
 arch/arm/boot/dts/exynos5250-arndale.dts           | 3 ++-
 arch/arm/boot/dts/exynos5250-smdk5250.dts          | 3 ++-
 arch/arm/boot/dts/exynos5250-snow-common.dtsi      | 3 ++-
 arch/arm/boot/dts/exynos5250-spring.dts            | 3 ++-
 arch/arm/boot/dts/exynos5260-xyref5260.dts         | 3 ++-
 arch/arm/boot/dts/exynos5260.dtsi                  | 4 ++--
 arch/arm/boot/dts/exynos5410-odroidxu.dts          | 3 ++-
 arch/arm/boot/dts/exynos5410-smdk5410.dts          | 3 ++-
 arch/arm/boot/dts/exynos5420-arndale-octa.dts      | 3 ++-
 arch/arm/boot/dts/exynos5420-peach-pit.dts         | 3 ++-
 arch/arm/boot/dts/exynos5420-smdk5420.dts          | 3 ++-
 arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 3 ++-
 arch/arm/boot/dts/exynos5440-sd5v1.dts             | 6 ++++++
 arch/arm/boot/dts/exynos5440-ssdk5440.dts          | 6 ++++++
 arch/arm/boot/dts/exynos5440.dtsi                  | 3 ++-
 arch/arm/boot/dts/exynos54xx.dtsi                  | 1 -
 arch/arm/boot/dts/exynos5800-peach-pi.dts          | 3 ++-
 35 files changed, 76 insertions(+), 34 deletions(-)

-- 
2.7.4

[toc] | [next] | [standalone]

#1474198 — [PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-09-01 11:10 +0200
Subject[PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS
Message-ID<scwau-7KC-29@gated-at.bofh.it>
In reply to#1474197
This patch fixes the following DTC warnings:

"Node /memory has a reg or ranges property, but no unit name"

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

Changes in v2: None

 arch/arm/boot/dts/exynos4210-origen.dts         | 2 +-
 arch/arm/boot/dts/exynos4210-smdkv310.dts       | 2 +-
 arch/arm/boot/dts/exynos4210-trats.dts          | 2 +-
 arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
 arch/arm/boot/dts/exynos4412-odroidu3.dts       | 2 +-
 arch/arm/boot/dts/exynos4412-odroidx.dts        | 2 +-
 arch/arm/boot/dts/exynos4412-odroidx2.dts       | 2 +-
 arch/arm/boot/dts/exynos4412-origen.dts         | 2 +-
 arch/arm/boot/dts/exynos4412-smdk4412.dts       | 2 +-
 arch/arm/boot/dts/exynos4412-tiny4412.dts       | 2 +-
 arch/arm/boot/dts/exynos4412-trats2.dts         | 2 +-
 11 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/exynos4210-origen.dts b/arch/arm/boot/dts/exynos4210-origen.dts
index 6b2c41a87a3e..a2c6a13fe67b 100644
--- a/arch/arm/boot/dts/exynos4210-origen.dts
+++ b/arch/arm/boot/dts/exynos4210-origen.dts
@@ -24,7 +24,7 @@
 	model = "Insignal Origen evaluation board based on Exynos4210";
 	compatible = "insignal,origen", "samsung,exynos4210", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x10000000
 		       0x50000000 0x10000000
diff --git a/arch/arm/boot/dts/exynos4210-smdkv310.dts b/arch/arm/boot/dts/exynos4210-smdkv310.dts
index 3c759baeadcc..af6ef3e6214d 100644
--- a/arch/arm/boot/dts/exynos4210-smdkv310.dts
+++ b/arch/arm/boot/dts/exynos4210-smdkv310.dts
@@ -23,7 +23,7 @@
 	model = "Samsung smdkv310 evaluation board based on Exynos4210";
 	compatible = "samsung,smdkv310", "samsung,exynos4210", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x80000000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4210-trats.dts b/arch/arm/boot/dts/exynos4210-trats.dts
index 5f7ddb0627fb..0ca1b4d355f2 100644
--- a/arch/arm/boot/dts/exynos4210-trats.dts
+++ b/arch/arm/boot/dts/exynos4210-trats.dts
@@ -20,7 +20,7 @@
 	model = "Samsung Trats based on Exynos4210";
 	compatible = "samsung,trats", "samsung,exynos4210", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg =  <0x40000000 0x10000000
 			0x50000000 0x10000000
diff --git a/arch/arm/boot/dts/exynos4210-universal_c210.dts b/arch/arm/boot/dts/exynos4210-universal_c210.dts
index 30d938e17c9f..f1172be318cd 100644
--- a/arch/arm/boot/dts/exynos4210-universal_c210.dts
+++ b/arch/arm/boot/dts/exynos4210-universal_c210.dts
@@ -20,7 +20,7 @@
 	model = "Samsung Universal C210 based on Exynos4210 rev0";
 	compatible = "samsung,universal_c210", "samsung,exynos4210", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg =  <0x40000000 0x10000000
 			0x50000000 0x10000000>;
diff --git a/arch/arm/boot/dts/exynos4412-odroidu3.dts b/arch/arm/boot/dts/exynos4412-odroidu3.dts
index cc1b87e09534..99634c54dca9 100644
--- a/arch/arm/boot/dts/exynos4412-odroidu3.dts
+++ b/arch/arm/boot/dts/exynos4412-odroidu3.dts
@@ -18,7 +18,7 @@
 	model = "Hardkernel ODROID-U3 board based on Exynos4412";
 	compatible = "hardkernel,odroid-u3", "samsung,exynos4412", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x7FF00000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-odroidx.dts b/arch/arm/boot/dts/exynos4412-odroidx.dts
index 518ca57837be..d0a07673ae8b 100644
--- a/arch/arm/boot/dts/exynos4412-odroidx.dts
+++ b/arch/arm/boot/dts/exynos4412-odroidx.dts
@@ -18,7 +18,7 @@
 	model = "Hardkernel ODROID-X board based on Exynos4412";
 	compatible = "hardkernel,odroid-x", "samsung,exynos4412", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x3FF00000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-odroidx2.dts b/arch/arm/boot/dts/exynos4412-odroidx2.dts
index 92cdeea1cfb3..4d228858f172 100644
--- a/arch/arm/boot/dts/exynos4412-odroidx2.dts
+++ b/arch/arm/boot/dts/exynos4412-odroidx2.dts
@@ -17,7 +17,7 @@
 	model = "Hardkernel ODROID-X2 board based on Exynos4412";
 	compatible = "hardkernel,odroid-x2", "samsung,exynos4412", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x7FF00000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-origen.dts b/arch/arm/boot/dts/exynos4412-origen.dts
index 9156a35a6117..54a9f6906ed2 100644
--- a/arch/arm/boot/dts/exynos4412-origen.dts
+++ b/arch/arm/boot/dts/exynos4412-origen.dts
@@ -22,7 +22,7 @@
 	model = "Insignal Origen evaluation board based on Exynos4412";
 	compatible = "insignal,origen4412", "samsung,exynos4412", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x40000000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-smdk4412.dts b/arch/arm/boot/dts/exynos4412-smdk4412.dts
index 2e868c4c307e..2e208d84ff6a 100644
--- a/arch/arm/boot/dts/exynos4412-smdk4412.dts
+++ b/arch/arm/boot/dts/exynos4412-smdk4412.dts
@@ -20,7 +20,7 @@
 	model = "Samsung SMDK evaluation board based on Exynos4412";
 	compatible = "samsung,smdk4412", "samsung,exynos4412", "samsung,exynos4";
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x40000000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-tiny4412.dts b/arch/arm/boot/dts/exynos4412-tiny4412.dts
index ad60f3840e24..5504398e6e37 100644
--- a/arch/arm/boot/dts/exynos4412-tiny4412.dts
+++ b/arch/arm/boot/dts/exynos4412-tiny4412.dts
@@ -23,7 +23,7 @@
 		stdout-path = &serial_0;
 	};
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg = <0x40000000 0x40000000>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts
index e1380bae8ee3..41ecd6d465a7 100644
--- a/arch/arm/boot/dts/exynos4412-trats2.dts
+++ b/arch/arm/boot/dts/exynos4412-trats2.dts
@@ -30,7 +30,7 @@
 		i2c12 = &i2c_max77693_fuel;
 	};
 
-	memory {
+	memory@40000000 {
 		device_type = "memory";
 		reg =  <0x40000000 0x40000000>;
 	};
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1474536 — Re: [PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS

FromKrzysztof Kozlowski <krzk@kernel.org>
Date2016-09-01 18:40 +0200
SubjectRe: [PATCH v2 6/7] ARM: dts: exynos: Add missing unit name to memory nodes in Exynos4 DTS
Message-ID<scDbY-4hA-33@gated-at.bofh.it>
In reply to#1474198
On Thu, Sep 01, 2016 at 11:06:55AM +0200, Javier Martinez Canillas wrote:
> This patch fixes the following DTC warnings:
> 
> "Node /memory has a reg or ranges property, but no unit name"
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> 
> Changes in v2: None
> 
>  arch/arm/boot/dts/exynos4210-origen.dts         | 2 +-
>  arch/arm/boot/dts/exynos4210-smdkv310.dts       | 2 +-
>  arch/arm/boot/dts/exynos4210-trats.dts          | 2 +-
>  arch/arm/boot/dts/exynos4210-universal_c210.dts | 2 +-
>  arch/arm/boot/dts/exynos4412-odroidu3.dts       | 2 +-
>  arch/arm/boot/dts/exynos4412-odroidx.dts        | 2 +-
>  arch/arm/boot/dts/exynos4412-odroidx2.dts       | 2 +-
>  arch/arm/boot/dts/exynos4412-origen.dts         | 2 +-
>  arch/arm/boot/dts/exynos4412-smdk4412.dts       | 2 +-
>  arch/arm/boot/dts/exynos4412-tiny4412.dts       | 2 +-
>  arch/arm/boot/dts/exynos4412-trats2.dts         | 2 +-
>  11 files changed, 11 insertions(+), 11 deletions(-)



Thanks, applied.

Best regards,
Krzysztof

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web