hwmon: (sch56xx-common) Replace msleep() with usleep_range()
authorArmin Wolf <W_Armin@gmx.de>
Mon, 31 Jan 2022 21:19:34 +0000 (22:19 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Mon, 28 Feb 2022 01:03:17 +0000 (17:03 -0800)
msleep(1) will often sleep more than 20ms, slowing down sensor
and watchdog reads/writes. Use usleep_range() as recommended
in timers-howto.rst to fix that.

Tested on a Fujitsu Esprimo P720.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
Link: https://lore.kernel.org/r/20220131211935.3656-4-W_Armin@gmx.de
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/sch56xx-common.c

index 0172aa1..82602b7 100644 (file)
@@ -139,7 +139,7 @@ static int sch56xx_send_cmd(u16 addr, u8 cmd, u16 reg, u8 v)
        /* EM Interface Polling "Algorithm" */
        for (i = 0; i < max_busy_polls + max_lazy_polls; i++) {
                if (i >= max_busy_polls)
-                       msleep(1);
+                       usleep_range(1000, 2000);
                /* Read Interrupt source Register */
                val = inb(addr + 8);
                /* Write Clear the interrupt source bits */