Author: blogic
Date: 2015-01-20 07:54:53 +0100 (Tue, 20 Jan 2015)
New Revision: 44056

Modified:
   trunk/package/base-files/files/lib/functions/leds.sh
Log:
base-files: Use 200ms LED timing as before for the new phase preinit_regular.

This restores normal pre-r43715 200ms blink-period for the System LED we're all 
accustomed to see while our OpenWrt routers are booting.

Failsafe possibility will now be signalled with a new 100ms blinking, which is 
easily recognizable from the normal 200ms booting.

So no existing user will be scared by a new 500ms LED pattern, since such a 
slow pattern could easily be mistaken for something wrong... I was like "ok why 
my router is collapsing now, is this a bad flash, a kernel panic, or what else" 
when I've seen it for the first time ;)

Sorry for not having explained myself better in v1 of this patch.

Original:
Preinit, failsafe is possible: 200ms
Preinit, failsafe not possible anymore, booting normally: 200ms
Failsafe entered: 50ms

Now (after preinit_regular has been introduced):
Preinit, failsafe is possible: 200ms
Preinit, failsafe not possible anymore, booting normally: 500ms *here is the 
"offending" change*
Failsafe entered: 50ms

With my proposed patch:
Preinit, failsafe is possible: 100ms *indicate this condition with a new 
timing, that prompts the user to press the key if they want to start failsafe*
Preinit, failsafe not possible anymore, booting normally: 200ms *keep this as 
before*
Failsafe entered: 50ms

Signed-off-by: Vittorio Gambaletta <[email protected]>

Modified: trunk/package/base-files/files/lib/functions/leds.sh
===================================================================
--- trunk/package/base-files/files/lib/functions/leds.sh        2015-01-20 
06:34:28 UTC (rev 44055)
+++ trunk/package/base-files/files/lib/functions/leds.sh        2015-01-20 
06:54:53 UTC (rev 44056)
@@ -60,7 +60,7 @@
 }
 
 status_led_blink_preinit() {
-       led_timer $status_led 200 200
+       led_timer $status_led 100 100
 }
 
 status_led_blink_failsafe() {
@@ -68,5 +68,5 @@
 }
 
 status_led_blink_preinit_regular() {
-       led_timer $status_led 500 500
+       led_timer $status_led 200 200
 }
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to