staging: rtl8192u: replace msleep(1) with usleep_range() in r819xU_phy.c
authorXenia Ragiadakou <burzalodowa@gmail.com>
Sun, 23 Jun 2013 03:15:18 +0000 (06:15 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 24 Jun 2013 23:08:33 +0000 (16:08 -0700)
This patch fixes the following checkpatch warning:
WARNING: msleep < 20ms can sleep for up to 20ms

The implementation of msleep() is based on jiffies timeout
subsystem. Hence, its resolution is constrained by the value
of HZ tick rate. When HZ variable is configured to 100,
the jiffies variable is updated every 10 ms, so a timeout
of 1ms cannot be met using jiffies (it will be rounded up
to 10ms).

This patch replaces msleep(1) with usleep_range(1000, 1000).
usleep_range() uses hrtimers subsystem which is not bound
to HZ value and can provide microsecond-level resolution.

Signed-off-by: Xenia Ragiadakou <burzalodowa@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192u/r819xU_phy.c

index edb5df2c25f541b7860aea699cb1feca3acdbbc6..92f0f90150e942f82367bf47232e11a49d44800c 100644 (file)
@@ -185,7 +185,7 @@ u32 rtl8192_phy_RFSerialRead(struct net_device *dev, RF90_RADIO_PATH_E eRFPath,
 
 
        /* TODO: we should not delay such a long time. Ask for help from SD3 */
-       msleep(1);
+       usleep_range(1000, 1000);
 
        ret = rtl8192_QueryBBReg(dev, pPhyReg->rfLSSIReadBack,
                                 bLSSIReadBackData);
@@ -741,11 +741,11 @@ u8 rtl8192_phy_checkBBAndRF(struct net_device *dev, HW90_BLOCK_E CheckBlock,
                                             bMask12Bits, WriteData[i]);
                        /* TODO: we should not delay for such a long time.
                           Ask SD3 */
-                       msleep(1);
+                       usleep_range(1000, 1000);
                        reg = rtl8192_phy_QueryRFReg(dev, eRFPath,
                                                     WriteAddr[HW90_BLOCK_RF],
                                                     bMask12Bits);
-                       msleep(1);
+                       usleep_range(1000, 1000);
                        break;
 
                default: