Hi Vinicius, * Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxxxxxx> [2011-06-20 17:01:56 -0300]: > Hi Gustavo, > > On 16:31 Mon 20 Jun, Gustavo F. Padovan wrote: > > * Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxxxxxx> [2011-06-17 22:46:25 -0300]: > > > > > When the connection is ready we should set the connection > > > to CONNECTED so userspace can use it. > > > > > > Signed-off-by: Vinicius Costa Gomes <vinicius.gomes@xxxxxxxxxxxxx> > > > --- > > > net/bluetooth/l2cap_core.c | 3 +-- > > > 1 files changed, 1 insertions(+), 2 deletions(-) > > > > > > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > > > index 25f31f4..4af61fb 100644 > > > --- a/net/bluetooth/l2cap_core.c > > > +++ b/net/bluetooth/l2cap_core.c > > > @@ -895,8 +895,7 @@ static void l2cap_chan_ready(struct sock *sk) > > > chan->conf_state = 0; > > > __clear_chan_timer(chan); > > > > > > - sk->sk_state = BT_CONNECTED; > > > - sk->sk_state_change(sk); > > > + l2cap_state_change(chan, BT_CONNECTED); > > > > Why are you removing the sk->sk_state_change() ? > > I was thinking that l2cap_state_change() would do this for me. May I ask > why it doesn't call sk->sk_state_change()? Because we don't call sk->sk_state_change() every time the state changes. Gustavo -- 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
