Skip to contentSkip to conversation

staging: rtl8723bs: use usleep_range() instead of msleep(1)

PATCH New Raushan Kumar 1 participant 1 message Aug 9 Staging

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 staging: rtl8723bs: use usleep_range() instead of msleep(1)

msleep() can sleep for up to 20ms for requested durations under 20ms, due to jiffies timer granularity, as noted in the kernel's own delay.h documentation. Replace msleep(1) with usleep_range() in the polling loops of LPS_RF_ON_check() and LPS_Leave_check() for more accurate short-duration sleeps, as flagged by checkpatch.pl.

---
 drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/staging/rtl8723bs/core/rtw_pwrctrl.c +2 -2
index b9f8cf1014ed..075f98c799c4 100644
@@ -414,7 +414,7 @@ s32 LPS_RF_ON_check(struct adapter *padapter, u32 delay_ms)
 			err = -1;
 			break;
 		}
-		msleep(1);
+		usleep_range(1000, 2000);
 	}
 
 	return err;
@@ -567,7 +567,7 @@ void LPS_Leave_check(struct adapter *padapter)
 		if (jiffies_to_msecs(jiffies - start_time) > 100)
 			break;
 
-		msleep(1);
+		usleep_range(1000, 2000);
 	}
 }
 
Signature · 1 line
2.55.0