Re: [patch, rfc] LEDs support for collie

["John Lenz" <lenz@cs.wisc.edu>]

On Wed, November 2, 2005 7:44 am, Russell King said:
> On Wed, Nov 02, 2005 at 02:21:45PM +0100, Pavel Machek wrote:
>> This adds support for controlling LEDs on sharp zaurus sl-5500. It may
>> look a little bit complex, but it probably needs to be complex --
>> blinking is pretty much mandatory when you only have two leds, and we
>> want to support charging led (controlled by kernel).
>
> Isn't "blinking" a kind of policy, as is brightness (== duty cycle of
> a high speed toggling)?  What if someone wants synchronised toggling?
>
> I still think anything over a very simple interface being exported to
> userspace is completely overkill and completely bloated.  Hell, I got
> laughed at for creating an abstracted LEDs interface in the first
> place because many thought the current version was far too bloated.

I can strip out the timer stuff.... without the frequency stuff the patch
is  an interface that allows userspace and other kernel code to adjust the
color and brightness of leds.

Then Richard can implement his triggers code, which uses the exported
in-kernel api for controlling leds.  That triggers code can do whatever,
including blink the led on a frequency.  (And the discussion of the
triggers stuff can continue unrelated to this basic led framework).

Now that I think about it more, the frequency stuff should really be
ripped out of the led code and put somewhere else (like either userspace
or some other triggers code).

>
> I _know_ people have issues with the current interface, whinging that
> "it only exports the colour" but that's something which is actually
> very trivially solvable and therefore _not_ a major problem to solve.
>

I believe it is already solved this using my patch ignoring the frequency....

John


-
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/