diff mbox series

[U-Boot,2/2] at91: fix devicetree warnings

Message ID 1533813121-4120-2-git-send-email-eugen.hristev@microchip.com
State Accepted
Commit 51330c72f048e03cfbf7f7d614b259dda4bca469
Delegated to: Tom Rini
Headers show
Series [U-Boot,1/2] ARM: dts: Makefile: fix build for at91family | expand

Commit Message

Eugen Hristev Aug. 9, 2018, 11:12 a.m. UTC
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(-)

Comments

Tom Rini Sept. 11, 2018, 12:25 p.m. UTC | #1
On Thu, Aug 09, 2018 at 02:12:01PM +0300, Eugen Hristev wrote:

> 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>

Applied to u-boot/master, thanks!
diff mbox series

Patch

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@fc024000 {
+				#address-cells = <1>;
+				#size-cells = <0>;
 				timer@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@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@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@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@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@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";