[PATCH 5/5] leds-gpio: Change blink_set callback to be able to turn off blinking

Benjamin Herrenschmidt benh at kernel.crashing.org
Wed May 26 07:20:08 EDT 2010


On Wed, 2010-05-26 at 12:05 +0100, Richard Purdie wrote:
> On Sat, 2010-05-22 at 20:54 +1000, Benjamin Herrenschmidt wrote:
> > The leds-gpio blink_set() callback follows the same prototype as the
> > main leds subsystem blink_set() one.
> > 
> > The problem is that to stop blink, normally, a leds driver does it
> > in the brightness_set() callback when asked to set a new fixed value.
> > 
> > However, with leds-gpio, the platform has no hook to do so, as this
> > later callback results in a standard GPIO manipulation.
> > 
> > This changes the leds-gpio specific callback to take a new argument
> > that indicates whether the LED should be blinking or not and in what
> > state it should be set if not. We also update the dns323 platform
> > which seems to be the only user of this so far.
> > 
> > Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
> > CC: Richard Purdie <rpurdie at rpsys.net>
> > CC: Grant Likely <grant.likely at secretlab.ca>
> > ---
> >  arch/arm/mach-orion5x/dns323-setup.c |   23 +++++++++++++----------
> >  drivers/leds/leds-gpio.c             |   31 ++++++++++++++++++++++++-------
> >  include/linux/leds.h                 |   12 ++++++++----
> >  3 files changed, 45 insertions(+), 21 deletions(-)
> 
> Queued in the leds tree, thanks.

You may need to fixup the dns323 bit if you apply before my dns323 rev C
patches ... but I suppose you figured that out already ;-) It should be
minor.

Cheers,
Ben.





More information about the linux-arm-kernel mailing list