[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 --- 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/usb.c @@ -485,9 +485,9

[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 --- 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 6c10c8

[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 --- drivers/usb/storage/transport.c | 2 +- 1 file chang

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 rep

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

[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 --- drivers/usb/storage/transport.c | 2 +- 1 file chang

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

This patch fixes checkpatch.pl warning c99 style commenting. Signed-off-by: Sunny Kumar --- 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 index aacedef..54d0a59 100644

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 > > --- > > drivers/usb/storage/sddr55.c | 16 +++

[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 --- dr