[PATCH v2 3/3] staging: rtl8723bs: Modify struct sta_info attribute qos_option and ieee8021x_blocked

Erick Karanja
2025-04-02T17:17:14Z

Standardize boolean representation by ensuring consistency,
replacing instances of 1/0 with true/false where boolean
logic is implied, as some definitions already use true/false.
This improves code clarity and aligns with the kernel’s bool type usage.

Signed-off-by: Erick Karanja <karanja99erick@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_ap.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8723bs/core/rtw_ap.c
index d46a04b9a05e..199727f04516 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -386,10 +386,10 @@ void update_bmc_sta(struct adapter *padapter)

pmlmeinfo->FW_sta_info[psta->mac_id].psta = psta;

- psta->qos_option = 0;
+ psta->qos_option = false;
psta->htpriv.ht_option = false;

- psta->ieee8021x_blocked = 0;
+ psta->ieee8021x_blocked = false;

memset((void *)&psta->sta_stats, 0, sizeof(struct stainfo_stats));

@@ -1967,17 +1967,17 @@ void sta_info_update(struct adapter *padapter, struct sta_info *psta)

/* update wmm cap. */
if (WLAN_STA_WME & flags)
- psta->qos_option = 1;
+ psta->qos_option = true;
else
- psta->qos_option = 0;
+ psta->qos_option = false;

if (pmlmepriv->qospriv.qos_option == 0)
- psta->qos_option = 0;
+ psta->qos_option = false;

/* update 802.11n ht cap. */
if (WLAN_STA_HT & flags) {
psta->htpriv.ht_option = true;
- psta->qos_option = 1;
+ psta->qos_option = true;
} else {
psta->htpriv.ht_option = false;
}
--
2.43.0