All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v1] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings
@ 2019-03-25 16:35 Marcel Ziswiler
  2019-03-25 16:40 ` Fabio Estevam
  0 siblings, 1 reply; 3+ messages in thread
From: Marcel Ziswiler @ 2019-03-25 16:35 UTC (permalink / raw)
  To: u-boot

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

This fixes the following warnings:

w+arch/arm/dts/imx6-apalis.dtb: Warning (reg_format): /soc/ipu at 2800000/
 port at 2/endpoint at 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 at 2800000/
 port at 2/endpoint at 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 at 2800000/
 port at 2/endpoint at 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 at 2800000/
 port at 2/endpoint at 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 at 2800000/
 port at 2/endpoint at 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 at 2800000/
 port at 3/endpoint at 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 at 2800000/
 port at 3/endpoint at 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 at 2800000/
 port at 3/endpoint at 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 at 2800000/
 port at 3/endpoint at 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 at 2800000/port at 2/endpoint at 0: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 0: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 1: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 1: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 2: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 2: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 3: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 3: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 4: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 2/endpoint at 4: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 1: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 1: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 2: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 2: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 3: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 3: Relying on default #size-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 4: Relying on default #address-cells
 value
w+arch/arm/dts/imx6-apalis.dtb: Warning (avoid_default_addr_size):
 /soc/ipu at 2800000/port at 3/endpoint at 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>

---

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

diff --git a/arch/arm/dts/imx6q.dtsi b/arch/arm/dts/imx6q.dtsi
index ab1716b6b0..8381d24eff 100644
--- a/arch/arm/dts/imx6q.dtsi
+++ b/arch/arm/dts/imx6q.dtsi
@@ -234,6 +234,8 @@
 			};
 
 			ipu2_di0: port at 2 {
+				#address-cells = <1>;
+				#size-cells = <0>;
 				reg = <2>;
 
 				ipu2_di0_disp0: endpoint at 0 {
@@ -262,6 +264,8 @@
 			};
 
 			ipu2_di1: port at 3 {
+				#address-cells = <1>;
+				#size-cells = <0>;
 				reg = <3>;
 
 				ipu2_di1_hdmi: endpoint at 1 {
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH v1] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings
  2019-03-25 16:35 [U-Boot] [PATCH v1] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings Marcel Ziswiler
@ 2019-03-25 16:40 ` Fabio Estevam
  2019-04-09 13:51   ` Marcel Ziswiler
  0 siblings, 1 reply; 3+ messages in thread
From: Fabio Estevam @ 2019-03-25 16:40 UTC (permalink / raw)
  To: u-boot

Hi Marcel,

On Mon, Mar 25, 2019 at 1:36 PM Marcel Ziswiler <marcel@ziswiler.com> wrote:

> diff --git a/arch/arm/dts/imx6q.dtsi b/arch/arm/dts/imx6q.dtsi
> index ab1716b6b0..8381d24eff 100644
> --- a/arch/arm/dts/imx6q.dtsi
> +++ b/arch/arm/dts/imx6q.dtsi
> @@ -234,6 +234,8 @@
>                         };
>
>                         ipu2_di0: port at 2 {
> +                               #address-cells = <1>;
> +                               #size-cells = <0>;

This change is already in the upstream imx6q.dtsi file in Linux.

What about syncing the U-Boot i.mx6 dts with the kernel?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH v1] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings
  2019-03-25 16:40 ` Fabio Estevam
@ 2019-04-09 13:51   ` Marcel Ziswiler
  0 siblings, 0 replies; 3+ messages in thread
From: Marcel Ziswiler @ 2019-04-09 13:51 UTC (permalink / raw)
  To: u-boot

On Mon, 2019-03-25 at 13:40 -0300, Fabio Estevam wrote:
> Hi Marcel,
> 
> On Mon, Mar 25, 2019 at 1:36 PM Marcel Ziswiler <marcel@ziswiler.com>
> wrote:
> 
> > diff --git a/arch/arm/dts/imx6q.dtsi b/arch/arm/dts/imx6q.dtsi
> > index ab1716b6b0..8381d24eff 100644
> > --- a/arch/arm/dts/imx6q.dtsi
> > +++ b/arch/arm/dts/imx6q.dtsi
> > @@ -234,6 +234,8 @@
> >                         };
> > 
> >                         ipu2_di0: port at 2 {
> > +                               #address-cells = <1>;
> > +                               #size-cells = <0>;
> 
> This change is already in the upstream imx6q.dtsi file in Linux.
> 
> What about syncing the U-Boot i.mx6 dts with the kernel?

Yeah, I agree. As a matter of fact I see only one single line of
further difference in the two. Let me add that as well in a v2 and
clearly state that it is a complete re-sync.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-04-09 13:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-25 16:35 [U-Boot] [PATCH v1] ARM: dts: i.MX6Q: fix avoid_unnecessary_addr_size warnings Marcel Ziswiler
2019-03-25 16:40 ` Fabio Estevam
2019-04-09 13:51   ` Marcel Ziswiler

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.