[U-Boot] [PATCH] AT91: Add support for blue_LED_* and add coloured_LED_init to at91/led.c

Albin Tonnerre albin.tonnerre at free-electrons.com
Tue Aug 18 10:49:58 CEST 2009


On Tue, Aug 18, 2009 at 12:51:48AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote :
> no please take a look on the other LED thread

Would you please provide a pointer to this thread ? THe only one remotely
related I can find is
http://lists.denx.de/pipermail/u-boot/2009-May/052160.html, and you did not
participate in this one ...

> I want to hape coloured-LED api and numbered led handle by the same api
> with the less size impact

When you have this same API, you'll still need code to turn LEDs on and off, and
you'll still have to beat the LED code out of cards-specific code because it
really ought to be in a common file. My patch does exactly that, does *not*
impact size, and is an actual step towards whatever you plan next. I'd prefer if
you were rejecting this on actual, justified grounds.

Regards,
-- 
Albin Tonnerre, Free Electrons
Kernel, drivers and embedded Linux development,
consulting, training and support.
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: Digital signature
Url : http://lists.denx.de/pipermail/u-boot/attachments/20090818/341f80a9/attachment.pgp 


More information about the U-Boot mailing list