Re: [PATCH 1/2] staging: rtl8723bs: fix line length warning

From: Greg KH

Date: Tue Jul 07 2026 - 07:05:30 EST


On Mon, Jun 01, 2026 at 05:37:56PM +0800, Eugene Mavick wrote:
> Split comment exceeding 100 columns to improve readability and clear
> checkpatch warning
>
> Signed-off-by: Eugene Mavick <mavick4022@xxxxxxxxx>
> ---
> drivers/staging/rtl8723bs/core/rtw_sta_mgt.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> index a1b7fe843979..02f8564d32a7 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> @@ -222,7 +222,8 @@ struct sta_info *rtw_alloc_stainfo(struct sta_priv *pstapriv, u8 *hwaddr)
>
> /* Commented by Albert 2009/08/13 */
> /* For the SMC router, the sequence number of first packet of WPS handshake will be 0. */
> - /* In this case, this packet will be dropped by recv_decache function if we use the 0x00 as the default value for tid_rxseq variable. */
> + /* This packet will be dropped by recv_decache function */
> + /* if we use the 0x00 as the default value for tid_rxseq variable. */
> /* So, we initialize the tid_rxseq variable as the 0xffff. */

Why not just turn this into a proper multi-line comment block?

thanks,

greg k-h