Hi Sergei, Thanks for your patch! On Fri, Apr 13, 2018 at 8:29 PM, Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote: > Add the pin I/O voltage level control to the R8A77980 PFC driver. Subject says r8a77970? > Loosely based on the original (and large) patch by Vladimir Barinov. > > Signed-off-by: Vladimir Barinov <vladimir.barinov@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> > > --- > drivers/pinctrl/sh-pfc/pfc-r8a77980.c | 50 +++++++++++++++++++++++++++++++--- > 1 file changed, 47 insertions(+), 3 deletions(-) > > Index: renesas-drivers/drivers/pinctrl/sh-pfc/pfc-r8a77980.c > =================================================================== > --- renesas-drivers.orig/drivers/pinctrl/sh-pfc/pfc-r8a77980.c > +++ renesas-drivers/drivers/pinctrl/sh-pfc/pfc-r8a77980.c Ah, pfc-r8a77980.c it is. > @@ -2779,8 +2779,51 @@ static const struct pinmux_cfg_reg pinmu > { }, > }; > > +enum ioctrl_regs { > + IOCTRL30, > + IOCTRL31, > + IOCTRL32, > +}; > + > +static const struct pinmux_ioctrl_reg pinmux_ioctrl_regs[] = { > + [IOCTRL30] = { 0xe6060380, }, > + [IOCTRL31] = { 0xe6060384, }, > + [IOCTRL32] = { 0xe6060388, }, > + { /* sentinel */ }, However, r8a77980 has 4 IOCTRL3x registers (r8a77970 has three)? Something is wrong: which SoC source file is this patch for? 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