Hi Mat, On Thu, Sep 13, 2012 at 09:26:27AM -0700, Mat Martineau wrote: ... > >+static void l2cap_choose_conn(struct l2cap_chan *chan) > > I find this function name a little confusing - "choose" sounds like > it is just making a decision. This function also takes the action > of starting physical link setup or sending a connection request, > which is more than just "choosing". > > l2cap_start_connection, maybe? Sounds good to me. Will 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