Hi Ulisses, On Fri, Apr 20, 2012 at 11:09:29AM -0300, Ulisses Furquim wrote: > > +static void a2mp_chan_state_change_cb(void *data, int state) > > +{ > > + struct amp_mgr *mgr = data; > > + struct l2cap_chan *chan; > > + > > + if (!mgr) > > + return; > > + > > + chan = mgr->a2mp_chan; > > + > > + BT_DBG("chan %p state %s", chan, state_to_string(state)); > > + > > + chan->state = state; > > Do we have to l2cap_chan_lock() here? Or is that taken care somewhere else? Yes by caller of state_change 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