Hi Bartosz, On Mon, Feb 1, 2021 at 9:59 AM Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> wrote: > On Wed, Jan 27, 2021 at 11:39 AM Wolfram Sang > <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > > On Tue, Jan 05, 2021 at 12:14:34PM +0100, Geert Uytterhoeven wrote: > > > On Tue, Jan 5, 2021 at 12:00 PM Wolfram Sang > > > <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > After refactoring, we had two variables for the same thing. Remove the > > > > second declaration, one is enough here. Found by cppcheck. > > > > > > > > drivers/gpio/gpiolib.c:2551:17: warning: Local variable 'ret' shadows outer variable [shadowVariable] > > > > > > > > Fixes: d377f56f34f5 ("gpio: gpiolib: Normalize return code variable name") > > > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> > > > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > > > Ccing Linus & Bartosz > > > > For some reason this is not on patchwork and I also don't have it in > my inbox, can you resend, please? https://lore.kernel.org/linux-gpio/20210105105914.13172-1-wsa+renesas@xxxxxxxxxxxxxxxxxxxx/ 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