Re: [PATCH RFC 1/3] net: bridge: log state when setting it

2016-02-13 Thread Vivien Didelot
Hi Ido, Ido Schimmel writes: > Fri, Feb 12, 2016 at 11:34:18PM IST, vivien.dide...@savoirfairelinux.com > wrote: > > Hi Vivien, > >>Every call to br_set_state is followed by a call to br_log_state. >>Directly call it within br_set_state instead. >> >>Signed-off-by: Vivien Didelot >>--- > > May

Re: [PATCH RFC 1/3] net: bridge: log state when setting it

2016-02-13 Thread Ido Schimmel
Fri, Feb 12, 2016 at 11:34:18PM IST, vivien.dide...@savoirfairelinux.com wrote: Hi Vivien, >Every call to br_set_state is followed by a call to br_log_state. >Directly call it within br_set_state instead. > >Signed-off-by: Vivien Didelot >--- Maybe I'm missing something, but there are other ins

[PATCH RFC 1/3] net: bridge: log state when setting it

2016-02-12 Thread Vivien Didelot
Every call to br_set_state is followed by a call to br_log_state. Directly call it within br_set_state instead. Signed-off-by: Vivien Didelot --- net/bridge/br_stp.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/net/bridge/br_stp.c b/net/bridge/br_stp.c index b3cca12..