* [PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
@ 2016-04-01 20:20 ` Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node Javier Martinez Canillas
` (4 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-04-01 20:20 UTC (permalink / raw)
To: linux-kernel
Cc: devicetree, linux-omap, Rob Herring, Tony Lindgren,
Javier Martinez Canillas
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
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes Javier Martinez Canillas
@ 2016-04-01 20:20 ` Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 3/5] ARM: dts: omap3: add missing unit name to PMU node Javier Martinez Canillas
` (3 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-04-01 20:20 UTC (permalink / raw)
To: linux-kernel
Cc: devicetree, linux-omap, Rob Herring, Tony Lindgren,
Javier Martinez Canillas
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
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 3/5] ARM: dts: omap3: add missing unit name to PMU node
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 1/5] ARM: dts: omap: add missing unit name to pbias regulator nodes Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 2/5] ARM: dts: n8x0: remove unneeded unit name for i2c node Javier Martinez Canillas
@ 2016-04-01 20:20 ` Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes Javier Martinez Canillas
` (2 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-04-01 20:20 UTC (permalink / raw)
To: linux-kernel
Cc: devicetree, linux-omap, Rob Herring, Tony Lindgren,
Javier Martinez Canillas
This patch fixes the following DTC warnings:
"pmu has a reg or ranges property, but no unit name"
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---
arch/arm/boot/dts/omap3.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index cbddde5d1e4f..e31ea5efb163 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -43,7 +43,7 @@
};
};
- pmu {
+ pmu@54000000 {
compatible = "arm,cortex-a8-pmu";
reg = <0x54000000 0x800000>;
interrupts = <3>;
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
` (2 preceding siblings ...)
2016-04-01 20:20 ` [PATCH 3/5] ARM: dts: omap3: add missing unit name to PMU node Javier Martinez Canillas
@ 2016-04-01 20:20 ` Javier Martinez Canillas
2016-04-01 20:20 ` [PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes Javier Martinez Canillas
2016-04-01 20:38 ` [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Tony Lindgren
5 siblings, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-04-01 20:20 UTC (permalink / raw)
To: linux-kernel
Cc: devicetree, linux-omap, Rob Herring, Tony Lindgren,
Javier Martinez Canillas
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
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
` (3 preceding siblings ...)
2016-04-01 20:20 ` [PATCH 4/5] ARM: dts: omap: remove unneeded unit name for sound nodes Javier Martinez Canillas
@ 2016-04-01 20:20 ` Javier Martinez Canillas
2016-04-01 20:38 ` [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Tony Lindgren
5 siblings, 0 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-04-01 20:20 UTC (permalink / raw)
To: linux-kernel
Cc: devicetree, linux-omap, Rob Herring, Tony Lindgren,
Javier Martinez Canillas
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
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings
2016-04-01 20:20 [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Javier Martinez Canillas
` (4 preceding siblings ...)
2016-04-01 20:20 ` [PATCH 5/5] ARM: dts: omap: add missing unit names to bandgap nodes Javier Martinez Canillas
@ 2016-04-01 20:38 ` Tony Lindgren
2016-04-11 19:39 ` Tony Lindgren
5 siblings, 1 reply; 8+ messages in thread
From: Tony Lindgren @ 2016-04-01 20:38 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, devicetree, linux-omap, Rob Herring, Tero Kristo
* 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
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings
2016-04-01 20:38 ` [PATCH 0/5] ARM: dts: omap: Fix some DTC warnings Tony Lindgren
@ 2016-04-11 19:39 ` Tony Lindgren
0 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2016-04-11 19:39 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, devicetree, linux-omap, Rob Herring, Tero Kristo
* 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
^ permalink raw reply [flat|nested] 8+ messages in thread