staging: rtl8723bs: fix line length warnings in os_intfs.c
PATCH v2 1/1 staging: rtl8723bs: fix line length warnings in os_intfs.c
Hi Greg,
This is a v2 patch addressing your feedback on comment block placement and providing the missing commit description text.
Thanks,
Bramwel
---
From 8fb05fdee4c523bcafc5b425851b95e69e22182a Mon Sep 17 00:00:00 2001
From: Bramwel Barack <[email protected]>
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.
--- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
drivers/staging/rtl8723bs/os_dep/os_intfs.c +6 -1
@@ -71,7 +71,12 @@ int rtw_ht_enable = 1;* 0x01 means enable 2.4G 40MHz */ static int rtw_bw_mode =0x01 ; -static int rtw_ampdu_enable =1 ;/* for enable tx_ampdu ,0: disable, 0x1:enable (but wifi_spec should be 0), 0x2: force enable (don't care wifi_spec) */ +/* for enable tx_ampdu: +* 0: disable +* 0x1:enable (but wifi_spec should be 0) +* 0x2: force enable (don't care wifi_spec) +*/ +static int rtw_ampdu_enable =1 ;static int rtw_rx_stbc =1 ;/* 0: disable, 1:enable 2.4g */ static int rtw_ampdu_amsdu;/* 0: disabled, 1:enabled, 2:auto . There is an IOT issu with DLINK DIR-629 when the flag turn on */ /* Short GI support Bit Map */
Signature · 1 line
2.55.0
Greg KH
maintainerQuoted 4 lines · Bramwel Barack
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.
Quoted 4 lines
Thanks,
Bramwel
Quoted 8 lines
---
Quoted 1 line
From
8fb05fdee4c523bcafc5b425851b95e69e22182aMon Sep 17 00:00:00 2001From: Bramwel Barack <[email protected]>
Date: Wed, 29 Jul 2026 09:33:57 +0300
Subject: [PATCH v2] staging: rtl8723bs: fix line length warnings inos_intfs.cFix 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
Quoted 5 lines · Bramwel Barack
+/* for enable
tx_ampdu:
+ * 0: disable
+ * 0x1:enable (butwifi_specshould be 0)
+ *0x2: force enable (don't carewifi_spec)
+ */
Does this really look beautiful to you?
There are a bunch for rules for v2 patches... staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/
regards,
dan carpenter