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

Richard Purdie rpurdie at rpsys.net
Wed May 26 07:44:10 EDT 2010


On Wed, 2010-05-26 at 21:20 +1000, Benjamin Herrenschmidt wrote:
> 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.

Yes, I noticed just after I hit send and applied. I'd rather get the
core change in sooner than later and the reject doesn't look serious so
I'm assuming you can fix up the missing active_low bit for the new
addition easily enough?

Cheers,

Richard




More information about the linux-arm-kernel mailing list