[LEDE-DEV] [PATCH] kirkwood: disable fault LEDs by default

[Mathias Kresin]

2016-10-22 21:46 GMT+02:00  <p.wassi at gmx.at>:
> From: P.Wassi <p.wassi at gmx.at>
>
> Set the default value for status-fault LEDs to '0'
>
> Signed-off-by: P.Wassi <p.wassi at gmx.at>

Full name please!

> ---
> The kirkwood-dockstar, kirkwood-goflex* and kirkwood-pogoplug devices have
> one duo-color status LED (green+orange / health+fault).
> For the dockstar and pogoplug the orange fault LED is enabled by default.
> (This is not the case for goflex*-devices).
> This patch unifies the behaviour and disabled the fault LED during bootup.

This has to go into the commit message. The commit message should make
clean why this change is required. Have a look at
http://chris.beams.io/posts/git-commit/#why-not-how for a more
detailed (and better) explanation.

>
> linux/kirkwood/base-files/etc/board.d/01_leds |    4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target/linux/kirkwood/base-files/etc/board.d/01_leds b/target/linux/kirkwood/base-files/etc/board.d/01_leds
> --- a/target/linux/kirkwood/base-files/etc/board.d/01_leds
> +++ b/target/linux/kirkwood/base-files/etc/board.d/01_leds
> @@ -13,7 +13,7 @@ board=$(kirkwood_board_name)
>  case "$board" in
>  "dockstar")
>         ucidef_set_led_default "health" "health" "status:green:health" "1"
> -       ucidef_set_led_default "fault" "fault" "status:orange:fault" "1"
> +       ucidef_set_led_default "fault" "fault" "status:orange:fault" "0"

status:orange:fault is on after boot because it is used as status_led.
What do you think about using the status:green:health led instead as
status_led in diag.sh (similar to the power led on other boards)?
Letting a fault indication led blinking (during boot) while everything
is fine, might not be the best idea.

Another benefit would be that both ucidef_set_led_default calls could
be dropped.

>         ;;
>  "linksys-audi")
>         ucidef_set_led_default "power" "power" "audi:green:power" "1"
> @@ -33,7 +33,7 @@ case "$board" in
>         ;;
>  "pogo_e02")
>         ucidef_set_led_default "health" "health" "pogo_e02:green:health" "1"
> -       ucidef_set_led_default "fault" "fault" "pogo_e02:orange:fault" "1"
> +       ucidef_set_led_default "fault" "fault" "pogo_e02:orange:fault" "0"

Same as above.

>         ;;
>  "guruplug-server-plus")
>         ucidef_set_led_timer "health" "health" "guruplug:red:health" "200" "800"
>
> _______________________________________________
> Lede-dev mailing list
> Lede-dev at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/lede-dev