Re: [LEDE-DEV] [PATCH] ar71xx: fix vlan settings for some boards

2017-02-21 Thread 郭传鈜
I think this patch needn't be reverted as it do fix the incorrect setting of marking port4 as eth1.What we need is just another commit to separate ar9331/ar934x devices from these to correct the port order of the 4 lan ports. Regards. Chuanhong Guo 2017-02-21 2:39 GMT+08:00 John Crispin : > > >

Re: [LEDE-DEV] [PATCH] ar71xx: fix vlan settings for some boards

2017-02-20 Thread John Crispin
On 20/02/2017 19:31, Piotr Dymacz wrote: > Hello, > > 2017-02-20 13:11 GMT+01:00 Conor O'Gorman : >> On 18/02/17 14:32, Weijie Gao wrote: >>> >>> For AR71XX devices, GMAC1 always connects port 0 of the built-in switch, >>> as the CPU port. >>> >>> This patch sets correct vlan for some devices wi

Re: [LEDE-DEV] [PATCH] ar71xx: fix vlan settings for some boards

2017-02-20 Thread Piotr Dymacz
Hello, 2017-02-20 13:11 GMT+01:00 Conor O'Gorman : > On 18/02/17 14:32, Weijie Gao wrote: >> >> For AR71XX devices, GMAC1 always connects port 0 of the built-in switch, >> as the CPU port. >> >> This patch sets correct vlan for some devices with wrong settings: >> a) mark port 0 as CPU port, tagge

Re: [LEDE-DEV] [PATCH] ar71xx: fix vlan settings for some boards

2017-02-20 Thread Conor O'Gorman
On 18/02/17 14:32, Weijie Gao wrote: For AR71XX devices, GMAC1 always connects port 0 of the built-in switch, as the CPU port. This patch sets correct vlan for some devices with wrong settings: a) mark port 0 as CPU port, tagged b) reverse port order, marking these ports untagged This change a

[LEDE-DEV] [PATCH] ar71xx: fix vlan settings for some boards

2017-02-18 Thread Weijie Gao
For AR71XX devices, GMAC1 always connects port 0 of the built-in switch, as the CPU port. This patch sets correct vlan for some devices with wrong settings: a) mark port 0 as CPU port, tagged b) reverse port order, marking these ports untagged Signed-off-by: Weijie Gao --- target/linux/ar71xx/b