Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-23 Thread Andrew Lunn
> 9:8Frame Mode: 00 -> Normal Network > 01 -> DSA (FRAME_MODE_DSA) > 10 -> Provider (FRAME_MODE_PROVIDER) > 11 -> Ether Type DSA (FRAME_ETHER_TYPE_DSA) Ah, there is one issue. This device supports EDSA. However, MV88E6XXX_FLAGS_F

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-23 Thread Stefan Eichenberger
Hi Vivien On Wed, Nov 23, 2016 at 10:59:13AM -0500, Vivien Didelot wrote: > Hi Stefan, > > Stefan Eichenberger writes: > > >> Now, the different families are not 100% compatible with each > >> other. We never had access to a 6097, so it has not been tested > >> recently, and we have probably br

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-23 Thread Vivien Didelot
Hi Stefan, Stefan Eichenberger writes: >> Now, the different families are not 100% compatible with each >> other. We never had access to a 6097, so it has not been tested >> recently, and we have probably broken it... My guess would be, >> anywhere mv88e6xxx_6095_family(chip) is used, there also

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-23 Thread Stefan Eichenberger
On Tue, Nov 22, 2016 at 08:02:06PM +0100, Andrew Lunn wrote: > On Tue, Nov 22, 2016 at 07:37:33PM +0100, Stefan Eichenberger wrote: > > Hi Andrew > > > > On Tue, Nov 22, 2016 at 04:03:30PM +0100, Andrew Lunn wrote: > > > On Tue, Nov 22, 2016 at 11:39:44AM +0100, Stefan Eichenberger wrote: > > > >

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-23 Thread Stefan Eichenberger
Hi Vivien On Tue, Nov 22, 2016 at 05:15:25PM -0500, Vivien Didelot wrote: > Hi Andrew, Stefan, > > Andrew Lunn writes: > > > What you might find useful is > > > > https://github.com/vivien/linux.git 161b96bd7d16d21b0f046c935b70c3b2d277ccc2 > > > > although it might need some changes for recent

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-22 Thread Vivien Didelot
Hi Andrew, Stefan, Andrew Lunn writes: > What you might find useful is > > https://github.com/vivien/linux.git 161b96bd7d16d21b0f046c935b70c3b2d277ccc2 > > although it might need some changes for recent commits. > > With that, you can see deeper into the switches registers. FYI, I have rebased

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-22 Thread Andrew Lunn
On Tue, Nov 22, 2016 at 07:37:33PM +0100, Stefan Eichenberger wrote: > Hi Andrew > > On Tue, Nov 22, 2016 at 04:03:30PM +0100, Andrew Lunn wrote: > > On Tue, Nov 22, 2016 at 11:39:44AM +0100, Stefan Eichenberger wrote: > > > Egress multicast and egress unicast is only enabled for CPU/DSA ports > >

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-22 Thread Stefan Eichenberger
Hi Andrew On Tue, Nov 22, 2016 at 04:03:30PM +0100, Andrew Lunn wrote: > On Tue, Nov 22, 2016 at 11:39:44AM +0100, Stefan Eichenberger wrote: > > Egress multicast and egress unicast is only enabled for CPU/DSA ports > > but for switching operation it seems it should be enabled for all ports. > > D

Re: [PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-22 Thread Andrew Lunn
On Tue, Nov 22, 2016 at 11:39:44AM +0100, Stefan Eichenberger wrote: > Egress multicast and egress unicast is only enabled for CPU/DSA ports > but for switching operation it seems it should be enabled for all ports. > Do I miss something here? > > I did the following test: > brctl addbr br0 > brct

[PATCH] net: dsa: mv88e6xxx: egress all frames

2016-11-22 Thread Stefan Eichenberger
Egress multicast and egress unicast is only enabled for CPU/DSA ports but for switching operation it seems it should be enabled for all ports. Do I miss something here? I did the following test: brctl addbr br0 brctl addif br0 lan0 brctl addif br0 lan1 In this scenario the unicast and multicast p