From: Yannic Moog <y.moog@phytec.de>
To: "shawnguo2@yeah.net" <shawnguo2@yeah.net>
Cc: "upstream@lists.phytec.de" <upstream@lists.phytec.de>,
"peng.fan@nxp.com" <peng.fan@nxp.com>,
"kernel@pengutronix.de" <kernel@pengutronix.de>,
"s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
"festevam@gmail.com" <festevam@gmail.com>,
"robh@kernel.org" <robh@kernel.org>,
"shawnguo@kernel.org" <shawnguo@kernel.org>,
"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"conor+dt@kernel.org" <conor+dt@kernel.org>,
"imx@lists.linux.dev" <imx@lists.linux.dev>,
Teresa Remmet <t.remmet@phytec.de>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH v5 0/4] Add display overlays for imx8mp-phyboard-pollux
Date: Mon, 20 Oct 2025 12:43:02 +0000 [thread overview]
Message-ID: <54efc169e9c82b52a96707f4fa8fedfcdc9242f6.camel@phytec.de> (raw)
In-Reply-To: <aPYnD25ljde33_tq@dragon>
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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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@1/__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
>
prev parent reply other threads:[~2025-10-20 12:43 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-13 6:25 [PATCH v5 0/4] Add display overlays for imx8mp-phyboard-pollux Yannic Moog
2025-10-13 6:25 ` [PATCH v5 1/4] arm64: dts: im8mp-phy{board,core}: update license Yannic Moog
2025-10-13 6:25 ` [PATCH v5 2/4] arm64: dts: imx8mp pollux: add display overlays Yannic Moog
2025-10-13 6:25 ` [PATCH v5 3/4] arm64: dts: imx8mp pollux: add expansion board overlay Yannic Moog
2025-10-13 6:25 ` [PATCH v5 4/4] arm64: dts: imx8mp pollux: add displays for expansion board Yannic Moog
2025-10-20 12:11 ` [PATCH v5 0/4] Add display overlays for imx8mp-phyboard-pollux Shawn Guo
2025-10-20 12:43 ` Yannic Moog [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54efc169e9c82b52a96707f4fa8fedfcdc9242f6.camel@phytec.de \
--to=y.moog@phytec.de \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=peng.fan@nxp.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo2@yeah.net \
--cc=shawnguo@kernel.org \
--cc=t.remmet@phytec.de \
--cc=upstream@lists.phytec.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox