Re: [PATCH] LED: OMAP PWM: Use work queue for brightness

[Tony Lindgren <tony@atomide.com>]

* Felipe Balbi <felipebalbi@users.sourceforge.net> [071030 03:01]:
> Hi,
> 
> On 10/30/07, Daniel Stone <daniel.stone@nokia.com> wrote:
> > It isn't safe to schedule from a LED brightness_set handler, so use a work queue.
> >
> > Signed-off-by: Daniel Stone <daniel.stone@nokia.com>
> > ---
> >  drivers/leds/leds-omap-pwm.c |   22 ++++++++++++++++++++--
> >  1 files changed, 20 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/leds/leds-omap-pwm.c b/drivers/leds/leds-omap-pwm.c
> > index 6b195d6..c5d7ff0 100644
> > --- a/drivers/leds/leds-omap-pwm.c
> > +++ b/drivers/leds/leds-omap-pwm.c

<snip>


> > @@ -134,9 +142,17 @@ static void omap_pwm_led_set(struct led_classdev *led_cdev,
> >  {
> >         struct omap_pwm_led *led = cdev_to_omap_pwm_led(led_cdev);
> >
> > -       if (value != LED_OFF) {
> > +       led->brightness = value;
> > +       schedule_work(&led->work);
> > +}
> > +
> > +static void omap_pwm_led_work(struct work_struct *work)
> > +{
> > +       struct omap_pwm_led *led = work_to_omap_pwm_led(work);
> > +
> > +       if (led->brightness != LED_OFF) {
> >                 omap_pwm_led_power_on(led);
> > -               omap_pwm_led_set_pwm_cycle(led, value);
> > +               omap_pwm_led_set_pwm_cycle(led, led->brightness);
> >         } else {
> >                 omap_pwm_led_power_off(led);
> >         }
> > @@ -228,6 +244,8 @@ static int omap_pwm_led_probe(struct platform_device *pdev)
> >         led->cdev.default_trigger = NULL;
> >         led->cdev.name = pdata->name;
> >         led->pdata = pdata;
> > +       led->brightness = 0;
> 
> led->brightness = LED_OFF;
> would look better.
> 
> besides the patch looks ok to me.

Pushing with Felipe's suggestion for LED_OFF.

Tony