]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
scsi: a100u2w: Replace mdelay() with msleep()
authorJia-Ju Bai <baijiaju1990@gmail.com>
Fri, 27 Jul 2018 09:09:13 +0000 (17:09 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 31 Jul 2018 03:17:53 +0000 (23:17 -0400)
wait_chip_ready() and wait_firmware_ready() are never called in atomic
context.  They call mdelay() to busy 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: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/a100u2w.c

index b2942ec3d45541199b37359030c947a76b51d7f9..23b17621b6d21158ca518f6ee3fbf447d47c5cd3 100644 (file)
@@ -143,7 +143,7 @@ static u8 wait_chip_ready(struct orc_host * host)
        for (i = 0; i < 10; i++) {      /* Wait 1 second for report timeout     */
                if (inb(host->base + ORC_HCTRL) & HOSTSTOP)     /* Wait HOSTSTOP set */
                        return 1;
-               mdelay(100);
+               msleep(100);
        }
        return 0;
 }
@@ -155,7 +155,7 @@ static u8 wait_firmware_ready(struct orc_host * host)
        for (i = 0; i < 10; i++) {      /* Wait 1 second for report timeout     */
                if (inb(host->base + ORC_HSTUS) & RREADY)               /* Wait READY set */
                        return 1;
-               mdelay(100);    /* wait 100ms before try again  */
+               msleep(100);    /* wait 100ms before try again  */
        }
        return 0;
 }