[PATCH v7 7/7] staging: rtl8723bs: align and split variable declarations

Khushal Chitturi posted 7 patches 1 month, 2 weeks ago
[PATCH v7 7/7] staging: rtl8723bs: align and split variable declarations
Posted by Khushal Chitturi 1 month, 2 weeks ago
Split multi variable declarations into single lines and move
trailing comments to the line above to fix line length and
alignment warnings.

Signed-off-by: Khushal Chitturi <khushalchitturi@gmail.com>
---
Changelog:
v6 -> v7: New patch to address warnings from earlier refactoring.

 drivers/staging/rtl8723bs/core/rtw_cmd.c     |  8 ++++++--
 drivers/staging/rtl8723bs/include/rtw_mlme.h | 13 ++++++++-----
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl8723bs/core/rtw_cmd.c
index f322a7f89b42..ab669032be40 100644
--- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
+++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
@@ -1131,8 +1131,12 @@ bool traffic_status_watchdog(struct adapter *padapter, bool from_timer)
 	u16 busy_threshold_high = 25;
 	u16 busy_threshold_low = 10;
 	u16 busy_threshold = busy_threshold_high;
-	bool busy_traffic = false, tx_busy_traffic = false, rx_busy_traffic = false;
-	bool higher_busy_traffic = false, higher_busy_rx_traffic = false, higher_busy_tx_traffic = false;
+	bool busy_traffic = false;
+	bool tx_busy_traffic = false;
+	bool rx_busy_traffic = false;
+	bool higher_busy_traffic = false;
+	bool higher_busy_rx_traffic = false;
+	bool higher_busy_tx_traffic = false;
 	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
 
 	collect_traffic_statistics(padapter);
diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h b/drivers/staging/rtl8723bs/include/rtw_mlme.h
index 65a9e4e9df55..cdb2d1dcb728 100644
--- a/drivers/staging/rtl8723bs/include/rtw_mlme.h
+++ b/drivers/staging/rtl8723bs/include/rtw_mlme.h
@@ -99,11 +99,14 @@ struct rt_link_detect_t {
 	bool			busy_traffic;
 	bool			tx_busy_traffic;
 	bool			rx_busy_traffic;
-	bool			higher_busy_traffic; /*  For interrupt migration purpose. */
-	bool			higher_busy_rx_traffic; /*  We may disable Tx interrupt according as Rx traffic. */
-	bool			higher_busy_tx_traffic; /*  We may disable Tx interrupt according as Tx traffic. */
-	u8 traffic_transition_count;
-	u32 low_power_transition_count;
+	/* For interrupt migration purpose. */
+	bool			higher_busy_traffic;
+	/* We may disable Tx interrupt according as Rx traffic. */
+	bool			higher_busy_rx_traffic;
+	/* We may disable Tx interrupt according as Tx traffic. */
+	bool			higher_busy_tx_traffic;
+	u8			traffic_transition_count;
+	u32			low_power_transition_count;
 };
 
 /* used for mlme_priv.roam_flags */
-- 
2.52.0