Input: wdt87xx_i2c - replace mdelay() with msleep() in wdt87xx_resume()
authorJia-Ju Bai <baijiaju1990@gmail.com>
Fri, 27 Jul 2018 18:29:56 +0000 (11:29 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 27 Jul 2018 18:59:05 +0000 (11:59 -0700)
wdt87xx_resume() is never called in atomic context.  It calls mdelay() to
busily wait, which is not necessary.  mdelay() can be replaced with
msleep().

This is found by a static analysis tool named DCNS written by myself.

Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/wdt87xx_i2c.c

index 20f7f3902757908413a05cfec0be559eedd81d99..166edeb77776b20d9875ea1732761cbf08e1fa15 100644 (file)
@@ -1142,7 +1142,7 @@ static int __maybe_unused wdt87xx_resume(struct device *dev)
         * The chip may have been reset while system is resuming,
         * give it some time to settle.
         */
-       mdelay(100);
+       msleep(100);
 
        error = wdt87xx_send_command(client, VND_CMD_START, 0);
        if (error)