2016-10-22 21:46 GMT+02:00 <p.wa...@gmx.at>: > From: P.Wassi <p.wa...@gmx.at> > > Set the default value for status-fault LEDs to '0' > > Signed-off-by: P.Wassi <p.wa...@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@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/lede-dev _______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
