Hi Vishal, On Thu, Jun 07, 2012, Vishal Agarwal wrote: > +static void le_connect_complete_cb(struct hci_conn *conn, u8 status) > +{ > + struct pending_cmd *cmd; > + > + BT_DBG("status %u", status); > + > + if (!status && conn->type == LE_LINK) > + return; The patch looks good to me except that this test for LE_LINK seems unnecessary since this callback is anyway only used for LE. Could you remove it? Johan -- 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