Re: [PATCH 0/4] net-next: dsa: set_addr should be optional

2016-09-20 Thread David Miller 
From: John Crispin 
Date: Mon, 19 Sep 2016 15:27:59 +0200

> The Marvell driver is the only one that actually sets the switches HW
> address. All other drivers have an empty stub. fix this by making the
> callback optional.

Series applied, thanks.

Re: [PATCH 0/4] net-next: dsa: set_addr should be optional

2016-09-19 Thread Florian Fainelli 
On 09/19/2016 06:27 AM, John Crispin wrote:
> The Marvell driver is the only one that actually sets the switches HW
> address. All other drivers have an empty stub. fix this by making the
> callback optional.
> 
> John Crispin (4):
>   net-next: dsa: fix duplicate invocation of set_addr()
>   net-next: dsa: make the set_addr() operation optional
>   net-next: dsa: b53: remove empty set_addr() stub
>   net-next: dsa: qca8k: remove empty set_addr() stub

Reviewed-by: Florian Fainelli 
-- 
Florian

Re: [PATCH 0/4] net-next: dsa: set_addr should be optional

2016-09-19 Thread Andrew Lunn 
On Mon, Sep 19, 2016 at 03:27:59PM +0200, John Crispin wrote:
> The Marvell driver is the only one that actually sets the switches HW
> address. All other drivers have an empty stub. fix this by making the
> callback optional.

Hi John

Thanks for doing this,

Reviewed-by: Andrew Lunn 

Andrew

[PATCH 0/4] net-next: dsa: set_addr should be optional

2016-09-19 Thread John Crispin 
The Marvell driver is the only one that actually sets the switches HW
address. All other drivers have an empty stub. fix this by making the
callback optional.

John Crispin (4):
  net-next: dsa: fix duplicate invocation of set_addr()
  net-next: dsa: make the set_addr() operation optional
  net-next: dsa: b53: remove empty set_addr() stub
  net-next: dsa: qca8k: remove empty set_addr() stub

 drivers/net/dsa/b53/b53_common.c |6 --
 drivers/net/dsa/qca8k.c  |8 
 net/dsa/dsa.c|8 +---
 net/dsa/dsa2.c   |   12 +---
 4 files changed, 10 insertions(+), 24 deletions(-)

-- 
1.7.10.4