Hi Sergii,

On Fri, Mar 31, 2017 at 10:29 PM, Sergei Shtylyov
<[email protected]> wrote:
> The IPSR field names in the comments have been fat-fingered in a couple
> places --  fix those silly typos...

Thanks!

> Fixes: 508845196238 ("pinctrl: sh-pfc: r8a7791 PFC support")
> Signed-off-by: Sergei Shtylyov <[email protected]>

Reviewed-by: Geert Uytterhoeven <[email protected]>

> ---
> This patch is against the 'fixes' branch of Linus Walleij's 
> 'linux-pinctrl.git'
> repo plus  3 R8A7791 fixes posted in the last couple days...
>
>  drivers/pinctrl/sh-pfc/pfc-r8a7791.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> Index: linux-pinctrl/drivers/pinctrl/sh-pfc/pfc-r8a7791.c
> ===================================================================
> --- linux-pinctrl.orig/drivers/pinctrl/sh-pfc/pfc-r8a7791.c
> +++ linux-pinctrl/drivers/pinctrl/sh-pfc/pfc-r8a7791.c
> @@ -5711,7 +5711,7 @@ static const struct pinmux_cfg_reg pinmu
>         },
>         { PINMUX_CFG_REG_VAR("IPSR2", 0xE6060028, 32,
>                              2, 3, 2, 2, 2, 2, 3, 3, 3, 3, 2, 2, 3) {
> -               /* IP2_31_20 [2] */
> +               /* IP2_31_30 [2] */
>                 0, 0, 0, 0,
>                 /* IP2_29_27 [3] */
>                 FN_EX_CS3_N, FN_ATADIR0_N, FN_MSIOF2_TXD,

There's another one a few lines below:

-                  /* IP2_12_0 [3] */
+                 /* IP2_12_10 [3] */

I'll fix that up when applying...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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

Reply via email to