[PATCH v2 1/2] staging: rtl8723bs: simplify LPS_RF_ON_check control flow
From: Hungyu Lin
Date: Thu May 21 2026 - 06:57:13 EST
Replace err + break pattern with direct returns to simplify
the control flow and improve readability.
No functional change intended.
Signed-off-by: Hungyu Lin <dennylin0707@xxxxxxxxx>
---
v2:
- Drop unrelated comment formatting changes
drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 18 ++++++------------
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
index ca1cb58fc801..56dbe0f36d65 100644
--- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
+++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
@@ -397,27 +397,21 @@ s32 LPS_RF_ON_check(struct adapter *padapter, u32 delay_ms)
{
unsigned long start_time;
u8 bAwake = false;
- s32 err = 0;
start_time = jiffies;
while (1) {
rtw_hal_get_hwreg(padapter, HW_VAR_FWLPS_RF_ON, &bAwake);
if (bAwake)
- break;
+ return 0;
- if (padapter->bSurpriseRemoved) {
- err = -2;
- break;
- }
+ if (padapter->bSurpriseRemoved)
+ return -2;
+
+ if (jiffies_to_msecs(jiffies - start_time) > delay_ms)
+ return -1;
- if (jiffies_to_msecs(jiffies - start_time) > delay_ms) {
- err = -1;
- break;
- }
msleep(1);
}
-
- return err;
}
/* Description: Enter the leisure power save mode. */
--
2.34.1