Hi Gustavo, On Wed, May 23, 2012 at 10:12:43PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > > Remove another socket usage from l2cap_core.c Good idea. ... > --- a/net/bluetooth/l2cap_sock.c > +++ b/net/bluetooth/l2cap_sock.c > @@ -623,10 +623,15 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, ch > break; > } > > - if (opt) > + if (opt) { > set_bit(BT_SK_DEFER_SETUP, &bt_sk(sk)->flags); > - else > + set_bit(CONF_DEFER_SETUP, &chan->conf_state); > + } else { > clear_bit(BT_SK_DEFER_SETUP, &bt_sk(sk)->flags); > + clear_bit(CONF_DEFER_SETUP, &chan->conf_state); Do we now have 2 similar flags? > + } > + > + Remove extra line Best regards Andrei Emeltchenko -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html