Re: [PATCH 2/2] leds: Add WM831x status LED driver

[Samuel Ortiz <sameo@linux.intel.com>]

Hi Richard,

On Mon, Sep 07, 2009 at 02:51:24PM +0100, Richard Purdie wrote:
> On Mon, 2009-09-07 at 15:22 +0200, Pavel Machek wrote:
> > On Sat 2009-09-05 14:09:21, Mark Brown wrote:
> > > The WM831x devices feature two software controlled status LEDs with
> > > hardware assisted blinking.
> > > 
> > > The device can also autonomously control the LEDs based on a selection
> > > of sources.  This can be configured at boot time using either platform
> > > data or the chip OTP.  A sysfs file in the style of that for triggers
> > > allowing the control source to be configured at run time.  Triggers
> > > can't be used here since they can't depend on the implementation details
> > > of a specific LED type.
> > 
> > I believe people *were* doing that with triggers in other drivers...?
> 
> We did talk about allowing the LED triggers to choose whether they were
> visible to a given LED. At present the mechanism doesn't exist because
> we've never had a user.
> 
> It would just be a case of adding a new optional function to the trigger
> to say whether it supported a given LED or not and calling that function
> from led_trigger_show().
> 
> I've queued this driver but that would be a nice enhancement.
Unless you're basing your tree on linux-next, you may have issues applying
this patch since the wm831x header is not in Linus tree yet.
If you have troubles with this patch, let me know and I'll queue it.

Cheers,
Samuel.


> Cheers,
> 
> Richard
> 
> -- 
> Richard Purdie
> Intel Open Source Technology Centre
> 

-- 
Intel Open Source Technology Centre
http://oss.intel.com/