Hi Claudiu, On Mon, Nov 20, 2023 at 8:01 AM Claudiu <claudiu.beznea@xxxxxxxxx> wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > Move arg in the main block of the function as this is used by 3 out of 4 > case blocks of switch-case. In this way some lines of code are removed. > > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> Thanks for your patch! > drivers/pinctrl/renesas/pinctrl-rzg2l.c | 11 ++++++----- > 1 file changed, 6 insertions(+), 5 deletions(-) Unfortunately your claim is not really backed by the diffstat. What about moving index, too? 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