Hi Mat, On Fri, Oct 26, 2012 at 10:01:01AM -0700, Mat Martineau wrote: ... > >@@ -3948,13 +3955,15 @@ static inline int l2cap_config_rsp(struct l2cap_conn *conn, > > goto done; > > } > > > >- /* check compatibility */ > >- > > if (!chan->ctrl_id) > > l2cap_send_efs_conf_rsp(chan, buf, cmd->ident, > > 0); > >- else > >- chan->ident = cmd->ident; > >+ else { > >+ if (l2cap_check_efs(chan)) { > >+ amp_create_logical_link(chan); > >+ chan->ident = cmd->ident; > >+ } > >+ } > > Minor style issue - if one block of an if/else needs braces, then > they all get braces. Sure. Will fix this. 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