Re: [PATCH net-next 1/3] net: dsa: Do not check for NULL dst in tag parsers

2017-04-07 Thread Vivien Didelot
Florian Fainelli writes: > dsa_switch_rcv() already tests for dst == NULL, so there is no need to > duplicate > the same check within the tag receive functions. > > Signed-off-by: Florian Fainelli Reviewed-by: Vivien Didelot

Re: [PATCH net-next 1/3] net: dsa: Do not check for NULL dst in tag parsers

2017-04-07 Thread Andrew Lunn
On Fri, Apr 07, 2017 at 01:41:52PM -0700, Florian Fainelli wrote: > dsa_switch_rcv() already tests for dst == NULL, so there is no need to > duplicate > the same check within the tag receive functions. > > Signed-off-by: Florian Fainelli Reviewed-by: Andrew Lunn Andrew

[PATCH net-next 1/3] net: dsa: Do not check for NULL dst in tag parsers

2017-04-07 Thread Florian Fainelli
dsa_switch_rcv() already tests for dst == NULL, so there is no need to duplicate the same check within the tag receive functions. Signed-off-by: Florian Fainelli --- net/dsa/tag_brcm.c| 3 --- net/dsa/tag_dsa.c | 3 --- net/dsa/tag_edsa.c| 3 --- net/dsa/tag_qca.c | 3 --- net/ds