Re: [PATCH v4 16/23] pinctrl: renesas: r8a779g0: add missing SCIF1_X

[Geert Uytterhoeven <geert@linux-m68k.org>]

Hi Morimoto-san,

On Fri, Jul 1, 2022 at 3:39 AM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>
> This patch adds missing SCIF1_X.
> Because Document (Rev.0.51) has 2xSCIF1 with no suffix (_A, _B),
> this patch name it as _X.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thanks for your patch!

> --- a/drivers/pinctrl/renesas/pfc-r8a779g0.c
> +++ b/drivers/pinctrl/renesas/pfc-r8a779g0.c

> @@ -2501,9 +2529,12 @@ static const struct sh_pfc_pin_group pinmux_groups[] = {
>         SH_PFC_PIN_GROUP(scif0_data),
>         SH_PFC_PIN_GROUP(scif0_clk),
>         SH_PFC_PIN_GROUP(scif0_ctrl),
> -       SH_PFC_PIN_GROUP(scif1_data),
> -       SH_PFC_PIN_GROUP(scif1_clk),
> -       SH_PFC_PIN_GROUP(scif1_ctrl),
> +       SH_PFC_PIN_GROUP(scif1_data),           /* suffix might be updated */
> +       SH_PFC_PIN_GROUP(scif1_data_x),         /* suffix might be updated */
> +       SH_PFC_PIN_GROUP(scif1_clk),            /* suffix might be updated */
> +       SH_PFC_PIN_GROUP(scif1_clk_x),          /* suffix might be updated */
> +       SH_PFC_PIN_GROUP(scif1_ctrl),           /* suffix might be updated */
> +       SH_PFC_PIN_GROUP(scif1_ctrl_x),         /* suffix might be updated */
>         SH_PFC_PIN_GROUP(scif3_data),           /* suffix might be updated */
>         SH_PFC_PIN_GROUP(scif3_data_a),         /* suffix might be updated */
>         SH_PFC_PIN_GROUP(scif3_clk),            /* suffix might be updated */
> @@ -2786,9 +2817,13 @@ static const char * const scif0_groups[] = {
>  };
>
>  static const char * const scif1_groups[] = {
> +       /* suffix might be updated */
>         "scif1_data",
> +       "scif1_data_x",
>         "scif1_clk",
> +       "scif1_clk_x",
>         "scif1_ctrl",
> +       "scif1_ctrl_x",
>  };
>
>  static const char * const scif3_groups[] = {

I'll sort the above while applying (all *_x last).

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-pinctrl-for-v5.20.

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