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

[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]

On 13:20 Sat 05 Sep     , Albin Tonnerre wrote:
> On Sat, 05 Sep 2009 01:47 +0200, Jean-Christophe PLAGNIOL-VILLARD wrote :
> > On 10:36 Thu 20 Aug     , Albin Tonnerre wrote:
> > > On Thu, Aug 20, 2009 at 02:00:45AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote :
> > > > On 10:49 Tue 18 Aug     , Albin Tonnerre wrote:
> > > > > 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've as I'm the one who ask Daniel Gorsulowski to base his new code (this patch)
> > > > against Ulf precedent patch and as other people have done the same comment as I will do
> > > > no need to repeat it
> > > 
> > > Ok, so I'm really confused now. This patch does exactly what you're arguing
> > > against in the rest of your mail, and you still don't provide a pointer to Ulf's
> > > patch.
> > > Would you mind *explaining* to me what your plan is? I just can't get it.
> > something like this for assembly
> > 
> > .macro set_led \num, \state
> > 	call (c or assembly) set_led_state num state
> > .endm
> > 
> > .macore set_led_red \state
> > 	call (c or assembly) set_led_state CONFIG_SYS_LED_RED state
> > .endm
> > 
> > and for c
> > 
> > void set_led(int num, int state)
> > {
> > 	....
> > }
> > 
> > void set_reg_led(int state)
> > {
> > 	set_led(CONFIG_SYS_LED_RED, state);
> > }
> > 
> > etc... for all colour
> > 
> 
> Ok. So what about implementing the current, existing LED API ?
> you'd get:
> 
> void status_led_set  (int led, int state);
> that sound a lot like your set_led function. And then
> 
> status_led_set(STATUS_LED_{YELLOW,BLUE,RED}, state)
> 
> which is basically the same as your set_reg_led?
so you use this one instead

Best Regards,
J.