All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: "Andrzej Hajda" <andrzej.hajda@intel.com>,
	"Neil Armstrong" <neil.armstrong@linaro.org>,
	"Robert Foss" <rfoss@kernel.org>,
	"Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>,
	"Jonas Karlman" <jonas@kwiboo.se>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Luca Ceresoli" <luca.ceresoli@bootlin.com>,
	"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
	"Maxime Ripard" <mripard@kernel.org>,
	"Thomas Zimmermann" <tzimmermann@suse.de>,
	"David Airlie" <airlied@gmail.com>,
	"Simona Vetter" <simona@ffwll.ch>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Guido Gúnther" <agx@sigxcpu.org>,
	"Robert Chiras" <robert.chiras@nxp.com>,
	"Tim Harvey" <tharvey@gateworks.com>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Pengutronix Kernel Team" <kernel@pengutronix.de>,
	"Fabio Estevam" <festevam@gmail.com>,
	Frank.Li@oss.nxp.com
Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org, linux@ew.tq-group.com,
	Frank Li <Frank.Li@nxp.com>
Subject: Re: [PATCH 6/8] arm64: dts: imx: remove unnecessary #address-cells and #size-cells
Date: Mon, 28 Sep 2026 11:45:54 +0200	[thread overview]
Message-ID: <2280028.irdbgypaU6@steina-w> (raw)
In-Reply-To: <20260925-cleanup_dtb_warning-v1-6-f999fafbca60@nxp.com>

Am Freitag, 25. September 2026, 18:37:55 CEST schrieb Frank.Li@oss.nxp.com:
> From: Frank Li <Frank.Li@nxp.com>
> 
> Remove unnecessary #address-cells and #size-cells to fix the dtb W=1 warning:
>   Warning (avoid_unnecessary_addr_size): /soc@0/bus@30800000/i2c@30a20000/gsc@20: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" or "ranges" property
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com> # TQMLX2160A

> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts       | 4 ----
>  arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi | 4 ----
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts    | 2 --
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts    | 2 --
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts    | 2 --
>  arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts    | 2 --
>  arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts    | 2 --
>  arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts         | 2 --
>  arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts      | 2 --
>  9 files changed, 22 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> index f51508952d51c..b762ad555ad9e 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-ten64.dts
> @@ -312,8 +312,6 @@ &qspi {
>  
>  	en25s64: flash@0 {
>  		compatible = "jedec,spi-nor";
> -		#address-cells = <1>;
> -		#size-cells = <1>;
>  		reg = <0>;
>  		spi-max-frequency = <20000000>;
>  		spi-rx-bus-width = <4>;
> @@ -363,8 +361,6 @@ partition@600000 {
>  
>  	nand: flash@1 {
>  		compatible = "spi-nand";
> -		#address-cells = <1>;
> -		#size-cells = <1>;
>  		reg = <1>;
>  		spi-max-frequency = <20000000>;
>  		spi-rx-bus-width = <4>;
> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
> index cc6b6427ec8a6..aac0ae0ad40c7 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-tqmlx2160a.dtsi
> @@ -41,8 +41,6 @@ &fspi {
>  	flash0: flash@0 {
>  		compatible = "jedec,spi-nor";
>  		reg = <0>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
>  		spi-max-frequency = <10000000>;
>  		spi-rx-bus-width = <1>;
>  		spi-tx-bus-width = <1>;
> @@ -57,8 +55,6 @@ partitions {
>  	flash1: flash@1 {
>  		compatible = "jedec,spi-nor";
>  		reg = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
>  		spi-max-frequency = <10000000>;
>  		spi-rx-bus-width = <1>;
>  		spi-tx-bus-width = <1>;
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> index 272c2b223d167..1bb42116fc0b8 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> @@ -366,8 +366,6 @@ gsc: gsc@20 {
>  		interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
>  		interrupt-controller;
>  		#interrupt-cells = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
>  
>  		adc {
>  			compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> index 468c7e993c52b..f53718221ad5b 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> @@ -317,8 +317,6 @@ gsc: gsc@20 {
>  		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
>  		interrupt-controller;
>  		#interrupt-cells = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
>  
>  		adc {
>  			compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> index 636daa3d6ca20..3eafff879e8d5 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> @@ -282,8 +282,6 @@ gsc: gsc@20 {
>  		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
>  		interrupt-controller;
>  		#interrupt-cells = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
>  
>  		adc {
>  			compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> index 99572961d9e16..fcbb40597b12c 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> @@ -335,8 +335,6 @@ gsc: gsc@20 {
>  		interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
>  		interrupt-controller;
>  		#interrupt-cells = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
>  
>  		adc {
>  			compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> index 5aa0e2cd155ec..ecc12645d7691 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> @@ -315,8 +315,6 @@ gsc: gsc@20 {
>  		interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
>  		interrupt-controller;
>  		#interrupt-cells = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
>  
>  		adc {
>  			compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts b/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
> index b91509bfc9c00..1043bf074f8e4 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mq-nitrogen.dts
> @@ -287,8 +287,6 @@ &lcdif {
>  };
>  
>  &mipi_dsi {
> -	#address-cells = <1>;
> -	#size-cells = <0>;
>  	status = "okay";
>  
>  	ports {
> diff --git a/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts b/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
> index c6019d76b9981..f3b647267c300 100644
> --- a/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
> +++ b/arch/arm64/boot/dts/freescale/imx91-11x11-frdm-s.dts
> @@ -202,8 +202,6 @@ &flexspi1 {
>  	flash@0 {
>  		compatible = "spi-nand";
>  		reg = <0>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
>  		spi-max-frequency = <104000000>;
>  		spi-rx-bus-width = <4>;
>  		spi-tx-bus-width = <4>;
> 
> 


-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/



  reply	other threads:[~2026-09-28  9:46 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-25 16:37 [PATCH 0/8] arm64: dts: imx: cleanup some warning with W=1 Frank.Li
2026-09-25 16:37 ` [PATCH 1/8] dt-bindings: display: remove #address-cells and #size-cells off requires Frank.Li
2026-09-28  9:42   ` Alexander Stein
2026-09-30  9:36   ` Krzysztof Kozlowski
2026-09-25 16:37 ` [PATCH 2/8] dt-bindings: embedded-controller: remove #address-cells and #size-cells from required Frank.Li
2026-09-30  9:59   ` Krzysztof Kozlowski
2026-09-30 15:56     ` Frank Li
2026-09-25 16:37 ` [PATCH 3/8] arm64: dts: s32n79: add @addr for *-bus node Frank.Li
2026-09-25 16:37 ` [PATCH 4/8] arm64: dts: imx8mq: remove #address-cells and #size-cells in dsi@30a00000 Frank.Li
2026-09-28  9:42   ` Alexander Stein
2026-09-25 16:37 ` [PATCH 5/8] arm64: dts: fsl-lx216x: remove #address-cells and #size-cells from pinctrl node Frank.Li
2026-09-25 16:37 ` [PATCH 6/8] arm64: dts: imx: remove unnecessary #address-cells and #size-cells Frank.Li
2026-09-28  9:45   ` Alexander Stein [this message]
2026-09-25 16:37 ` [PATCH 7/8] arm64: dts: imx95: rename bus@4c100000 to hsio-bus Frank.Li
2026-09-28  9:47   ` Alexander Stein
2026-09-25 16:37 ` [PATCH 8/8] arm64: dts: imx952: rename soc to soc@0 Frank.Li
2026-09-28 11:18   ` Daniel Baluta

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=2280028.irdbgypaU6@steina-w \
    --to=alexander.stein@ew.tq-group.com \
    --cc=Frank.Li@nxp.com \
    --cc=Frank.Li@oss.nxp.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=agx@sigxcpu.org \
    --cc=airlied@gmail.com \
    --cc=andrzej.hajda@intel.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=imx@lists.linux.dev \
    --cc=jernej.skrabec@gmail.com \
    --cc=jonas@kwiboo.se \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@ew.tq-group.com \
    --cc=luca.ceresoli@bootlin.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=rfoss@kernel.org \
    --cc=robert.chiras@nxp.com \
    --cc=robh@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=simona@ffwll.ch \
    --cc=tharvey@gateworks.com \
    --cc=tzimmermann@suse.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 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.