Hi Geert, Neat series. I always worry when my eyes start to cross each other when looking at the big tables I will make mistakes like this ;-) On 2020-01-10 14:19:14 +0100, Geert Uytterhoeven wrote: > Hi all, > > This patch series contains various improvements for the builtin pin > control table runtime checks of the Renesas Pin Function Controller > driver. These checks are enabled with CONFIG_DEBUG_PINCTRL=y, which can > be combined with CONFIG_COMPILE_TEST=y to increase coverage to all > Renesas ARM and SuperH SoCs.. > > Note that all issues detected by this have already been fixed in "[PATCH > 0/6] pinctrl: sh-pfc: More miscellenaous fixes"[1], and are now part of > linux-next. > > I plan to queue this in sh-pfc-for-v5.7. With the fix you point out yourself in 13/13 for CONFIG_PINCTRL_SH_FUNC_GPIO=n feel free to add Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> For the whole series. > > Thanks for your comments! > > [1] https://lore.kernel.org/linux-renesas-soc/20191218194812.12741-1-geert+renesas@xxxxxxxxx/ > > Geert Uytterhoeven (13): > pinctrl: sh-pfc: checker: Move data before code > pinctrl: sh-pfc: checker: Add helpers for reporting > pinctrl: sh-pfc: checker: Add helper for safe name comparison > pinctrl: sh-pfc: checker: Add check for config register conflicts > pinctrl: sh-pfc: checker: Add check for enum ID conflicts > pinctrl: sh-pfc: checker: Improve pin checks > pinctrl: sh-pfc: checker: Improve pin function checks > pinctrl: sh-pfc: checker: Improve pin group checks > pinctrl: sh-pfc: checker: Add drive strength register checks > pinctrl: sh-pfc: checker: Add bias register checks > pinctrl: sh-pfc: checker: Add ioctrl register checks > pinctrl: sh-pfc: checker: Add data register checks > pinctrl: sh-pfc: checker: Add function GPIO checks > > drivers/pinctrl/sh-pfc/core.c | 312 +++++++++++++++++++++++++++------- > 1 file changed, 250 insertions(+), 62 deletions(-) > > -- > 2.17.1 > > 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 -- Regards, Niklas Söderlund