ar71xx: base-files: cleanups in etc/diag.sh

This includes:

- code style fixes
- use of local 'board' variable in LED names, wherever possible

Signed-off-by: Piotr Dymacz <pepe2k@gmail.com>

target/linux/ar71xx/base-files/etc/diag.sh

@@ -5,7 +5,9 @@
 . /lib/ar71xx.sh
 
 get_status_led() {
-	case $(ar71xx_board_name) in
+	local board=$(ar71xx_board_name)
+
+	case $board in
 	alfa-nx)
 		status_led="alfa:green:led_8"
 		;;
@@ -13,16 +15,16 @@ get_status_led() {
 		status_led="eap7660d:green:ds4"
 		;;
 	antminer-s1)
-		status_led="antminer-s1:green:system"
+		status_led="$board:green:system"
 		;;
 	antminer-s3)
-		status_led="antminer-s3:green:system"
+		status_led="$board:green:system"
 		;;
 	antminer-r1)
-		status_led="antminer-r1:green:system"
+		status_led="$board:green:system"
 		;;
 	ap132)
-		status_led="ap132:green:status"
+		status_led="$board:green:status"
 		;;
 	ap136-010|\
 	ap136-020)
@@ -35,21 +37,27 @@ get_status_led() {
 		status_led="ap135:green:status"
 		;;
 	ap81)
-		status_led="ap81:green:status"
+		status_led="$board:green:status"
 		;;
 	ap83)
-		status_led="ap83:green:power"
+		status_led="$board:green:power"
 		;;
 	ap96)
-		status_led="ap96:green:led2"
+		status_led="$board:green:led2"
 		;;
 	aw-nr580)
-		status_led="aw-nr580:green:ready"
+		status_led="$board:green:ready"
 		;;
 	bsb)
-		status_led="bsb:red:sys"
+		status_led="$board:red:sys"
 		;;
-	bullet-m | rocket-m | rocket-m-xw | nano-m | nanostation-m | nanostation-m-xw | loco-m-xw)
+	bullet-m|\
+	rocket-m|\
+	rocket-m-xw|\
+	nano-m|\
+	nanostation-m|\
+	nanostation-m-xw|\
+	loco-m-xw)
 		status_led="ubnt:green:link4"
 		;;
 	rocket-m-ti)
@@ -62,13 +70,13 @@ get_status_led() {
 		status_led="pcs:green:power"
 		;;
 	c-55)
-		status_led="c-55:green:pwr"
+		status_led="$board:green:pwr"
 		;;
 	cap4200ag)
 		status_led="senao:green:pwr"
 		;;
 	cf-e316n-v2)
-		status_led="$(ar71xx_board_name):blue:wan"
+		status_led="$board:blue:wan"
 		;;
 	cpe510)
 		status_led="tp-link:green:link4"
@@ -80,7 +88,7 @@ get_status_led() {
 		status_led="pcs:amber:power"
 		;;
 	db120)
-		status_led="db120:green:status"
+		status_led="$board:green:status"
 		;;
 	dgl-5500-a1|\
 	dhp-1565-a1|\
@@ -111,25 +119,26 @@ get_status_led() {
 		status_led="devolo:status:wlan"
 		;;
 	dr344)
-		status_led="dr344:green:status"
+		status_led="$board:green:status"
 		;;
 	dr531)
-		status_led="dr531:green:sig4"
+		status_led="$board:green:sig4"
 		;;
 	dragino2)
-		status_led="dragino2:red:system"
+		status_led="$board:red:system"
 		;;
 	eap300v2)
 		status_led="engenius:blue:power"
 		;;
 	eap7660d)
-		status_led="eap7660d:green:ds4"
+		status_led="$board:green:ds4"
 		;;
 	el-mini|\
 	el-m150)
 		status_led="easylink:green:system"
 		;;
-	ew-dorin | ew-dorin-router)
+	ew-dorin|\
+	ew-dorin-router)
 		status_led="dorin:green:status"
 		;;
 	f9k1115v2)
@@ -139,10 +148,10 @@ get_status_led() {
 		status_led="gl-connect:green:lan"
 		;;
 	epg5000)
-		status_led="epg5000:amber:power"
+		status_led="$board:amber:power"
 		;;
 	esr1750)
-		status_led="esr1750:amber:power"
+		status_led="$board:amber:power"
 		;;
 	esr900)
 		status_led="engenius:amber:power"
@@ -159,7 +168,7 @@ get_status_led() {
 		status_led="jjplus:green:led1"
 		;;
 	jwap230)
-		status_led="jwap230:green:led1"
+		status_led="$board:green:led1"
 		;;
 	ls-sr71)
 		status_led="ubnt:green:d22"
@@ -168,16 +177,16 @@ get_status_led() {
 		status_led="mercury:green:system"
 		;;
 	mr12)
-		status_led="mr12:green:power"
+		status_led="$board:green:power"
 		;;
 	mr16)
-		status_led="mr16:green:power"
+		status_led="$board:green:power"
 		;;
 	mr18)
-		status_led="mr18:green:tricolor0"
+		status_led="$board:green:tricolor0"
 		;;
 	mr600)
-		status_led="mr600:orange:power"
+		status_led="$board:orange:power"
 		;;
 	mr600v2)
 		status_led="mr600:blue:power"
@@ -205,10 +214,10 @@ get_status_led() {
 		status_led="nbg460n:green:power"
 		;;
 	nbg6616)
-		status_led="nbg6616:green:power"
+		status_led="$board:green:power"
 		;;
 	nbg6716)
-		status_led="nbg6716:white:power"
+		status_led="$board:white:power"
 		;;
 	om2p|\
 	om2pv2|\
@@ -236,14 +245,20 @@ get_status_led() {
 		status_led="onion:amber:system"
 		;;
 	pb44)
-		status_led="pb44:amber:jump1"
+		status_led="$board:amber:jump1"
 		;;
 	rb-2011l|\
 	rb-2011uas|\
 	rb-2011uas-2hnd)
 		status_led="rb:green:usr"
 		;;
-	rb-411 | rb-411u | rb-433 | rb-433u | rb-450 | rb-450g | rb-493)
+	rb-411|\
+	rb-411u|\
+	rb-433|\
+	rb-433u|\
+	rb-450|\
+	rb-450g|\
+	rb-493)
 		status_led="rb4xx:yellow:user"
 		;;
 	rb-750)
@@ -263,20 +278,21 @@ get_status_led() {
 	rb-sxt5n)
 		status_led="rb:green:power"
 		;;
-	routerstation | routerstation-pro)
+	routerstation|\
+	routerstation-pro)
 		status_led="ubnt:green:rf"
 		;;
 	rw2458n)
-		status_led="rw2458n:green:d3"
+		status_led="$board:green:d3"
 		;;
 	smart-300)
 		status_led="nc-link:green:system"
 		;;
 	minibox-v1)
-		status_led="minibox-v1:green:system"
+		status_led="$board:green:system"
 		;;
 	oolite)
-		status_led="oolite:red:system"
+		status_led="$board:red:system"
 		;;
 	qihoo-c301)
 		status_led="qihoo:green:status"
@@ -285,10 +301,10 @@ get_status_led() {
 		status_led="tellstick:white:system"
 		;;
 	som9331)
-		status_led="som9331:green:system"
+		status_led="$board:green:system"
 		;;
 	tew-632brp)
-		status_led="tew-632brp:green:status"
+		status_led="$board:green:status"
 		;;
 	tew-673gru)
 		status_led="trendnet:blue:wps"
@@ -406,33 +422,34 @@ get_status_led() {
 		status_led="netgear:green:power"
 		;;
 	wp543)
-		status_led="wp543:green:diag"
+		status_led="$board:green:diag"
 		;;
 	wpj342)
-		status_led="wpj342:green:sig3"
+		status_led="$board:green:sig3"
 		;;
 	wpj344)
-		status_led="wpj344:green:status"
+		status_led="$board:green:status"
 		;;
 	wpj531)
-		status_led="wpj531:green:sig3"
+		status_led="$board:green:sig3"
 		;;
 	wpj558)
-		status_led="wpj558:green:sig3"
+		status_led="$board:green:sig3"
 		;;
 	wrt400n)
-		status_led="wrt400n:blue:wps"
+		status_led="$board:blue:wps"
 		;;
 	wrt160nl)
-		status_led="wrt160nl:blue:wps"
+		status_led="$board:blue:wps"
 		;;
 	z1)
-		status_led="z1:green:tricolor0"
+		status_led="$board:green:tricolor0"
 		;;
 	zbt-we1526)
-		status_led="zbt-we1526:green:status"
+		status_led="$board:green:status"
 		;;
-	zcn-1523h-2 | zcn-1523h-5)
+	zcn-1523h-2|\
+	zcn-1523h-5)
 		status_led="zcn-1523h:amber:init"
 		;;
 	wlr8100)