Hi Andrei,
On Nov 7, 2011, at 11:09 AM, Andrei Emeltchenko wrote:
> Hi Andre,
>
> On Mon, Nov 07, 2011 at 10:58:29AM -0300, Andre Guedes wrote:
>> Hi Andrei,
>>
>> On Nov 7, 2011, at 6:03 AM, Andrei Emeltchenko wrote:
>>
>>> Hi Andre,
>>>
>>> On Fri, Nov 04, 2011 at 02:16:51PM -0300, Andre Guedes wrote:
>>>> This patch adds a function to hci_core to carry out inquiry.
>>>>
>>>> All inquiry code from start_discovery() were replaced by a
>>>> hci_do_inquiry() call.
>>>>
>>>> Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx>
>>>> ---
>>>> include/net/bluetooth/hci_core.h | 2 ++
>>>> net/bluetooth/hci_core.c | 17 +++++++++++++++++
>>>> net/bluetooth/mgmt.c | 9 +--------
>>>> 3 files changed, 20 insertions(+), 8 deletions(-)
>>>>
>>>> diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
>>>> index f97792c..ae36ac0 100644
>>>> --- a/include/net/bluetooth/hci_core.h
>>>> +++ b/include/net/bluetooth/hci_core.h
>>>> @@ -970,4 +970,6 @@ void hci_le_start_enc(struct hci_conn *conn, __le16 ediv, __u8 rand[8],
>>>> void hci_le_ltk_reply(struct hci_conn *conn, u8 ltk[16]);
>>>> void hci_le_ltk_neg_reply(struct hci_conn *conn);
>>>>
>>>> +int hci_do_inquiry(struct hci_dev *hdev, u8 length);
>>>> +
>>>> #endif /* __HCI_CORE_H */
>>>> diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
>>>> index b7f6b5b..098f26c 100644
>>>> --- a/net/bluetooth/hci_core.c
>>>> +++ b/net/bluetooth/hci_core.c
>>>> @@ -2560,3 +2560,20 @@ static void hci_cmd_task(unsigned long arg)
>>>> }
>>>> }
>>>> }
>>>> +
>>>> +int hci_do_inquiry(struct hci_dev *hdev, u8 length)
>>>> +{
>>>> + u8 lap[3] = { 0x33, 0x8b, 0x9e };
>>>
>>> what are those numbers above?
>>
>> This is the general inquiry LAP value defined in page 314 of
>> Core spec.
>
> Can we define those numbers (like HCI_GENERAL_INQ_LAP) or put comments?
Sure, I'll add a comment.
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
