lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ