Re: [PATCH v2] staging: rtl8723bs: fix line length warnings in os_intfs.c

[Greg KH <gregkh@linuxfoundation.org>]

On Wed, Jul 29, 2026 at 11:54:30AM +0300, Bramwel Barack wrote:
> Hi Greg,
> 
> This is a v2 patch addressing your feedback on comment block placement
> and providing the missing commit description text.

As per the documentation, the v2 stuff goes below the --- line, right?

Take a look at the mailing list for hundreds of examples to follow.
This style here will not work well.

> 
> Thanks,
> Bramwel
> 
> ---
> 
> >From 8fb05fdee4c523bcafc5b425851b95e69e22182a Mon Sep 17 00:00:00 2001
> From: Bramwel Barack <bramwelbarack89@gmail.com>
> Date: Wed, 29 Jul 2026 09:33:57 +0300
> Subject: [PATCH v2] staging: rtl8723bs: fix line length warnings in os_intfs.c
> 
> Fix coding style warnings where comment lines exceeded the 100-character limit. Move the split multi-line comment block above the target variable declaration to comply with standard kernel comment formatting.

The irony of a line length warning patch having a line-length issue in
the changelog text itself is quite high :)

Always run checkpatch.pl on your changes before sending them out.
Again, look at the tool 'b4', it has an option to run checkpatch on the
patch before it is sent out.

thanks,

greg k-h