[PATCH v2 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

According to Documentation/timers/timers-howto.txt udelay() is only called once from a place where sleeping is allowed. We can replace it with a call to usleep_range() with a reasonable upper limit. This is new version of patch created after discussion. Signed-off-by: Sunny Kumar sunny.kumar

[PATCH 1/1] usb: storage : Remove c99 style commenting

This patch fixes checkpatch.pl warning c99 style commenting. Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/sddr55.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/usb/storage/sddr55.c b/drivers/usb/storage/sddr55.c

Re: [PATCH 1/1] usb: storage : Remove c99 style commenting

On Fri, Jun 26, 2015 at 10:08:42AM -0400, Alan Stern wrote: On Fri, 26 Jun 2015, Sunny Kumar wrote: This patch fixes checkpatch.pl warning c99 style commenting. Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/sddr55.c | 16 1 file

[PATCH 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

According to Documentation/timers/timers-howto.txt udelay() is only called once from a place where sleeping is allowed. We can replace it with a call to usleep_range() with a reasonable upper limit. Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/transport.c | 2

[PATCH 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

According to Documentation/timers/timers-howto.txt udelay() is only called once from a place where sleeping is allowed. We can replace it with a call to usleep_range() with a reasonable upper limit. Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/transport.c | 2

Re: [PATCH 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

Ok so what about putting lower limit to 125 and increase upper limit 200 Sunny On Wed, Jun 24, 2015 at 12:24:29PM +0530, Sunny Kumar wrote: According to Documentation/timers/timers-howto.txt udelay() is only called once from a place where sleeping is allowed. We can replace it with a call

Re: [PATCH 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

Ok then, This might be the final lower and upper limit value. Sunny Kumar On Wed, Jun 24, 2015 at 12:24:29PM +0530, Sunny Kumar wrote: According to Documentation/timers/timers-howto.txt udelay() is only called once from a place where sleeping is allowed. We can replace it with a call

[PATCH 1/1] usb: Fix warning simple_strtoul is obsolete, use kstrtoul instead

This patch fixes simple_strtoul is obsolete, use kstrtoul instead warning in usb.c. Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/usb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/storage/usb.c b/drivers/usb/storage/usb.c

[PATCH 1/1] usb: Fix warning simple_strtoul is obsolete, use kstrtoul instead This patch fixes simple_strtoul is obsolete, use kstrtoul instead warning in usb.c.

Signed-off-by: Sunny Kumar sunny.kumar@gmail.com --- drivers/usb/storage/usb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/storage/usb.c b/drivers/usb/storage/usb.c index 6c10c88..a0c0f66 100644 --- a/drivers/usb/storage/usb.c +++ b/drivers/usb/storage