Re: [PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-25 Thread Egil Hjelmeland
On 24. juli 2017 18:57, Florian Fainelli wrote: On 07/20/2017 06:57 AM, Egil Hjelmeland wrote: Workaround for dsa_switch_mdb_add adding CPU port to group, but forgetting to remove it: Should not we move this logic one layer above into DSA then such that insertions and removals are strictly

Re: [PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-25 Thread Egil Hjelmeland
On 24. juli 2017 18:57, Florian Fainelli wrote: On 07/20/2017 06:57 AM, Egil Hjelmeland wrote: Workaround for dsa_switch_mdb_add adding CPU port to group, but forgetting to remove it: Should not we move this logic one layer above into DSA then such that insertions and removals are strictly

Re: [PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-24 Thread Florian Fainelli
On 07/20/2017 06:57 AM, Egil Hjelmeland wrote: > Workaround for dsa_switch_mdb_add adding CPU port to group, > but forgetting to remove it: > > Remove port 0 if only port 0 is only port left. > > Signed-off-by: Egil Hjelmeland > --- > drivers/net/dsa/lan9303-core.c

Re: [PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-24 Thread Florian Fainelli
On 07/20/2017 06:57 AM, Egil Hjelmeland wrote: > Workaround for dsa_switch_mdb_add adding CPU port to group, > but forgetting to remove it: > > Remove port 0 if only port 0 is only port left. > > Signed-off-by: Egil Hjelmeland > --- > drivers/net/dsa/lan9303-core.c | 11 +++ > 1 file

[PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-24 Thread Egil Hjelmeland
Workaround for dsa_switch_mdb_add adding CPU port to group, but forgetting to remove it: Remove port 0 if only port 0 is only port left. Signed-off-by: Egil Hjelmeland --- drivers/net/dsa/lan9303-core.c | 11 +++ 1 file changed, 11 insertions(+) diff --git

[PATCH 13/13] net: dsa: lan9303: lan9303_port_mdb_del remove port 0

2017-07-24 Thread Egil Hjelmeland
Workaround for dsa_switch_mdb_add adding CPU port to group, but forgetting to remove it: Remove port 0 if only port 0 is only port left. Signed-off-by: Egil Hjelmeland --- drivers/net/dsa/lan9303-core.c | 11 +++ 1 file changed, 11 insertions(+) diff --git