diff mbox series

[U-Boot,v2] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings

Message ID 20190409152113.10678-1-marcel@ziswiler.com
State Superseded
Delegated to: Stefano Babic
Headers show
Series [U-Boot,v2] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings | expand

Commit Message

Marcel Ziswiler April 9, 2019, 3:21 p.m. UTC 
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

Re-synced the device tree with Linux 5.0.

This fixes the following warnings:

w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@2/endpoint@0:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@2/endpoint@1:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@2/endpoint@2:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@2/endpoint@3:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@2/endpoint@4:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@3/endpoint@1:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@3/endpoint@2:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@3/endpoint@3:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
 port@3/endpoint@4:reg: property has invalid length (4 bytes)
 (#address-cells == 2, #size-cells == 1)
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@0: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@0: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@1: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@1: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@2: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@2: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@3: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@3: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@4: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@2/endpoint@4: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@1: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@1: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@2: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@2: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@3: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@3: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@4: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu@2800000/port@3/endpoint@4: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_unnecessary_addr_size):
 Failed prerequisite 'avoid_default_addr_size'

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

---

Changes in v2:
- Re-synced the device tree with Linux 5.0 as suggested by Fabio.

 arch/arm/dts/imx6q.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Fabio Estevam April 9, 2019, 3:23 p.m. UTC | #1 
On Tue, Apr 9, 2019 at 12:21 PM Marcel Ziswiler <marcel@ziswiler.com> wrote:
>
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> Re-synced the device tree with Linux 5.0.
>
> This fixes the following warnings:
>
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@2/endpoint@0:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@2/endpoint@1:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@2/endpoint@2:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@2/endpoint@3:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@2/endpoint@4:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@3/endpoint@1:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@3/endpoint@2:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@3/endpoint@3:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu@2800000/
>  port@3/endpoint@4:reg: property has invalid length (4 bytes)
>  (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@0: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@0: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@1: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@1: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@2: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@2: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@3: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@3: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@4: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@4: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@1: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@1: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@2: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@2: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@3: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@3: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@4: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@4: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_unnecessary_addr_size):
>  Failed prerequisite 'avoid_default_addr_size'
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

Reviewed-by: Fabio Estevam <festevam@gmail. com>
Lukasz Majewski April 10, 2019, 5:08 a.m. UTC | #2 
On Tue,  9 Apr 2019 17:21:13 +0200
Marcel Ziswiler <marcel@ziswiler.com> wrote:

> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> Re-synced the device tree with Linux 5.0.
> 
> This fixes the following warnings:
> 
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@2/endpoint@0:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@2/endpoint@1:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@2/endpoint@2:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@2/endpoint@3:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@2/endpoint@4:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@3/endpoint@1:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@3/endpoint@2:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@3/endpoint@3:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning
> (reg_format): /soc/ipu@2800000/ port@3/endpoint@4:reg: property has
> invalid length (4 bytes) (#address-cells == 2, #size-cells == 1)
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@0: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@0: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@1: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@1: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@2: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@2: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@3: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@3: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@4: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@2/endpoint@4: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@1: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@1: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@2: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@2: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@3: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@3: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@4: Relying on default #address-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
>  /soc/ipu@2800000/port@3/endpoint@4: Relying on default #size-cells
>  value
> w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_unnecessary_addr_size):
>  Failed prerequisite 'avoid_default_addr_size'
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> ---
> 
> Changes in v2:
> - Re-synced the device tree with Linux 5.0 as suggested by Fabio.
> 
>  arch/arm/dts/imx6q.dtsi | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/dts/imx6q.dtsi b/arch/arm/dts/imx6q.dtsi
> index ab1716b6b0..d038f41170 100644
> --- a/arch/arm/dts/imx6q.dtsi
> +++ b/arch/arm/dts/imx6q.dtsi
> @@ -202,6 +202,7 @@
>  				 <&clks IMX6QDL_CLK_GPU2D_CORE>;
>  			clock-names = "bus", "core";
>  			power-domains = <&pd_pu>;
> +			#cooling-cells = <2>;
>  		};
>  
>  		ipu2: ipu@2800000 {
> @@ -234,6 +235,8 @@
>  			};
>  
>  			ipu2_di0: port@2 {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
>  				reg = <2>;
>  
>  				ipu2_di0_disp0: endpoint@0 {
> @@ -262,6 +265,8 @@
>  			};
>  
>  			ipu2_di1: port@3 {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
>  				reg = <3>;
>  
>  				ipu2_di1_hdmi: endpoint@1 {

Reviewed-by: Lukasz Majewski <lukma@denx.de>


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
diff mbox series

Patch

diff --git a/arch/arm/dts/imx6q.dtsi b/arch/arm/dts/imx6q.dtsi
index ab1716b6b0..d038f41170 100644
--- a/arch/arm/dts/imx6q.dtsi
+++ b/arch/arm/dts/imx6q.dtsi
@@ -202,6 +202,7 @@ 
 				 <&clks IMX6QDL_CLK_GPU2D_CORE>;
 			clock-names = "bus", "core";
 			power-domains = <&pd_pu>;
+			#cooling-cells = <2>;
 		};
 
 		ipu2: ipu@2800000 {
@@ -234,6 +235,8 @@ 
 			};
 
 			ipu2_di0: port@2 {
+				#address-cells = <1>;
+				#size-cells = <0>;
 				reg = <2>;
 
 				ipu2_di0_disp0: endpoint@0 {
@@ -262,6 +265,8 @@ 
 			};
 
 			ipu2_di1: port@3 {
+				#address-cells = <1>;
+				#size-cells = <0>;
 				reg = <3>;
 
 				ipu2_di1_hdmi: endpoint@1 {