Hi Dmitry, > -----Original Message----- > From: openwrt-devel [mailto:[email protected]] On > Behalf Of Dmitry Tunin > Sent: Montag, 23. September 2019 17:12 > To: Adrian Schmutzler <[email protected]> > Cc: OpenWrt Development List <[email protected]> > Subject: Re: [OpenWrt-Devel] [PATCH] ipq40xx: remove redundant > ucidef_set_interfaces_* in 02_network > > пн, 23 сент. 2019 г. в 16:34, Adrian Schmutzler > <[email protected]>: > > > > 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 <[email protected]> > > > > --- > > > > This is not tested on the affected devices. > > > > However, the same approach is taken for other targets, and > > network setup is essentially device-independent concerning the > > changes done here. > > --- > > target/linux/ipq40xx/base-files/etc/board.d/02_network | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/target/linux/ipq40xx/base-files/etc/board.d/02_network > > b/target/linux/ipq40xx/base-files/etc/board.d/02_network > > index e5ba7260f3..b7631a301c 100755 > > --- a/target/linux/ipq40xx/base-files/etc/board.d/02_network > > +++ b/target/linux/ipq40xx/base-files/etc/board.d/02_network > > @@ -24,14 +24,14 @@ ipq40xx_setup_interfaces() > > ;; > > asus,rt-ac58u|\ > > zyxel,nbg6617) > > - ucidef_set_interfaces_lan_wan "eth0" "eth1" > > + ucidef_set_interface_wan "eth1" > > ucidef_add_switch "switch0" \ > > "0u@eth0" "1:lan:4" "2:lan:3" "3:lan:2" "4:lan:1" > > ;; > > I planned to investigate this, but didn't find time yet. The config > initially is wrong. > For asus,rt-ac58u|\ > > zyxel,nbg6617) > > There should be port 5 on vid 2 for eth 1 to enable vlan working. > Currently saving any config in luci breaks vlans, because port 5 isn't > defined here. The port 5 is removed. > I suggest lo leave this alone for a while. I am going to find some > kind of a solultion and suggest it.
Fine with me, as my patch is purely cosmetical and low prio anyway. Are you just talking about asus/zyxel case or should I remove the entire patch (-> mark as deferred)? Best Adrian > > _______________________________________________ > openwrt-devel mailing list > [email protected] > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
openpgp-digital-signature.asc
Description: PGP signature
_______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/mailman/listinfo/openwrt-devel
