Re: [PATCH 4/4] arm64: dts: renesas: r9a08g046l48-smarc: Enable I2C{2,3} devices

[Geert Uytterhoeven]

Hi Biju,

On Tue, 5 May 2026 at 09:02, Biju <biju.das.au@xxxxxxxxx> wrote:
> From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
>
> Enable I2C{2,3} on RZ/G3L SMARC EVK board. The I2C3 is enabled by
> setting SW SYS.2 to OFF position.
>
> Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc.dts
> @@ -11,6 +11,7 @@
>  #define RZ_BOOT_MODE3          0
>  #define SW_DPI_EN              0
>  #define SW_GPIO4               0
> +#define SW_I3C_EN              0

FTR, this macro is active-high, like SW_DPI_EN, good.

>  #define PMOD_GPIO4             0
>  #define PMOD_GPIO6             0

> --- a/arch/arm64/boot/dts/renesas/rzg3l-smarc-som.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg3l-smarc-som.dtsi
> @@ -9,6 +9,10 @@
>   * Please set the below switch position on the SoM and the corresponding macro
>   * on the board DTS:
>   *
> + * Switch position SYS.2, Macro SW_I3C_EN:
> + *      0 - SMARC_I2C_GP is enabled
> + *      1 - I3C is enabled

while the SW_I3C_EN# is active-low, so setting the switch ON pulls
the signal low, and enables I3C, good.

Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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