On 5 June 2020 16:37:48 CEST, "Petr Štetiar" <yn...@true.cz> wrote:
>Commit 5acd1ed0be0d ("ramips: mt7621: fix Ubiquiti ER-X ports names and
>MAC addresses") didn't only changed naming, but also removed WAN
>functionality from eth0 port, which is not desired, so lets add it
>back.

Hi Petr, 

as far as I'm concerned, this was/is desired. There has been an older 
discussion on the list and a more recent PR similar to what you do here:

https://github.com/openwrt/openwrt/pull/2961

I do think that all-LAN is closer to the default configuration, and thus would 
keep it, but I don't intend to repeat the discussion a third time. 

So, as in the PR, I would not merge this, but won't keep you from doing it if 
you have a strong preference. There are much more important things to spent 
time on discussing than this :-) 

Best

Adrian


>
>Cc: Chuanhong Guo <gch981...@gmail.com>
>Cc: Adrian Schmutzler <freif...@adrianschmutzler.de>
>Cc: DENG Qingfang <deng...@mail2.sysu.edu.cn>
>Fixes: 5acd1ed0be0d ("ramips: mt7621: fix Ubiquiti ER-X ports names and
>MAC addresses")
>Signed-off-by: Petr Štetiar <yn...@true.cz>
>---
>
>As in
>https://openwrt.org/toh/ubiquiti/ubiquiti_edgerouter_x_er-x_ka#switch_ports_for_vlans
>
> target/linux/ramips/mt7621/base-files/etc/board.d/02_network | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git
>a/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
>b/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
>index ba9f7fb7b613..5b0f7ff2cdef 100755
>--- a/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
>+++ b/target/linux/ramips/mt7621/base-files/etc/board.d/02_network
>@@ -44,7 +44,7 @@ ramips_setup_interfaces()
>               ucidef_set_interfaces_lan_wan "lan2 lan3 lan4 lan5" "wan"
>               ;;
>       ubnt,edgerouter-x)
>-              ucidef_set_interface_lan "eth0 eth1 eth2 eth3 eth4"
>+              ucidef_set_interface_lan_wan "eth1 eth2 eth3 eth4" "eth0"
>               ;;
>       ubnt,edgerouter-x-sfp)
>               ucidef_set_interface_lan "eth0 eth1 eth2 eth3 eth4 eth5"

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

Reply via email to