Re: [PATCH] zorro: fix checkpatch error by avoiding assignment in if-statement.
From: Geert Uytterhoeven
Date: Tue Jul 29 2025 - 11:50:27 EST
Hi Jogi,
On Tue, 29 Jul 2025 at 14:10, Jogi Dishank <jogidishank503@xxxxxxxxx> wrote:
>
> Hi,
>
> Thank you for the suggestion.
>
> Additionally, the conditional logic is inverted to reduce indentation
> and improve readability, aligning with kernel coding style.
Thanks, but this is not how you send a v2.
Please read Documentation/process/submitting-patches.rst.
> Suggested-by: Christophe Leroy <christophe.leroy@xxxxxxxxxx>
Unless I am mistaken, Christophe merely pointed out a bug in
your first submission, he did not suggest this patch.
> Signed-off-by: Dishank Jogi <jogidishank503@xxxxxxxxx>
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