Re: [PATCH v2 net-next] net: dsa: mv88e6xxx: assert SMI lock

2015-11-02 Thread David Miller
From: Vivien Didelot Date: Fri, 30 Oct 2015 18:56:45 -0400 > It's easy to forget to lock the smi_mutex before calling the low-level > _mv88e6xxx_reg_{read,write}, so add a assert_smi_lock function in them. > > Signed-off-by: Vivien Didelot Applied. -- To unsubscribe from this list: send the li

Re: [PATCH v2 net-next] net: dsa: mv88e6xxx: assert SMI lock

2015-11-02 Thread Vivien Didelot
On Nov. Monday 02 (45) 04:02 PM, Andrew Lunn wrote: > On Fri, Oct 30, 2015 at 06:56:45PM -0400, Vivien Didelot wrote: > > It's easy to forget to lock the smi_mutex before calling the low-level > > _mv88e6xxx_reg_{read,write}, so add a assert_smi_lock function in them. > > > > Signed-off-by: Vivien

Re: [PATCH v2 net-next] net: dsa: mv88e6xxx: assert SMI lock

2015-11-02 Thread Vivien Didelot
Hi Andrew, On Nov. Monday 02 (45) 04:02 PM, Andrew Lunn wrote: > On Fri, Oct 30, 2015 at 06:56:45PM -0400, Vivien Didelot wrote: > > It's easy to forget to lock the smi_mutex before calling the low-level > > _mv88e6xxx_reg_{read,write}, so add a assert_smi_lock function in them. > > > > Signed-of

Re: [PATCH v2 net-next] net: dsa: mv88e6xxx: assert SMI lock

2015-11-02 Thread Andrew Lunn
On Fri, Oct 30, 2015 at 06:56:45PM -0400, Vivien Didelot wrote: > It's easy to forget to lock the smi_mutex before calling the low-level > _mv88e6xxx_reg_{read,write}, so add a assert_smi_lock function in them. > > Signed-off-by: Vivien Didelot Acked-by: Andrew Lunn Since there is no followup

[PATCH v2 net-next] net: dsa: mv88e6xxx: assert SMI lock

2015-10-30 Thread Vivien Didelot
It's easy to forget to lock the smi_mutex before calling the low-level _mv88e6xxx_reg_{read,write}, so add a assert_smi_lock function in them. Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6xxx.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --g