[PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
@ 2023-03-28 18:51 ` Fabio Estevam
  0 siblings, 0 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-03-28 18:51 UTC (permalink / raw)
  To: shawnguo
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

Building with W=1 leads to the following dtc warning:

arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property

Remove unnecessary #address-cells/#size-cells to fix it.

Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/imx7d-remarkable2.dts | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts
index dc954e4f63e0..92cb45dacda6 100644
--- a/arch/arm/boot/dts/imx7d-remarkable2.dts
+++ b/arch/arm/boot/dts/imx7d-remarkable2.dts
@@ -321,8 +321,6 @@ sy7636a: pmic@62 {
 		reg = <0x62>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_epdpmic>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 		#thermal-sensor-cells = <0>;
 		epd-pwr-good-gpios = <&gpio6 21 GPIO_ACTIVE_HIGH>;
 
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
@ 2023-03-28 18:51 ` Fabio Estevam
  0 siblings, 0 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-03-28 18:51 UTC (permalink / raw)
  To: shawnguo
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

Building with W=1 leads to the following dtc warning:

arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property

Remove unnecessary #address-cells/#size-cells to fix it.

Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/imx7d-remarkable2.dts | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts
index dc954e4f63e0..92cb45dacda6 100644
--- a/arch/arm/boot/dts/imx7d-remarkable2.dts
+++ b/arch/arm/boot/dts/imx7d-remarkable2.dts
@@ -321,8 +321,6 @@ sy7636a: pmic@62 {
 		reg = <0x62>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_epdpmic>;
-		#address-cells = <1>;
-		#size-cells = <0>;
 		#thermal-sensor-cells = <0>;
 		epd-pwr-good-gpios = <&gpio6 21 GPIO_ACTIVE_HIGH>;
 
-- 
2.34.1


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

* [PATCH 2/2] ARM: dts: imx6ull-colibri: Remove unnecessary #address-cells/#size-cells
  2023-03-28 18:51 ` Fabio Estevam
@ 2023-03-28 18:51   ` Fabio Estevam
  -1 siblings, 0 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-03-28 18:51 UTC (permalink / raw)
  To: shawnguo
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

Building with W=1 leads to the following dtc warning:

arch/arm/boot/dts/imx6ull-colibri.dtsi:36.9-46.5: Warning (graph_child_address): /connector/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary

Since a single port is used, 'ports' can be removed, as well as the
unnecessary #address-cells/#size-cells.

Fixes: bd5880e10982 ("ARM: dts: colibri-imx6ull: Enable dual-role switching")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/imx6ull-colibri.dtsi | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/imx6ull-colibri.dtsi b/arch/arm/boot/dts/imx6ull-colibri.dtsi
index bf64ba84b358..fde8a19aac0f 100644
--- a/arch/arm/boot/dts/imx6ull-colibri.dtsi
+++ b/arch/arm/boot/dts/imx6ull-colibri.dtsi
@@ -33,15 +33,9 @@ connector {
 		self-powered;
 		type = "micro";
 
-		ports {
-			#address-cells = <1>;
-			#size-cells = <0>;
-
-			port@0 {
-				reg = <0>;
-				usb_dr_connector: endpoint {
-					remote-endpoint = <&usb1_drd_sw>;
-				};
+		port {
+			usb_dr_connector: endpoint {
+				remote-endpoint = <&usb1_drd_sw>;
 			};
 		};
 	};
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 2/2] ARM: dts: imx6ull-colibri: Remove unnecessary #address-cells/#size-cells
@ 2023-03-28 18:51   ` Fabio Estevam
  0 siblings, 0 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-03-28 18:51 UTC (permalink / raw)
  To: shawnguo
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

Building with W=1 leads to the following dtc warning:

arch/arm/boot/dts/imx6ull-colibri.dtsi:36.9-46.5: Warning (graph_child_address): /connector/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary

Since a single port is used, 'ports' can be removed, as well as the
unnecessary #address-cells/#size-cells.

Fixes: bd5880e10982 ("ARM: dts: colibri-imx6ull: Enable dual-role switching")
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/imx6ull-colibri.dtsi | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/imx6ull-colibri.dtsi b/arch/arm/boot/dts/imx6ull-colibri.dtsi
index bf64ba84b358..fde8a19aac0f 100644
--- a/arch/arm/boot/dts/imx6ull-colibri.dtsi
+++ b/arch/arm/boot/dts/imx6ull-colibri.dtsi
@@ -33,15 +33,9 @@ connector {
 		self-powered;
 		type = "micro";
 
-		ports {
-			#address-cells = <1>;
-			#size-cells = <0>;
-
-			port@0 {
-				reg = <0>;
-				usb_dr_connector: endpoint {
-					remote-endpoint = <&usb1_drd_sw>;
-				};
+		port {
+			usb_dr_connector: endpoint {
+				remote-endpoint = <&usb1_drd_sw>;
 			};
 		};
 	};
-- 
2.34.1


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

* Re: [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
  2023-03-28 18:51 ` Fabio Estevam
@ 2023-03-28 21:02   ` Alistair
  -1 siblings, 0 replies; 8+ messages in thread
From: Alistair @ 2023-03-28 21:02 UTC (permalink / raw)
  To: Fabio Estevam, Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel,
	philippe.schenker, Fabio Estevam

On Wed, 29 Mar 2023, at 4:51 AM, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Building with W=1 leads to the following dtc warning:
> 
> arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Remove unnecessary #address-cells/#size-cells to fix it.
> 
> Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Alistair Francis <alistair@alistair23.me>

Alistair

> ---
> arch/arm/boot/dts/imx7d-remarkable2.dts | 2 --
> 1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts
> index dc954e4f63e0..92cb45dacda6 100644
> --- a/arch/arm/boot/dts/imx7d-remarkable2.dts
> +++ b/arch/arm/boot/dts/imx7d-remarkable2.dts
> @@ -321,8 +321,6 @@ sy7636a: pmic@62 {
> reg = <0x62>;
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_epdpmic>;
> - #address-cells = <1>;
> - #size-cells = <0>;
> #thermal-sensor-cells = <0>;
> epd-pwr-good-gpios = <&gpio6 21 GPIO_ACTIVE_HIGH>;
>  
> -- 
> 2.34.1
> 
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
@ 2023-03-28 21:02   ` Alistair
  0 siblings, 0 replies; 8+ messages in thread
From: Alistair @ 2023-03-28 21:02 UTC (permalink / raw)
  To: Fabio Estevam, Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-arm-kernel,
	philippe.schenker, Fabio Estevam

On Wed, 29 Mar 2023, at 4:51 AM, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Building with W=1 leads to the following dtc warning:
> 
> arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Remove unnecessary #address-cells/#size-cells to fix it.
> 
> Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Alistair Francis <alistair@alistair23.me>

Alistair

> ---
> arch/arm/boot/dts/imx7d-remarkable2.dts | 2 --
> 1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx7d-remarkable2.dts b/arch/arm/boot/dts/imx7d-remarkable2.dts
> index dc954e4f63e0..92cb45dacda6 100644
> --- a/arch/arm/boot/dts/imx7d-remarkable2.dts
> +++ b/arch/arm/boot/dts/imx7d-remarkable2.dts
> @@ -321,8 +321,6 @@ sy7636a: pmic@62 {
> reg = <0x62>;
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_epdpmic>;
> - #address-cells = <1>;
> - #size-cells = <0>;
> #thermal-sensor-cells = <0>;
> epd-pwr-good-gpios = <&gpio6 21 GPIO_ACTIVE_HIGH>;
>  
> -- 
> 2.34.1
> 
> 

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

* Re: [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
  2023-03-28 18:51 ` Fabio Estevam
@ 2023-04-05 14:31   ` Shawn Guo
  -1 siblings, 0 replies; 8+ messages in thread
From: Shawn Guo @ 2023-04-05 14:31 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

On Tue, Mar 28, 2023 at 03:51:46PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Building with W=1 leads to the following dtc warning:
> 
> arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Remove unnecessary #address-cells/#size-cells to fix it.
> 
> Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied both, thanks!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells
@ 2023-04-05 14:31   ` Shawn Guo
  0 siblings, 0 replies; 8+ messages in thread
From: Shawn Guo @ 2023-04-05 14:31 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: robh+dt, krzysztof.kozlowski+dt, devicetree, linux-arm-kernel,
	alistair, philippe.schenker, Fabio Estevam

On Tue, Mar 28, 2023 at 03:51:46PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Building with W=1 leads to the following dtc warning:
> 
> arch/arm/boot/dts/imx7d-remarkable2.dts:319.19-335.4: Warning (avoid_unnecessary_addr_size): /soc/bus@30800000/i2c@30a50000/pmic@62: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Remove unnecessary #address-cells/#size-cells to fix it.
> 
> Fixes: 9076cbaa7757 ("ARM: dts: imx7d-remarkable2: Enable silergy,sy7636a")
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied both, thanks!

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

end of thread, other threads:[~2023-04-05 14:32 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-28 18:51 [PATCH 1/2] ARM: dts: imx7d-remarkable2: Remove unnecessary #address-cells/#size-cells Fabio Estevam
2023-03-28 18:51 ` Fabio Estevam
2023-03-28 18:51 ` [PATCH 2/2] ARM: dts: imx6ull-colibri: " Fabio Estevam
2023-03-28 18:51   ` Fabio Estevam
2023-03-28 21:02 ` [PATCH 1/2] ARM: dts: imx7d-remarkable2: " Alistair
2023-03-28 21:02   ` Alistair
2023-04-05 14:31 ` Shawn Guo
2023-04-05 14:31   ` Shawn Guo

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.