Re: [patch, rfc] LEDs support for collie

[Russell King]

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 _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.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 Serial core
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/