All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells
@ 2020-08-17 18:01 Fabio Estevam
  2020-08-17 18:01 ` [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name Fabio Estevam
  2020-08-23  2:06 ` [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells Shawn Guo
  0 siblings, 2 replies; 4+ messages in thread
From: Fabio Estevam @ 2020-08-17 18:01 UTC (permalink / raw)
  To: shawnguo; +Cc: o.rempel, tharvey, robh+dt, Fabio Estevam, linux-arm-kernel

The following dtc warning is seen when building with W=1:

arch/arm/boot/dts/imx6qdl-gw51xx.dtsi:23.12-68.4: Warning (avoid_unnecessary_addr_size): /gpio-keys: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property

Remove the unneeded #address-cells/#size-cells properties to fix the issue.

Fixes: 64bf0a0af18d ("ARM: dts: imx6qdl-gw: add Gateworks System Controller support")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/imx6qdl-gw51xx.dtsi | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/imx6qdl-gw51xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw51xx.dtsi
index 7705285d9e3c..4d01c3300b97 100644
--- a/arch/arm/boot/dts/imx6qdl-gw51xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw51xx.dtsi
@@ -22,8 +22,6 @@
 
 	gpio-keys {
 		compatible = "gpio-keys";
-		#address-cells = <1>;
-		#size-cells = <0>;
 
 		user-pb {
 			label = "user_pb";
-- 
2.17.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] 4+ messages in thread

* [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name
  2020-08-17 18:01 [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells Fabio Estevam
@ 2020-08-17 18:01 ` Fabio Estevam
  2020-08-18  5:07   ` Oleksij Rempel
  2020-08-23  2:06 ` [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells Shawn Guo
  1 sibling, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2020-08-17 18:01 UTC (permalink / raw)
  To: shawnguo; +Cc: o.rempel, tharvey, robh+dt, Fabio Estevam, linux-arm-kernel

The following dtc warning is seen when building with W=1:

arch/arm/boot/dts/imx6q-prtwd2.dts:33.8-43.4: Warning (unit_address_vs_reg): /i2c@4: node has a unit name, but no reg or ranges property

Remove the unneeded i2c unit name to fix the issue.

Fixes: 88010b8174ab ("ARM: dts: add Protonic WD2 board")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/imx6q-prtwd2.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/imx6q-prtwd2.dts b/arch/arm/boot/dts/imx6q-prtwd2.dts
index dffafbcaa7af..349959d38020 100644
--- a/arch/arm/boot/dts/imx6q-prtwd2.dts
+++ b/arch/arm/boot/dts/imx6q-prtwd2.dts
@@ -30,7 +30,7 @@
 	};
 
 	/* PRTWD2 rev 1 bitbang I2C for Ethernet Switch */
-	i2c@4 {
+	i2c {
 		compatible = "i2c-gpio";
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_i2c4>;
-- 
2.17.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] 4+ messages in thread

* Re: [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name
  2020-08-17 18:01 ` [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name Fabio Estevam
@ 2020-08-18  5:07   ` Oleksij Rempel
  0 siblings, 0 replies; 4+ messages in thread
From: Oleksij Rempel @ 2020-08-18  5:07 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: tharvey, robh+dt, shawnguo, linux-arm-kernel

On Mon, Aug 17, 2020 at 03:01:43PM -0300, Fabio Estevam wrote:
> The following dtc warning is seen when building with W=1:
> 
> arch/arm/boot/dts/imx6q-prtwd2.dts:33.8-43.4: Warning (unit_address_vs_reg): /i2c@4: node has a unit name, but no reg or ranges property
> 
> Remove the unneeded i2c unit name to fix the issue.
> 
> Fixes: 88010b8174ab ("ARM: dts: add Protonic WD2 board")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Thank you!

Reviewed-by: Oleksij Rempel <o.rempel@pengutronix.de>

> ---
>  arch/arm/boot/dts/imx6q-prtwd2.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/imx6q-prtwd2.dts b/arch/arm/boot/dts/imx6q-prtwd2.dts
> index dffafbcaa7af..349959d38020 100644
> --- a/arch/arm/boot/dts/imx6q-prtwd2.dts
> +++ b/arch/arm/boot/dts/imx6q-prtwd2.dts
> @@ -30,7 +30,7 @@
>  	};
>  
>  	/* PRTWD2 rev 1 bitbang I2C for Ethernet Switch */
> -	i2c@4 {
> +	i2c {
>  		compatible = "i2c-gpio";
>  		pinctrl-names = "default";
>  		pinctrl-0 = <&pinctrl_i2c4>;
> -- 
> 2.17.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
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] 4+ messages in thread

* Re: [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells
  2020-08-17 18:01 [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells Fabio Estevam
  2020-08-17 18:01 ` [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name Fabio Estevam
@ 2020-08-23  2:06 ` Shawn Guo
  1 sibling, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2020-08-23  2:06 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: o.rempel, tharvey, robh+dt, linux-arm-kernel

On Mon, Aug 17, 2020 at 03:01:42PM -0300, Fabio Estevam wrote:
> The following dtc warning is seen when building with W=1:
> 
> arch/arm/boot/dts/imx6qdl-gw51xx.dtsi:23.12-68.4: Warning (avoid_unnecessary_addr_size): /gpio-keys: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> 
> Remove the unneeded #address-cells/#size-cells properties to fix the issue.
> 
> Fixes: 64bf0a0af18d ("ARM: dts: imx6qdl-gw: add Gateworks System Controller support")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

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] 4+ messages in thread

end of thread, other threads:[~2020-08-23  2:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-17 18:01 [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells Fabio Estevam
2020-08-17 18:01 ` [PATCH 2/2] ARM: dts: imx6q-prtwd2: Remove unneeded i2c unit name Fabio Estevam
2020-08-18  5:07   ` Oleksij Rempel
2020-08-23  2:06 ` [PATCH 1/2] ARM: dts: imx6qdl-gw51xx: Remove unneeded #address-cells/#size-cells 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.