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

Groups > linux.kernel > #1369575 > unrolled thread

[PATCH 0/5] ARM: dts: omap: Fix some DTC warnings

Started byJavier Martinez Canillas <javier@osg.samsung.com>
First post2016-04-01 22:30 +0200
Last post2016-04-11 21:50 +0200
Articles 7 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas <javier@osg.samsung.com> - 2016-04-01 22:30 +0200
    [PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node Javier Martinez Canillas <javier@osg.samsung.com> - 2016-04-01 22:30 +0200
    [PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes Javier Martinez Canillas <javier@osg.samsung.com> - 2016-04-01 22:30 +0200
    [PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes Javier Martinez Canillas <javier@osg.samsung.com> - 2016-04-01 22:30 +0200
    [PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes Javier Martinez Canillas <javier@osg.samsung.com> - 2016-04-01 22:30 +0200
    Re: [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Tony Lindgren <tony@atomide.com> - 2016-04-01 22:40 +0200
      Re: [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Tony Lindgren <tony@atomide.com> - 2016-04-11 21:50 +0200

#1369575 — [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-04-01 22:30 +0200
Subject[PATCH 0/5] ARM: dts: omap: Fix some DTC warnings
Message-ID<rjdRD-18Z-3@gated-at.bofh.it>
Hello Tony

DTC reports a lot of warnings on omap DTS due a mismatch between unit
names and reg properties in different device nodes. This series tries
to fix at least some of them, but there are still a lot remaining.

Best regards,
Javier


Javier Martinez Canillas (5):
  ARM: dts: omap: add missing unit name to pbias regulator nodes
  ARM: dts: n8x0: remove unneeded unit name for i2c node
  ARM: dts: omap3: add missing unit name to PMU node
  ARM: dts: omap: remove unneeded unit name for sound nodes
  ARM: dts: omap: add missing unit names to bandgap nodes

 arch/arm/boot/dts/am437x-gp-evm.dts         | 2 +-
 arch/arm/boot/dts/am43x-epos-evm.dts        | 2 +-
 arch/arm/boot/dts/am57xx-beagle-x15.dts     | 2 +-
 arch/arm/boot/dts/am57xx-cl-som-am57x.dts   | 2 +-
 arch/arm/boot/dts/dra7-evm.dts              | 2 +-
 arch/arm/boot/dts/dra7.dtsi                 | 2 +-
 arch/arm/boot/dts/dra72-evm.dts             | 2 +-
 arch/arm/boot/dts/omap2420-n8x0-common.dtsi | 2 +-
 arch/arm/boot/dts/omap2430.dtsi             | 2 +-
 arch/arm/boot/dts/omap3.dtsi                | 4 ++--
 arch/arm/boot/dts/omap34xx.dtsi             | 2 +-
 arch/arm/boot/dts/omap36xx.dtsi             | 2 +-
 arch/arm/boot/dts/omap4-var-som-om44.dtsi   | 2 +-
 arch/arm/boot/dts/omap4.dtsi                | 2 +-
 arch/arm/boot/dts/omap443x.dtsi             | 2 +-
 arch/arm/boot/dts/omap4460.dtsi             | 2 +-
 arch/arm/boot/dts/omap5.dtsi                | 2 +-
 17 files changed, 18 insertions(+), 18 deletions(-)

-- 
2.5.0

[toc] | [next] | [standalone]

#1369576 — [PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-04-01 22:30 +0200
Subject[PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node
Message-ID<rjdRE-18Z-7@gated-at.bofh.it>
In reply to#1369575
This patch fixes the following DTC warnings:

"i2c@0 has a unit name, but no reg property"

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

 arch/arm/boot/dts/omap2420-n8x0-common.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/omap2420-n8x0-common.dtsi b/arch/arm/boot/dts/omap2420-n8x0-common.dtsi
index 8491f46c61b7..db95aadcca70 100644
--- a/arch/arm/boot/dts/omap2420-n8x0-common.dtsi
+++ b/arch/arm/boot/dts/omap2420-n8x0-common.dtsi
@@ -7,7 +7,7 @@
 	};
 
 	ocp {
-		i2c@0 {
+		i2c0 {
 			compatible = "i2c-cbus-gpio";
 			gpios = <&gpio3 2 GPIO_ACTIVE_HIGH /* gpio66 clk */
 				 &gpio3 1 GPIO_ACTIVE_HIGH /* gpio65 dat */
-- 
2.5.0

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

#1369578 — [PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-04-01 22:30 +0200
Subject[PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes
Message-ID<rjdRE-18Z-11@gated-at.bofh.it>
In reply to#1369575
This patch fixes the following DTC warning:

"sound@0 has a unit name, but no reg property"

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

 arch/arm/boot/dts/am437x-gp-evm.dts       | 2 +-
 arch/arm/boot/dts/am43x-epos-evm.dts      | 2 +-
 arch/arm/boot/dts/am57xx-beagle-x15.dts   | 2 +-
 arch/arm/boot/dts/am57xx-cl-som-am57x.dts | 2 +-
 arch/arm/boot/dts/dra7-evm.dts            | 2 +-
 arch/arm/boot/dts/dra72-evm.dts           | 2 +-
 arch/arm/boot/dts/omap4-var-som-om44.dtsi | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
index 8889be1ca1c3..95ec82a77e67 100644
--- a/arch/arm/boot/dts/am437x-gp-evm.dts
+++ b/arch/arm/boot/dts/am437x-gp-evm.dts
@@ -119,7 +119,7 @@
 		clock-frequency = <32768>;
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "AM437x-GP-EVM";
 		simple-audio-card,widgets =
diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index d5dd72047a7e..04125b43e3a9 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -107,7 +107,7 @@
 		default-brightness-level = <8>;
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "AM43-EPOS-EVM";
 		simple-audio-card,widgets =
diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
index 0a5fc5d02ce2..75bd09b175cd 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
+++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
@@ -151,7 +151,7 @@
 		};
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "BeagleBoard-X15";
 		simple-audio-card,widgets =
diff --git a/arch/arm/boot/dts/am57xx-cl-som-am57x.dts b/arch/arm/boot/dts/am57xx-cl-som-am57x.dts
index 14f912a1b4fd..378b142ef88c 100644
--- a/arch/arm/boot/dts/am57xx-cl-som-am57x.dts
+++ b/arch/arm/boot/dts/am57xx-cl-som-am57x.dts
@@ -51,7 +51,7 @@
 		regulator-max-microvolt = <3300000>;
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "CL-SOM-AM57x-Sound-Card";
 		simple-audio-card,format = "i2s";
diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts
index d9b87236019d..074d70b191e2 100644
--- a/arch/arm/boot/dts/dra7-evm.dts
+++ b/arch/arm/boot/dts/dra7-evm.dts
@@ -67,7 +67,7 @@
 		gpio = <&gpio7 11 GPIO_ACTIVE_HIGH>;
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "DRA7xx-EVM";
 		simple-audio-card,widgets =
diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
index 6affe2d137da..6006ced8f6b3 100644
--- a/arch/arm/boot/dts/dra72-evm.dts
+++ b/arch/arm/boot/dts/dra72-evm.dts
@@ -104,7 +104,7 @@
 		};
 	};
 
-	sound0: sound@0 {
+	sound0: sound0 {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "DRA7xx-EVM";
 		simple-audio-card,widgets =
diff --git a/arch/arm/boot/dts/omap4-var-som-om44.dtsi b/arch/arm/boot/dts/omap4-var-som-om44.dtsi
index 49d032b846be..a17997f4e9aa 100644
--- a/arch/arm/boot/dts/omap4-var-som-om44.dtsi
+++ b/arch/arm/boot/dts/omap4-var-som-om44.dtsi
@@ -17,7 +17,7 @@
 		reg = <0x80000000 0x40000000>; /* 1 GB */
 	};
 
-	sound: sound@0 {
+	sound: sound {
 		compatible = "ti,abe-twl6040";
 		ti,model = "VAR-SOM-OM44";
 
-- 
2.5.0

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

#1369579 — [PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-04-01 22:30 +0200
Subject[PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes
Message-ID<rjdRE-18Z-13@gated-at.bofh.it>
In reply to#1369575
This patch fixes the following DTC warnings:

"pbias_regulator has a reg or ranges property, but no unit name"

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

 arch/arm/boot/dts/dra7.dtsi     | 2 +-
 arch/arm/boot/dts/omap2430.dtsi | 2 +-
 arch/arm/boot/dts/omap3.dtsi    | 2 +-
 arch/arm/boot/dts/omap4.dtsi    | 2 +-
 arch/arm/boot/dts/omap5.dtsi    | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index 13ac88279427..ed740d02ec17 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -123,7 +123,7 @@
 					#size-cells = <1>;
 					ranges = <0 0x0 0x1400>;
 
-					pbias_regulator: pbias_regulator {
+					pbias_regulator: pbias_regulator@e00 {
 						compatible = "ti,pbias-dra7", "ti,pbias-omap";
 						reg = <0xe00 0x4>;
 						syscon = <&scm_conf>;
diff --git a/arch/arm/boot/dts/omap2430.dtsi b/arch/arm/boot/dts/omap2430.dtsi
index 798dda072b2a..59828b0e34f1 100644
--- a/arch/arm/boot/dts/omap2430.dtsi
+++ b/arch/arm/boot/dts/omap2430.dtsi
@@ -63,7 +63,7 @@
 						#size-cells = <0>;
 					};
 
-					pbias_regulator: pbias_regulator {
+					pbias_regulator: pbias_regulator@230 {
 						compatible = "ti,pbias-omap2", "ti,pbias-omap";
 						reg = <0x230 0x4>;
 						syscon = <&scm_conf>;
diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index b41d07e8e765..cbddde5d1e4f 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -119,7 +119,7 @@
 					#size-cells = <1>;
 					ranges = <0 0x270 0x330>;
 
-					pbias_regulator: pbias_regulator {
+					pbias_regulator: pbias_regulator@2b0 {
 						compatible = "ti,pbias-omap3", "ti,pbias-omap";
 						reg = <0x2b0 0x4>;
 						syscon = <&scm_conf>;
diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
index 421fe9f8a9eb..c0547647f64b 100644
--- a/arch/arm/boot/dts/omap4.dtsi
+++ b/arch/arm/boot/dts/omap4.dtsi
@@ -198,7 +198,7 @@
 					#size-cells = <1>;
 					ranges = <0 0x5a0 0x170>;
 
-					pbias_regulator: pbias_regulator {
+					pbias_regulator: pbias_regulator@60 {
 						compatible = "ti,pbias-omap4", "ti,pbias-omap";
 						reg = <0x60 0x4>;
 						syscon = <&omap4_padconf_global>;
diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
index 38805ebbe2ba..b2297a560825 100644
--- a/arch/arm/boot/dts/omap5.dtsi
+++ b/arch/arm/boot/dts/omap5.dtsi
@@ -187,7 +187,7 @@
 					#size-cells = <1>;
 					ranges = <0 0x5a0 0xec>;
 
-					pbias_regulator: pbias_regulator {
+					pbias_regulator: pbias_regulator@60 {
 						compatible = "ti,pbias-omap5", "ti,pbias-omap";
 						reg = <0x60 0x4>;
 						syscon = <&omap5_padconf_global>;
-- 
2.5.0

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

#1369583 — [PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-04-01 22:30 +0200
Subject[PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes
Message-ID<rjdRF-18Z-23@gated-at.bofh.it>
In reply to#1369575
This patch fixes the following DTC warnings:

"bandgap has a reg or ranges property, but no unit name"

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

---

 arch/arm/boot/dts/omap34xx.dtsi | 2 +-
 arch/arm/boot/dts/omap36xx.dtsi | 2 +-
 arch/arm/boot/dts/omap443x.dtsi | 2 +-
 arch/arm/boot/dts/omap4460.dtsi | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/omap34xx.dtsi b/arch/arm/boot/dts/omap34xx.dtsi
index 387dc31822fe..5cdba1f6499f 100644
--- a/arch/arm/boot/dts/omap34xx.dtsi
+++ b/arch/arm/boot/dts/omap34xx.dtsi
@@ -55,7 +55,7 @@
 			};
 		};
 
-		bandgap {
+		bandgap@48002524 {
 			reg = <0x48002524 0x4>;
 			compatible = "ti,omap34xx-bandgap";
 			#thermal-sensor-cells = <0>;
diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
index f19c87bd6bf3..ce1e242d4dc0 100644
--- a/arch/arm/boot/dts/omap36xx.dtsi
+++ b/arch/arm/boot/dts/omap36xx.dtsi
@@ -87,7 +87,7 @@
 			};
 		};
 
-		bandgap {
+		bandgap@48002524 {
 			reg = <0x48002524 0x4>;
 			compatible = "ti,omap36xx-bandgap";
 			#thermal-sensor-cells = <0>;
diff --git a/arch/arm/boot/dts/omap443x.dtsi b/arch/arm/boot/dts/omap443x.dtsi
index 0adfa1d1ef20..fc6a8610c24c 100644
--- a/arch/arm/boot/dts/omap443x.dtsi
+++ b/arch/arm/boot/dts/omap443x.dtsi
@@ -35,7 +35,7 @@
 	};
 
 	ocp {
-		bandgap: bandgap {
+		bandgap: bandgap@4a002260 {
 			reg = <0x4a002260 0x4
 			       0x4a00232C 0x4>;
 			compatible = "ti,omap4430-bandgap";
diff --git a/arch/arm/boot/dts/omap4460.dtsi b/arch/arm/boot/dts/omap4460.dtsi
index 5fa68f191af7..ef66e12e0a67 100644
--- a/arch/arm/boot/dts/omap4460.dtsi
+++ b/arch/arm/boot/dts/omap4460.dtsi
@@ -40,7 +40,7 @@
 	};
 
 	ocp {
-		bandgap: bandgap {
+		bandgap: bandgap@4a002260 {
 			reg = <0x4a002260 0x4
 			       0x4a00232C 0x4
 			       0x4a002378 0x18>;
-- 
2.5.0

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

#1369586

FromTony Lindgren <tony@atomide.com>
Date2016-04-01 22:40 +0200
Message-ID<rje1k-1cN-11@gated-at.bofh.it>
In reply to#1369575
* Javier Martinez Canillas <javier@osg.samsung.com> [160401 13:21]:
> Hello Tony
> 
> DTC reports a lot of warnings on omap DTS due a mismatch between unit
> names and reg properties in different device nodes. This series tries
> to fix at least some of them, but there are still a lot remaining.

Great, adding Tero to Cc as I think he is already looking at fixing
the clock related warnings.

Regards,

Tony


> Javier Martinez Canillas (5):
>   ARM: dts: omap: add missing unit name to pbias regulator nodes
>   ARM: dts: n8x0: remove unneeded unit name for i2c node
>   ARM: dts: omap3: add missing unit name to PMU node
>   ARM: dts: omap: remove unneeded unit name for sound nodes
>   ARM: dts: omap: add missing unit names to bandgap nodes
> 
>  arch/arm/boot/dts/am437x-gp-evm.dts         | 2 +-
>  arch/arm/boot/dts/am43x-epos-evm.dts        | 2 +-
>  arch/arm/boot/dts/am57xx-beagle-x15.dts     | 2 +-
>  arch/arm/boot/dts/am57xx-cl-som-am57x.dts   | 2 +-
>  arch/arm/boot/dts/dra7-evm.dts              | 2 +-
>  arch/arm/boot/dts/dra7.dtsi                 | 2 +-
>  arch/arm/boot/dts/dra72-evm.dts             | 2 +-
>  arch/arm/boot/dts/omap2420-n8x0-common.dtsi | 2 +-
>  arch/arm/boot/dts/omap2430.dtsi             | 2 +-
>  arch/arm/boot/dts/omap3.dtsi                | 4 ++--
>  arch/arm/boot/dts/omap34xx.dtsi             | 2 +-
>  arch/arm/boot/dts/omap36xx.dtsi             | 2 +-
>  arch/arm/boot/dts/omap4-var-som-om44.dtsi   | 2 +-
>  arch/arm/boot/dts/omap4.dtsi                | 2 +-
>  arch/arm/boot/dts/omap443x.dtsi             | 2 +-
>  arch/arm/boot/dts/omap4460.dtsi             | 2 +-
>  arch/arm/boot/dts/omap5.dtsi                | 2 +-
>  17 files changed, 18 insertions(+), 18 deletions(-)
> 
> -- 
> 2.5.0
>

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

#1376256

FromTony Lindgren <tony@atomide.com>
Date2016-04-11 21:50 +0200
Message-ID<rmQ0p-2ML-5@gated-at.bofh.it>
In reply to#1369586
* Tony Lindgren <tony@atomide.com> [160401 13:39]:
> * Javier Martinez Canillas <javier@osg.samsung.com> [160401 13:21]:
> > Hello Tony
> > 
> > DTC reports a lot of warnings on omap DTS due a mismatch between unit
> > names and reg properties in different device nodes. This series tries
> > to fix at least some of them, but there are still a lot remaining.
> 
> Great, adding Tero to Cc as I think he is already looking at fixing
> the clock related warnings.

OK adding these finally into omap-for-v4.7/dt thanks.

Tony

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web