All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too
@ 2022-01-21 15:42 Geert Uytterhoeven
  2022-01-22 11:04 ` Biju Das
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2022-01-21 15:42 UTC (permalink / raw)
  To: Lad Prabhakar, Biju Das; +Cc: linux-renesas-soc, Geert Uytterhoeven

"make dtbs" complains when "pinctrl-names" is present without "pinctrl-0".

Fixes: 5c0d1874072bef54 ("arm64: dts: renesas: Add initial device tree for RZ/G2LC SMARC EVK")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
To be folded into the original in renesas-arm-dt-for-v5.18.

 arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
index 53845823d0dce9cc..728a2275ea8d21ac 100644
--- a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
+++ b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
@@ -19,46 +19,55 @@ / {
 
 &canfd {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &ehci0 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &ehci1 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &hsusb {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &i2c0 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &i2c1 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &i2c3 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &ohci0 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &ohci1 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
@@ -68,32 +77,38 @@ &phyrst {
 
 &scif2 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &sdhi1 {
 	/delete-property/ pinctrl-0;
 	/delete-property/ pinctrl-1;
+	/delete-property/ pinctrl-names;
 	/delete-property/ vmmc-supply;
 	status = "disabled";
 };
 
 &spi1 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &ssi0 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &usb2_phy0 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
 
 &usb2_phy1 {
 	/delete-property/ pinctrl-0;
+	/delete-property/ pinctrl-names;
 	status = "disabled";
 };
-- 
2.25.1


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

* RE: [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too
  2022-01-21 15:42 [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too Geert Uytterhoeven
@ 2022-01-22 11:04 ` Biju Das
  2022-01-24 10:06 ` Niklas Söderlund
  2022-01-24 10:48 ` Prabhakar Mahadev Lad
  2 siblings, 0 replies; 4+ messages in thread
From: Biju Das @ 2022-01-22 11:04 UTC (permalink / raw)
  To: Geert Uytterhoeven, Prabhakar Mahadev Lad; +Cc: linux-renesas-soc

Hi Geert,

Thank you for fixing this.

> Subject: [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded
> pinctrl-names, too
> 
> "make dtbs" complains when "pinctrl-names" is present without "pinctrl-0".
> 
> Fixes: 5c0d1874072bef54 ("arm64: dts: renesas: Add initial device tree for
> RZ/G2LC SMARC EVK")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Biju Das <biju.das.jz@bp.renesas.com>

Regards,
Biju

> ---
> To be folded into the original in renesas-arm-dt-for-v5.18.
> 
>  arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> index 53845823d0dce9cc..728a2275ea8d21ac 100644
> --- a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> @@ -19,46 +19,55 @@ / {
> 
>  &canfd {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ehci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ehci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &hsusb {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c3 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ohci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ohci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
> @@ -68,32 +77,38 @@ &phyrst {
> 
>  &scif2 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &sdhi1 {
>  	/delete-property/ pinctrl-0;
>  	/delete-property/ pinctrl-1;
> +	/delete-property/ pinctrl-names;
>  	/delete-property/ vmmc-supply;
>  	status = "disabled";
>  };
> 
>  &spi1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ssi0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &usb2_phy0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &usb2_phy1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> --
> 2.25.1


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

* Re: [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too
  2022-01-21 15:42 [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too Geert Uytterhoeven
  2022-01-22 11:04 ` Biju Das
@ 2022-01-24 10:06 ` Niklas Söderlund
  2022-01-24 10:48 ` Prabhakar Mahadev Lad
  2 siblings, 0 replies; 4+ messages in thread
From: Niklas Söderlund @ 2022-01-24 10:06 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Lad Prabhakar, Biju Das, linux-renesas-soc

Hi Geert,

Thanks for your work.

On 2022-01-21 16:42:17 +0100, Geert Uytterhoeven wrote:
> "make dtbs" complains when "pinctrl-names" is present without "pinctrl-0".
> 
> Fixes: 5c0d1874072bef54 ("arm64: dts: renesas: Add initial device tree for RZ/G2LC SMARC EVK")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> ---
> To be folded into the original in renesas-arm-dt-for-v5.18.
> 
>  arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> index 53845823d0dce9cc..728a2275ea8d21ac 100644
> --- a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> @@ -19,46 +19,55 @@ / {
>  
>  &canfd {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &ehci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &ehci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &hsusb {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &i2c0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &i2c1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &i2c3 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &ohci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &ohci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
> @@ -68,32 +77,38 @@ &phyrst {
>  
>  &scif2 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &sdhi1 {
>  	/delete-property/ pinctrl-0;
>  	/delete-property/ pinctrl-1;
> +	/delete-property/ pinctrl-names;
>  	/delete-property/ vmmc-supply;
>  	status = "disabled";
>  };
>  
>  &spi1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &ssi0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &usb2_phy0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
>  
>  &usb2_phy1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> -- 
> 2.25.1
> 

-- 
Kind Regards,
Niklas Söderlund

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

* RE: [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too
  2022-01-21 15:42 [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too Geert Uytterhoeven
  2022-01-22 11:04 ` Biju Das
  2022-01-24 10:06 ` Niklas Söderlund
@ 2022-01-24 10:48 ` Prabhakar Mahadev Lad
  2 siblings, 0 replies; 4+ messages in thread
From: Prabhakar Mahadev Lad @ 2022-01-24 10:48 UTC (permalink / raw)
  To: Geert Uytterhoeven, Biju Das; +Cc: linux-renesas-soc

Hi Geert,

Thank you for the patch.

> -----Original Message-----
> From: Geert Uytterhoeven <geert+renesas@glider.be>
> Sent: 21 January 2022 15:42
> To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>; Biju Das
> <biju.das.jz@bp.renesas.com>
> Cc: linux-renesas-soc@vger.kernel.org; Geert Uytterhoeven <geert+renesas@glider.be>
> Subject: [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too
> 
> "make dtbs" complains when "pinctrl-names" is present without "pinctrl-0".
> 
> Fixes: 5c0d1874072bef54 ("arm64: dts: renesas: Add initial device tree for RZ/G2LC SMARC EVK")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> To be folded into the original in renesas-arm-dt-for-v5.18.
> 
>  arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Cheers,
Prabhakar

> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> index 53845823d0dce9cc..728a2275ea8d21ac 100644
> --- a/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a07g044c2-smarc.dts
> @@ -19,46 +19,55 @@ / {
> 
>  &canfd {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ehci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ehci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &hsusb {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &i2c3 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ohci0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ohci1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
> @@ -68,32 +77,38 @@ &phyrst {
> 
>  &scif2 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &sdhi1 {
>  	/delete-property/ pinctrl-0;
>  	/delete-property/ pinctrl-1;
> +	/delete-property/ pinctrl-names;
>  	/delete-property/ vmmc-supply;
>  	status = "disabled";
>  };
> 
>  &spi1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &ssi0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &usb2_phy0 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> 
>  &usb2_phy1 {
>  	/delete-property/ pinctrl-0;
> +	/delete-property/ pinctrl-names;
>  	status = "disabled";
>  };
> --
> 2.25.1


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

end of thread, other threads:[~2022-01-24 10:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-21 15:42 [PATCH] arm64: dts: renesas: r9a07g044c2-smarc: Drop unneeded pinctrl-names, too Geert Uytterhoeven
2022-01-22 11:04 ` Biju Das
2022-01-24 10:06 ` Niklas Söderlund
2022-01-24 10:48 ` Prabhakar Mahadev Lad

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.