Hi Johan,
On Nov 12, 2011, at 6:54 AM, Johan Hedberg wrote:
> Hi Andre,
>
> On Fri, Nov 11, 2011, Andre Guedes wrote:
>> @@ -1669,7 +1678,16 @@ static int start_discovery(struct sock *sk, u16 index)
>> goto failed;
>> }
>>
>> - err = hci_do_inquiry(hdev, INQUIRY_LEN_BREDR);
>> + if (lmp_host_le_capable(hdev)) {
>> + if (lmp_bredr_capable(hdev))
>> + err = -ENOSYS;
>> + else
>> + err = hci_do_le_scan(hdev, LE_SCAN_TYPE, LE_SCAN_INT,
>> + LE_SCAN_WIN, LE_SCAN_TIMEOUT_LE_ONLY);
>> + } else {
>> + err = hci_do_inquiry(hdev, INQUIRY_LEN_BREDR);
>> + }
>
> This should really be looking at type parameter that gets passed in the
> mgmt_start_discovery command. User space has been sending it already for
> some time but the kernel side wasn't yet updated to care about it. I
> just sent a patch to add the necessary mgmt.h struct and to pass the
> data to the start_discovery function.
The bluetooth-next tree didn't have those patches you are talking about
when I sent this series.
I'll consider the type parameter in v2. Thanks.
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
