[PATCH] staging: rtl8723bs: Wrap long comments to fit line length

Mariyam Shahid posted 1 patch 1 month, 1 week ago
drivers/staging/rtl8723bs/core/rtw_mlme.c | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
[PATCH] staging: rtl8723bs: Wrap long comments to fit line length
Posted by Mariyam Shahid 1 month, 1 week ago
Wrap comments exceeding 100 columns across multiple lines
to fix checkpatch warnings.

Signed-off-by: Mariyam Shahid <mariyam.shahid135@gmail.com>
---
 drivers/staging/rtl8723bs/core/rtw_mlme.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
index b090d162b37f..4dfa98239f9d 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
@@ -1022,7 +1022,7 @@ static struct sta_info *rtw_joinbss_update_stainfo(struct adapter *padapter, str
 		}
 
 		/* for A-MPDU Rx reordering buffer control for bmc_sta & sta_info */
-		/* if A-MPDU Rx is enabled, resetting rx_ordering_ctrl wstart_b(indicate_seq) 
+		/* if A-MPDU Rx is enabled, resetting rx_ordering_ctrl wstart_b(indicate_seq)
 		 * to default value = 0xffff
 		 */
 		/* todo: check if AP can send A-MPDU packets */
@@ -1041,7 +1041,7 @@ static struct sta_info *rtw_joinbss_update_stainfo(struct adapter *padapter, str
 				preorder_ctrl->enable = false;
 				preorder_ctrl->indicate_seq = 0xffff;
 				preorder_ctrl->wend_b = 0xffff;
-				/* max_ampdu_sz;ex. 32(kbytes) -> wsize_b =32 */				
+				/* max_ampdu_sz;ex. 32(kbytes) -> wsize_b =32 */
 				preorder_ctrl->wsize_b = 64;
 			}
 		}
@@ -1070,7 +1070,8 @@ static void rtw_joinbss_update_network(struct adapter *padapter, struct wlan_net
 	padapter->recvpriv.signal_strength = ptarget_wlan->network.phy_info.signal_strength;
 	padapter->recvpriv.signal_qual = ptarget_wlan->network.phy_info.signal_quality;
 	/* the ptarget_wlan->network.rssi is raw data,
-	 * we use ptarget_wlan->network.phy_info.signal_strength instead (has scaled) */
+	 * we use ptarget_wlan->network.phy_info.signal_strength instead (has scaled)
+	 */
 	padapter->recvpriv.rssi = translate_percentage_to_dbm(ptarget_wlan->network.phy_info.signal_strength);
 
 	rtw_set_signal_stat_timer(&padapter->recvpriv);
@@ -1243,7 +1244,7 @@ void rtw_joinbss_event_prehandle(struct adapter *adapter, u8 *pbuf)
 
 			/* s3. find ptarget_sta & update ptarget_sta after update cur_network
 			 * only for station mode
-			 * */
+			 */
 			if (check_fwstate(pmlmepriv, WIFI_STATION_STATE)) {
 				ptarget_sta = rtw_joinbss_update_stainfo(adapter, pnetwork);
 				if (!ptarget_sta) {
@@ -1375,7 +1376,7 @@ void rtw_stassoc_event_callback(struct adapter *adapter, u8 *pbuf)
 	if (psta) {
 		/* the sta have been in sta_info_queue => do nothing */
 
-		/* between drv has received this event before and 
+		/* between drv has received this event before and
 		 * fw have not yet to set key to CAM_ENTRY)
 		 */
 		return;
@@ -2182,7 +2183,7 @@ void rtw_update_registrypriv_dev_network(struct adapter *adapter)
 	struct wlan_bssid_ex    *pdev_network = &pregistrypriv->dev_network;
 	struct	security_priv *psecuritypriv = &adapter->securitypriv;
 	struct	wlan_network	*cur_network = &adapter->mlmepriv.cur_network;
-	
+
 	/* adhoc no 802.1x  */
 	pdev_network->privacy = (psecuritypriv->dot11PrivacyAlgrthm > 0 ? 1 : 0);
 
-- 
2.43.0
Re: [PATCH] staging: rtl8723bs: Wrap long comments to fit line length
Posted by Dan Carpenter 1 month, 1 week ago
On Thu, Feb 26, 2026 at 03:31:20PM +0500, Mariyam Shahid wrote:
> Wrap comments exceeding 100 columns across multiple lines
> to fix checkpatch warnings.
> 

Not even one of these changes has to do with the 100 character limit.
Please, split them up so you fix one type of warning per patch.

regards,
dan carpenter
Re: [PATCH] staging: rtl8723bs: Wrap long comments to fit line length
Posted by Mariyam Shahid 1 month, 1 week ago
On Thu, 26 Feb 2026, 15:51 Dan Carpenter, <dan.carpenter@linaro.org> wrote:
>
> On Thu, Feb 26, 2026 at 03:31:20PM +0500, Mariyam Shahid wrote:
> > Wrap comments exceeding 100 columns across multiple lines
> > to fix checkpatch warnings.
> >
>
> Not even one of these changes has to do with the 100 character limit.
> Please, split them up so you fix one type of warning per patch.
>
> regards,
> dan carpenter
>

Thank you for the review. I will split the patches and fix the commit
message in v2.
regards,
Mariyam Shahid