Messages in this thread | | | From | Jingoo Han <> | Subject | Re: [PATCH 1/3] backlight: da9052: Use usleep_range() instead of msleep() for small sleeps | Date | Mon, 06 Aug 2012 13:59:38 +0900 |
| |
On Monday, August 06, 2012 1:48 PM Sachin Kamat wrote: > > Hi Jingoo, > > On 6 August 2012 10:13, Jingoo Han <jg1.han@samsung.com> wrote: > > Since msleep might not sleep for the desired amount when less > > than 20ms, use usleep_range. > > > > Cc: Ashish Jangam <ashish.jangam@kpitcummins.com> > > Cc: Richard Purdie <rpurdie@rpsys.net> > > Signed-off-by: Jingoo Han <jg1.han@samsung.com> > > --- > > drivers/video/backlight/da9052_bl.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/video/backlight/da9052_bl.c b/drivers/video/backlight/da9052_bl.c > > index b628d68..10485c9 100644 > > --- a/drivers/video/backlight/da9052_bl.c > > +++ b/drivers/video/backlight/da9052_bl.c > > @@ -72,7 +72,7 @@ static int da9052_adjust_wled_brightness(struct da9052_bl *wleds) > > if (ret < 0) > > return ret; > > > > - msleep(10); > > + usleep_range(10000, 11000); > > Can't we just use usleep(10000) instead?
usleep() is not available. For more details, refer to Documentation/timers/timers-howto.txt.
> > > > > > if (wleds->brightness) { > > ret = da9052_reg_write(wleds->da9052, wled_bank[wleds->led_reg], > > -- > > 1.7.1 > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Please read the FAQ at http://www.tux.org/lkml/ > > > > -- > With warm regards, > Sachin
| |