[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <54efc169e9c82b52a96707f4fa8fedfcdc9242f6.camel@phytec.de>
Date: Mon, 20 Oct 2025 12:43:02 +0000
From: Yannic Moog <y.moog@...tec.de>
To: "shawnguo2@...h.net" <shawnguo2@...h.net>
CC: "upstream@...ts.phytec.de" <upstream@...ts.phytec.de>, "peng.fan@....com"
<peng.fan@....com>, "kernel@...gutronix.de" <kernel@...gutronix.de>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>, "festevam@...il.com"
<festevam@...il.com>, "robh@...nel.org" <robh@...nel.org>,
"shawnguo@...nel.org" <shawnguo@...nel.org>, "krzk+dt@...nel.org"
<krzk+dt@...nel.org>, "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
"imx@...ts.linux.dev" <imx@...ts.linux.dev>, Teresa Remmet
<t.remmet@...tec.de>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>
Subject: Re: [PATCH v5 0/4] Add display overlays for imx8mp-phyboard-pollux
Hi Shawn,
On Mon, 2025-10-20 at 20:11 +0800, Shawn Guo wrote:
> On Mon, Oct 13, 2025 at 08:25:05AM +0200, Yannic Moog wrote:
> > Yannic Moog (4):
> > arm64: dts: im8mp-phy{board,core}: update license
> > arm64: dts: imx8mp pollux: add display overlays
> > arm64: dts: imx8mp pollux: add expansion board overlay
> > arm64: dts: imx8mp pollux: add displays for expansion board
> >
> > arch/arm64/boot/dts/freescale/Makefile | 15 ++
> > .../imx8mp-phyboard-pollux-etml1010g3dra.dtso | 44 +++++
> > ...mp-phyboard-pollux-peb-av-10-etml1010g3dra.dtso | 45 +++++
> > ...8mp-phyboard-pollux-peb-av-10-ph128800t006.dtso | 45 +++++
> > .../imx8mp-phyboard-pollux-peb-av-10.dtsi | 196 +++++++++++++++++++++
> > .../imx8mp-phyboard-pollux-peb-av-10.dtso | 9 +
> > .../imx8mp-phyboard-pollux-ph128800t006.dtso | 45 +++++
> > .../dts/freescale/imx8mp-phyboard-pollux-rdk.dts | 52 ++----
> > .../boot/dts/freescale/imx8mp-phycore-som.dtsi | 3 +-
> > 9 files changed, 411 insertions(+), 43 deletions(-)
>
> Can we do something to avoid the warnings?
Yes, sorry about that; I didn't do proper due diligence on review comments.
>
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:102.3-16: Warning
> (reg_format): /fragment@..._overlay__/codec@18:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:117.3-16: Warning
> (reg_format): /fragment@..._overlay__/eeprom@57:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtbo: Warning (pci_device_reg):
> Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtbo: Warning
> (pci_device_bus_num): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtbo: Warning (i2c_bus_reg):
> Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtbo: Warning (spi_bus_reg):
> Failed prerequisite 'reg_format'
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #size-cells value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #size-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:102.3-16: Warning
> (reg_format): /fragment@..._overlay__/codec@18:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:117.3-16: Warning
> (reg_format): /fragment@..._overlay__/eeprom@57:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-etml1010g3dra.dtbo: Warning
> (pci_device_reg): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-etml1010g3dra.dtbo: Warning
> (pci_device_bus_num): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-etml1010g3dra.dtbo: Warning
> (i2c_bus_reg): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-etml1010g3dra.dtbo: Warning
> (spi_bus_reg): Failed prerequisite 'reg_format'
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #size-cells value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #size-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:102.3-16: Warning
> (reg_format): /fragment@..._overlay__/codec@18:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:117.3-16: Warning
> (reg_format): /fragment@..._overlay__/eeprom@57:reg: property has invalid length (4 bytes)
> (#address-cells == 2, #size-cells == 1)
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-ph128800t006.dtbo: Warning
> (pci_device_reg): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-ph128800t006.dtbo: Warning
> (pci_device_bus_num): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-ph128800t006.dtbo: Warning
> (i2c_bus_reg): Failed prerequisite 'reg_format'
> arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10-ph128800t006.dtbo: Warning
> (spi_bus_reg): Failed prerequisite 'reg_format'
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:100.18-113.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/codec@18: Relying on default #size-cells value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #address-cells
> value
> ../arch/arm64/boot/dts/freescale/imx8mp-phyboard-pollux-peb-av-10.dtsi:115.12-120.4: Warning
> (avoid_default_addr_size): /fragment@..._overlay__/eeprom@57: Relying on default #size-cells
> value
i2c4 node in peb-av-10.dtsi does need
#address-cells and #size-cells set.
I'll re-add them and resubmit.
Yannic
>
> Shawn
>
Powered by blists - more mailing lists