[OpenWrt-Devel] [PATCH] ath79: remove redundant ucidef_set_interfaces_* calls in 02_network

[Adrian Schmutzler]

If already included in ucidef_add_switch, you do not have to
additionally set the interface mode in ucidef_set_interfaces_*
functions.

This patch removes/adjusts such redundant cases.

Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

---

This is not tested on the affected devices.

However, since network setup is essentially device-independent,
the other cases which are already set up the same way should be
sufficient as "test".
---
 target/linux/ath79/base-files/etc/board.d/02_network | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/target/linux/ath79/base-files/etc/board.d/02_network b/target/linux/ath79/base-files/etc/board.d/02_network
index 0608a1fc43..15371a1dd9 100755
--- a/target/linux/ath79/base-files/etc/board.d/02_network
+++ b/target/linux/ath79/base-files/etc/board.d/02_network
@@ -116,7 +116,6 @@ ath79_setup_interfaces()
 		;;
 	comfast,cf-wr650ac-v1|\
 	comfast,cf-wr650ac-v2)
-		ucidef_set_interfaces_lan_wan "eth0" "eth1"
 		ucidef_add_switch "switch0" \
 			"0 at eth0" "1:lan" "2:lan" "3:lan" "4:lan" "5:wan" "6 at eth1"
 		;;
@@ -196,7 +195,7 @@ ath79_setup_interfaces()
 	netgear,wndr3700v2|\
 	netgear,wndr3800|\
 	netgear,wndr3800ch)
-		ucidef_set_interfaces_lan_wan "eth0" "eth1"
+		ucidef_set_interface_wan "eth1"
 		ucidef_add_switch "switch0" \
 			"0:lan:4" "1:lan:3" "2:lan:2" "3:lan:1" "5u at eth0"
 		ucidef_add_switch_attr "switch0" "blinkrate" 2
-- 
2.20.1


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