On 12 May 2015 at 10:16, Hante Meuleman <meule...@broadcom.com> wrote: > Do not mimic the vlan configuration in some devices like R8000 as the > configuration is to be used with SW accelerators not available in OpenWRT. > It is not possible to use the switch with vlan port configuration 0 1 2 3 5 7 > 8, > without making some huge modifications. When this configuration is used > the packets received will be without vlan tag (identified by status field in > gmac). This mode is used for an optimization which is not easily to be used > within the current OpenWRT driver framework.
I think I fixed it now, with indeed few modifications, but not that huge ones after all. If you could test latest OpenWrt trunk/master it would be great. > The patch below will modify the existing vlanport configuration so it will > work > on the r8000 with the existing opensource drivers. The problem with this patch (apart from S-o-b) was the silent way of handling R8000. You dropped code that recognized R8000 specific config just silently falling back to the default one. It would be much safer to leave R8000 case detection, just change the vlans config and add a proper comment. Just for the future :) _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel