Hi Lizardo, On Thu, Jan 26, 2012 at 9:55 AM, Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx> wrote: > Hi Andre, > > On Wed, Jan 25, 2012 at 6:52 PM, Andre Guedes > <andre.guedes@xxxxxxxxxxxxx> wrote: >> - err = hci_do_inquiry(hdev, INQUIRY_LEN_BREDR); >> + if (test_bit(MGMT_ADDR_BREDR, &discov_type)) >> + err = hci_do_inquiry(hdev, INQUIRY_LEN_BREDR); >> + else if (test_bit(MGMT_ADDR_LE_PUBLIC, &discov_type) && >> + test_bit(MGMT_ADDR_LE_RANDOM, &discov_type)) > > Should it be "||" in the line above? mgmt-api.txt says both bits should be set. > >> + err = hci_le_scan(hdev, LE_SCAN_TYPE, LE_SCAN_INT, >> + LE_SCAN_WIN, LE_SCAN_TIMEOUT_LE_ONLY); >> + else >> + err = -EINVAL; >> + >> if (err < 0) >> mgmt_pending_remove(cmd); >> else >> -- >> 1.7.8.4 >> >> -- >> 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 > > > Regards, > -- > Anderson Lizardo > Instituto Nokia de Tecnologia - INdT > Manaus - Brazil -- 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