[PATCH v2] arm64: dts: renesas: rzg2lc-smarc: Disable CAN-FD channel0

Biju posted 1 patch 2 months ago
arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
[PATCH v2] arm64: dts: renesas: rzg2lc-smarc: Disable CAN-FD channel0
Posted by Biju 2 months ago
From: Biju Das <biju.das.jz@bp.renesas.com>

On RZ/G2LC SMARC EVK, CAN-FD channel0 is not populated and channel0 is
a required property. Currently we are deleting a wrong node. Fixing the
wrong node invoked dtb warning message. Disable CAN-FD channel0 instead
of deleting the node.

Fixes: 46da632734a5 ("arm64: dts: renesas: rzg2lc-smarc: Enable CANFD channel 1")
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
v1->v2:
 * Updated the commit header and description.
 * Fixed the bot warning by disabling the channel instead of deleting it.
---
 arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
index 345b779e4f60..0d357516e0be 100644
--- a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
+++ b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
@@ -48,7 +48,9 @@ sound_card {
 #if (SW_SCIF_CAN || SW_RSPI_CAN)
 &canfd {
 	pinctrl-0 = <&can1_pins>;
-	/delete-node/ channel@0;
+	channel0 {
+		status = "disabled";
+	};
 };
 #else
 &canfd {
-- 
2.43.0
Re: [PATCH v2] arm64: dts: renesas: rzg2lc-smarc: Disable CAN-FD channel0
Posted by Geert Uytterhoeven 2 months ago
Hi Biju,

Thanks for your patch!

On Fri, 1 Aug 2025 at 14:20, Biju <biju.das.au@gmail.com> wrote:
> From: Biju Das <biju.das.jz@bp.renesas.com>
>
> On RZ/G2LC SMARC EVK, CAN-FD channel0 is not populated and channel0 is
> a required property. Currently we are deleting a wrong node. Fixing the

s/wrong/nonexistent/

> wrong node invoked dtb warning message. Disable CAN-FD channel0 instead
> of deleting the node.
>
> Fixes: 46da632734a5 ("arm64: dts: renesas: rzg2lc-smarc: Enable CANFD channel 1")
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> v1->v2:
>  * Updated the commit header and description.
>  * Fixed the bot warning by disabling the channel instead of deleting it.
> ---
>  arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> index 345b779e4f60..0d357516e0be 100644
> --- a/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg2lc-smarc.dtsi
> @@ -48,7 +48,9 @@ sound_card {
>  #if (SW_SCIF_CAN || SW_RSPI_CAN)
>  &canfd {
>         pinctrl-0 = <&can1_pins>;
> -       /delete-node/ channel@0;

Please insert a blank line between properties and nodes.

> +       channel0 {
> +               status = "disabled";
> +       };
>  };
>  #else
>  &canfd {

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.18 with the above fixed.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds