lantiq: cleanup led handling functions
authorMathias Kresin <dev@kresin.me>
Tue, 4 Oct 2016 18:30:34 +0000 (20:30 +0200)
committerMathias Kresin <dev@kresin.me>
Wed, 5 Oct 2016 17:23:23 +0000 (19:23 +0200)
Use a more generic name for the lantiq_get_dt_led_chosen function.

Drop the lantiq_is_dt_led_chosen function. The lantiq_get_dt_led
function can be used to achieve the same.

Signed-off-by: Mathias Kresin <dev@kresin.me>
target/linux/lantiq/base-files/etc/board.d/01_leds
target/linux/lantiq/base-files/etc/diag.sh
target/linux/lantiq/base-files/lib/functions/lantiq.sh

index 4c3dab7..d215393 100755 (executable)
@@ -9,19 +9,26 @@
 
 board_config_update
 
-[ -n "$(lantiq_is_dt_led_chosen wifi)" ] && ucidef_set_led_wlan "wifi" "wifi" "$(lantiq_get_dt_led_chosen wifi)" "phy0tpt"
-[ -n "$(lantiq_is_dt_led_chosen usb)" ] && ucidef_set_led_usbdev "usb" "usb" "$(lantiq_get_dt_led_chosen usb)" "1-1"
-[ -n "$(lantiq_is_dt_led_chosen usb2)" ] && ucidef_set_led_usbdev "usb2" "usb2" "$(lantiq_get_dt_led_chosen usb2)" "2-1"
+led_wifi="$(lantiq_get_dt_led wifi)"
+[ -n "$led_wifi" ] && ucidef_set_led_wlan "wifi" "wifi" "$led_wifi" "phy0tpt"
 
-[ -n "$(lantiq_is_dt_led_chosen internet)" ] && {
+led_usb="$(lantiq_get_dt_led usb)"
+[ -n "$led_usb" ] && ucidef_set_led_usbdev "usb" "usb" "$led_usb" "1-1"
+
+led_usb2="$(lantiq_get_dt_led usb2)"
+[ -n "$led_usb2" ] && ucidef_set_led_usbdev "usb2" "usb2" "$led_usb2" "2-1"
+
+led_internet="$(lantiq_get_dt_led internet)"
+[ -n "$led_internet" ] && {
        interface="nas0"
 
-       [ -n "$(lantiq_is_dt_led_chosen dsl)" ] && {
+       led_dsl="$(lantiq_get_dt_led dsl)"
+       [ -n "$led_dsl" ] && {
                interface="pppoe-wan"
-               ucidef_set_led_default "dsl" "dsl" "$(lantiq_get_dt_led_chosen dsl)" "0"
+               ucidef_set_led_default "dsl" "dsl" "$led_dsl" "0"
        }
 
-       ucidef_set_led_netdev "internet" "internet" "$(lantiq_get_dt_led_chosen internet)" "$interface"
+       ucidef_set_led_netdev "internet" "internet" "$led_internet" "$interface"
 }
 
 board=$(lantiq_board_name)
index f6363b3..70ed25c 100644 (file)
@@ -4,9 +4,9 @@
 . /lib/functions/leds.sh
 . /lib/functions/lantiq.sh
 
-boot="$(lantiq_get_dt_led_chosen boot)"
-failsafe="$(lantiq_get_dt_led_chosen failsafe)"
-running="$(lantiq_get_dt_led_chosen running)"
+boot="$(lantiq_get_dt_led boot)"
+failsafe="$(lantiq_get_dt_led failsafe)"
+running="$(lantiq_get_dt_led running)"
 
 set_state() {
     status_led="$boot"
index 642ae6b..456e54b 100644 (file)
@@ -28,18 +28,14 @@ lantiq_board_name() {
        echo "$name"
 }
 
-lantiq_is_dt_led_chosen() {
-       [ -f "/sys/firmware/devicetree/base/chosen/leds/$1" ] && echo "true"
-}
-
-lantiq_get_dt_led_chosen() {
+lantiq_get_dt_led() {
        local label
-       local nodepath
+       local ledpath
        local basepath="/sys/firmware/devicetree/base"
-       local chosenpath="$basepath/chosen/leds/$1"
+       local nodepath="$basepath/chosen/leds/$1"
 
-       [ -f "$chosenpath" ] && nodepath=$(cat "$chosenpath")
-       [ -n "$nodepath" ] && label=$(cat "$basepath$nodepath/label")
+       [ -f "$nodepath" ] && ledpath=$(cat "$nodepath")
+       [ -n "$ledpath" ] && label=$(cat "$basepath$ledpath/label")
 
        echo "$label"
 }