Re: [PATCH] [RFC PATCH] lib/string: fix coding style issues

[Greg KH <gregkh@linuxfoundation.org>]

On Sat, Feb 14, 2026 at 07:21:07PM +0530, Samyak wrote:
> - Add spaces around binary operators
> - remove spaces after casts
> - Add blank line after variable declarations
> - Fix constant on left side of a conditional expression (0 < count ->
>   count > 0)
> - Remove un-needed braces around single statements
> 
> Signed-off-by: Samyak Bambole <samyak.bambole07@gmail.com>
> ---
> This is my first linux kernel patch. I have compiled and tested the
> changes on QEMU. I would appreciate any feedback. Thanks.
> 
>  lib/string.c | 27 ++++++++++++++++-----------

If you want to get involved in kernel work, please do coding style
changes in drivers/staging/ which is there to learn on how to do this.
Only after doing this a lot, and getting experience, should you venture
out of drivers/staging/

Also, many subsystems do not want coding style fixes for obvious reasons
(i.e. they change over time and keeping the whole codebase up to date
like this would be constant churn.)

Hint, this patch would need to be broken up anyway, you should only do
"one logical thing" per patch.  And "fix all coding style issues" is not
one thing :)

thanks,

greg k-h