[OpenWrt-Devel] [PATCH] base-files: Use the same LED timing as before for preinit_regular.

Rafał Miłecki zajec5 at gmail.com
Fri Dec 26 03:53:12 EST 2014


On 26 December 2014 at 09:51, Rafał Miłecki <zajec5 at gmail.com> wrote:
> On 25 December 2014 at 02:28, Vittorio G (VittGam) <openwrt at vittgam.net> wrote:
>> Signed-off-by: Vittorio Gambaletta <openwrt at vittgam.net>
>>
>> diff --git a/package/base-files/files/lib/functions/leds.sh b/package/base-files/files/lib/functions/leds.sh
>> index 4ac8145..d4d4512 100644
>> --- a/package/base-files/files/lib/functions/leds.sh
>> +++ b/package/base-files/files/lib/functions/leds.sh
>> @@ -60,7 +60,7 @@ status_led_blink_fast() {
>>  }
>>
>>  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_failsafe() {
>>  }
>>
>>  status_led_blink_preinit_regular() {
>> -       led_timer $status_led 500 500
>> +       led_timer $status_led 200 200
>>  }
>
> 1) Why?
> 2) Explain "as before"
> 3) I don't think there is enough difference between 100ms and 200ms

4) You don't mention  status_led_blink_preinit(), but your patch also touches it

-- 
Rafał
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list