Hi Gustavo, On Wed, Jul 13, 2011 at 5:21 PM, Gustavo Padovan <padovan@xxxxxxxxxxxxxx> wrote: >> @@ -1340,13 +1335,16 @@ static inline void hci_inquiry_complete_evt(struct hci_dev *hdev, struct sk_buff >> >> BT_DBG("%s status %d", hdev->name, status); >> >> - if (test_bit(HCI_MGMT, &hdev->flags) && >> - test_and_clear_bit(HCI_INQUIRY, &hdev->flags)) >> - mgmt_discovering(hdev->id, 0); >> + hci_conn_check_pending(hdev); >> + >> + if (!test_bit(HCI_INQUIRY, &hdev->flags)) >> + return; >> + >> + clear_bit(HCI_INQUIRY, &hdev->flags); > > this can be test_and_clear_bit Yes, I could have done like this. However it would be replaced by test_bit and clear_bit anyway later on patch 03/16. Andre -- 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