* [U-Boot] [PATCH 2/2] at91: fix devicetree warnings
2018-08-09 11:12 [U-Boot] [PATCH 1/2] ARM: dts: Makefile: fix build for at91family Eugen Hristev
@ 2018-08-09 11:12 ` Eugen Hristev
2018-09-11 12:25 ` [U-Boot] [U-Boot,2/2] " Tom Rini
2018-09-11 12:25 ` [U-Boot] [U-Boot, 1/2] ARM: dts: Makefile: fix build for at91family Tom Rini
1 sibling, 1 reply; 4+ messages in thread
From: Eugen Hristev @ 2018-08-09 11:12 UTC (permalink / raw)
To: u-boot
This commit fixes various devicetree warnings in the at91 devicetrees
like:
arch/arm/dts/sama5d36ek_cmp.dtb: Warning (avoid_unnecessary_addr_size): /gpio_keys: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
These warnings make buildman report failed builds for most of at91 boards.
Thus, fixing it.
Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
---
arch/arm/dts/at91-vinco.dts | 2 ++
arch/arm/dts/at91sam9260.dtsi | 3 ---
arch/arm/dts/at91sam9261.dtsi | 3 ---
arch/arm/dts/at91sam9263.dtsi | 3 ---
arch/arm/dts/at91sam9g45.dtsi | 9 ---------
arch/arm/dts/at91sam9n12.dtsi | 3 ---
arch/arm/dts/at91sam9rl.dtsi | 5 -----
arch/arm/dts/at91sam9x5.dtsi | 5 -----
arch/arm/dts/sama5d27_som1.dtsi | 1 -
arch/arm/dts/sama5d3.dtsi | 4 ----
arch/arm/dts/sama5d3xmb_cmp.dtsi | 2 --
arch/arm/dts/usb_a9263.dts | 2 --
12 files changed, 2 insertions(+), 40 deletions(-)
diff --git a/arch/arm/dts/at91-vinco.dts b/arch/arm/dts/at91-vinco.dts
index ff6d2e3..0ecb64e 100644
--- a/arch/arm/dts/at91-vinco.dts
+++ b/arch/arm/dts/at91-vinco.dts
@@ -152,6 +152,8 @@
};
tcb2: timer at fc024000 {
+ #address-cells = <1>;
+ #size-cells = <0>;
timer at 0 {
compatible = "atmel,tcb-timer";
reg = <0>;
diff --git a/arch/arm/dts/at91sam9260.dtsi b/arch/arm/dts/at91sam9260.dtsi
index 69d9cea..476ad1d 100644
--- a/arch/arm/dts/at91sam9260.dtsi
+++ b/arch/arm/dts/at91sam9260.dtsi
@@ -37,9 +37,6 @@
spi0 = &spi0;
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
diff --git a/arch/arm/dts/at91sam9261.dtsi b/arch/arm/dts/at91sam9261.dtsi
index 69c2d6e..b6357d3 100644
--- a/arch/arm/dts/at91sam9261.dtsi
+++ b/arch/arm/dts/at91sam9261.dtsi
@@ -34,9 +34,6 @@
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
diff --git a/arch/arm/dts/at91sam9263.dtsi b/arch/arm/dts/at91sam9263.dtsi
index 0b594be..61b0562 100644
--- a/arch/arm/dts/at91sam9263.dtsi
+++ b/arch/arm/dts/at91sam9263.dtsi
@@ -36,9 +36,6 @@
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
diff --git a/arch/arm/dts/at91sam9g45.dtsi b/arch/arm/dts/at91sam9g45.dtsi
index 1be1270..c9b2e46 100644
--- a/arch/arm/dts/at91sam9g45.dtsi
+++ b/arch/arm/dts/at91sam9g45.dtsi
@@ -41,9 +41,6 @@
pwm0 = &pwm0;
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
@@ -1042,8 +1039,6 @@
};
adc0: adc at fffb0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "atmel,at91sam9g45-adc";
reg = <0xfffb0000 0x100>;
interrupts = <20 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -1086,10 +1081,6 @@
clocks = <&isi_clk>;
clock-names = "isi_clk";
status = "disabled";
- port {
- #address-cells = <1>;
- #size-cells = <0>;
- };
};
pwm0: pwm at fffb8000 {
diff --git a/arch/arm/dts/at91sam9n12.dtsi b/arch/arm/dts/at91sam9n12.dtsi
index 7ba7116..024be13 100644
--- a/arch/arm/dts/at91sam9n12.dtsi
+++ b/arch/arm/dts/at91sam9n12.dtsi
@@ -38,9 +38,6 @@
spi0 = &spi0;
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
diff --git a/arch/arm/dts/at91sam9rl.dtsi b/arch/arm/dts/at91sam9rl.dtsi
index 4602cd2..6d6aee5 100644
--- a/arch/arm/dts/at91sam9rl.dtsi
+++ b/arch/arm/dts/at91sam9rl.dtsi
@@ -38,9 +38,6 @@
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
@@ -253,8 +250,6 @@
};
adc0: adc at fffd0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "atmel,at91sam9rl-adc";
reg = <0xfffd0000 0x100>;
interrupts = <20 IRQ_TYPE_LEVEL_HIGH 0>;
diff --git a/arch/arm/dts/at91sam9x5.dtsi b/arch/arm/dts/at91sam9x5.dtsi
index a422c98..ea319cc 100644
--- a/arch/arm/dts/at91sam9x5.dtsi
+++ b/arch/arm/dts/at91sam9x5.dtsi
@@ -41,9 +41,6 @@
};
cpus {
- #address-cells = <0>;
- #size-cells = <0>;
-
cpu {
compatible = "arm,arm926ej-s";
device_type = "cpu";
@@ -1048,8 +1045,6 @@
};
adc0: adc at f804c000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "atmel,at91sam9x5-adc";
reg = <0xf804c000 0x100>;
interrupts = <19 IRQ_TYPE_LEVEL_HIGH 0>;
diff --git a/arch/arm/dts/sama5d27_som1.dtsi b/arch/arm/dts/sama5d27_som1.dtsi
index 0c44a97..9efcf63 100644
--- a/arch/arm/dts/sama5d27_som1.dtsi
+++ b/arch/arm/dts/sama5d27_som1.dtsi
@@ -55,7 +55,6 @@
aliases {
spi0 = &qspi1;
- u-boot,dm-pre-reloc;
};
ahb {
diff --git a/arch/arm/dts/sama5d3.dtsi b/arch/arm/dts/sama5d3.dtsi
index ee0e14e..ba707b0 100644
--- a/arch/arm/dts/sama5d3.dtsi
+++ b/arch/arm/dts/sama5d3.dtsi
@@ -238,10 +238,6 @@
clocks = <&isi_clk>;
clock-names = "isi_clk";
status = "disabled";
- port {
- #address-cells = <1>;
- #size-cells = <0>;
- };
};
sfr: sfr at f0038000 {
diff --git a/arch/arm/dts/sama5d3xmb_cmp.dtsi b/arch/arm/dts/sama5d3xmb_cmp.dtsi
index 77f989c..9792b2a 100644
--- a/arch/arm/dts/sama5d3xmb_cmp.dtsi
+++ b/arch/arm/dts/sama5d3xmb_cmp.dtsi
@@ -221,8 +221,6 @@
compatible = "gpio-keys";
status = "okay";
- #address-cells = <1>;
- #size-cells = <0>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_gpio_keys>;
diff --git a/arch/arm/dts/usb_a9263.dts b/arch/arm/dts/usb_a9263.dts
index bfc48a2..9adc12a 100644
--- a/arch/arm/dts/usb_a9263.dts
+++ b/arch/arm/dts/usb_a9263.dts
@@ -127,8 +127,6 @@
gpio_keys {
compatible = "gpio-keys";
- #address-cells = <1>;
- #size-cells = <0>;
user_pb {
label = "user_pb";
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread