From: Eva Rachel Retuya Date: Sat, 24 Oct 2015 17:24:51 +0000 (+0800) Subject: Staging: mt29f_spinand: Replace udelay function with usleep_range X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=c75e057744fb16d00668674cf41d63b61ee3cd2d;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git Staging: mt29f_spinand: Replace udelay function with usleep_range Use 'usleep_range' instead of 'udelay' to elapse time. For spinand_reset, define the upper limit by a factor of 2 to keep the wait short while still allowing a "good enough" range for wakeup. Define the range 250us - 1ms for spinand_cmdfunc to provide enough leeway before issuing spinand_reset. Checkpatch found this issue. CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt Signed-off-by: Eva Rachel Retuya Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/mt29f_spinand/mt29f_spinand.c b/drivers/staging/mt29f_spinand/mt29f_spinand.c index a8292a1a8e41..b6a1ffca0a3f 100644 --- a/drivers/staging/mt29f_spinand/mt29f_spinand.c +++ b/drivers/staging/mt29f_spinand/mt29f_spinand.c @@ -736,7 +736,7 @@ static void spinand_reset(struct spi_device *spi_nand) pr_info("spinand reset failed!\n"); /* elapse 1ms before issuing any other command */ - udelay(1000); + usleep_range(1000, 2000); if (wait_till_ready(spi_nand)) dev_err(&spi_nand->dev, "wait timedout!\n"); @@ -802,7 +802,7 @@ static void spinand_cmdfunc(struct mtd_info *mtd, unsigned int command, if (wait_till_ready(info->spi)) dev_err(&info->spi->dev, "WAIT timedout!!!\n"); /* a minimum of 250us must elapse before issuing RESET cmd*/ - udelay(250); + usleep_range(250, 1000); spinand_reset(info->spi); break; default: