This arranges the code in 02_network to resemble the structure using
lan_macaddr and wan_macaddr variables like in other targets as close
as possible (without becoming non-cosmetic).

Signed-off-by: Adrian Schmutzler <freif...@adrianschmutzler.de>
Cc: Rafał Miłecki <ra...@milecki.pl>
---
 .../bcm53xx/base-files/etc/board.d/02_network | 27 ++++++++-----------
 1 file changed, 11 insertions(+), 16 deletions(-)

diff --git a/target/linux/bcm53xx/base-files/etc/board.d/02_network 
b/target/linux/bcm53xx/base-files/etc/board.d/02_network
index f86f12407f..8492ec3277 100755
--- a/target/linux/bcm53xx/base-files/etc/board.d/02_network
+++ b/target/linux/bcm53xx/base-files/etc/board.d/02_network
@@ -71,37 +71,32 @@ bcm53xx_setup_interfaces()
 bcm53xx_setup_macs()
 {
        local board="$1"
+       local lan_macaddr
+       local wan_macaddr="$(nvram get wan_hwaddr)"
+       local etX_macaddr
 
-       case "$board" in
-       dlink,dir-885l | \
-       netgear,r7900 | \
-       netgear,r8000 | \
-       netgear,r8500)
-               # As vendor doesn't use eth0 its MAC may be missing. Use one 
from eth2.
-               et2macaddr="$(nvram get et2macaddr)"
-               [ -n "$et2macaddr" ] && ucidef_set_interface_macaddr "lan" 
"$et2macaddr"
-               ;;
-       esac
-
-       wan_macaddr="$(nvram get wan_hwaddr)"
        case "$board" in
        asus,rt-ac87u)
-               etXmacaddr=$(nvram get et1macaddr)
+               etX_macaddr=$(nvram get et1macaddr)
                ;;
        dlink,dir-885l | \
        netgear,r7900 | \
        netgear,r8000 | \
        netgear,r8500)
-               etXmacaddr=$(nvram get et2macaddr)
+               # As vendor doesn't use eth0 its MAC may be missing. Use one 
from eth2.
+               lan_macaddr=$(nvram get et2macaddr)
+               etX_macaddr=$(nvram get et2macaddr)
                ;;
        *)
-               etXmacaddr=$(nvram get et0macaddr)
+               etX_macaddr=$(nvram get et0macaddr)
                ;;
        esac
 
        # If WAN MAC isn't explicitly set, calculate it using base MAC as 
reference.
-       [ -z "$wan_macaddr" -a -n "$etXmacaddr" ] && wan_macaddr=$(macaddr_add 
"$etXmacaddr" 1)
+       # However, best things would be to set the proper wan_macaddr directly 
above
+       [ -z "$wan_macaddr" -a -n "$etX_macaddr" ] && wan_macaddr=$(macaddr_add 
"$etX_macaddr" 1)
 
+       [ -n "$lan_macaddr" ] && ucidef_set_interface_macaddr "lan" 
"$lan_macaddr"
        [ -n "$wan_macaddr" ] && ucidef_set_interface_macaddr "wan" 
"$wan_macaddr"
 }
 
-- 
2.20.1


_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to