[U-Boot] [PATCH 1/2] ARM: dts: Makefile: fix build for at91family

For at91family, makefile was building unwanted DTBs Moved them under specific target configuration.
Signed-off-by: Eugen Hristev eugen.hristev@microchip.com --- arch/arm/dts/Makefile | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile index ebfa227..79fdc0d 100644 --- a/arch/arm/dts/Makefile +++ b/arch/arm/dts/Makefile @@ -1,9 +1,9 @@ # SPDX-License-Identifier: GPL-2.0+
-dtb-$(CONFIG_AT91FAMILY) += at91sam9260-smartweb.dtb \ - at91sam9g20-taurus.dtb \ - at91sam9g45-corvus.dtb \ - at91sam9g45-gurnard.dtb +dtb-$(CONFIG_TARGET_SMARTWEB) += at91sam9260-smartweb.dtb +dtb-$(CONFIG_TARGET_TAURUS) += at91sam9g20-taurus.dtb +dtb-$(CONFIG_TARGET_CORVUS) += at91sam9g45-corvus.dtb +dtb-$(CONFIG_TARGET_GURNARD) += at91sam9g45-gurnard.dtb
dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb

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@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";

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!

On Thu, Aug 09, 2018 at 02:12:00PM +0300, Eugen Hristev wrote:
For at91family, makefile was building unwanted DTBs Moved them under specific target configuration.
Signed-off-by: Eugen Hristev eugen.hristev@microchip.com
Applied to u-boot/master, thanks!
participants (2)
-
Eugen Hristev
-
Tom Rini