[U-Boot] [PATCH 6/7] status_led: Add support for inverted LEDs

Fabio Estevam festevam at gmail.com
Sat Sep 28 18:49:23 CEST 2013


On Sat, Sep 28, 2013 at 11:17 AM, Benoît Thébaudeau
<benoit.thebaudeau at advansee.com> wrote:
> Dear Otavio Salvador,
>
> On Saturday, September 28, 2013 5:24:17 AM, Otavio Salvador wrote:
>> There're cases we want to use active-low LEDs and the 'inverted' logic
>> needs to be added. This includes it using the STATUS_LED_INVERT macro.
>
> There is already a STATUS_LED_ACTIVE definition (though not one per LED) in
> include/status_led.h for some platforms. Wouldn't it be worth keeping the same
> naming here for consistency (i.e. STATUS_LED_ACTIVEn, which would also imply
> exchanging 0/1 values)?

I agree. "INVERT" is confusing, because we don't know what is the normal state.

Doing like Benoît suggests would be clearer: STATUS_LED_ACTIVE0 or
STATUS_LED_ACTIVE1.


More information about the U-Boot mailing list