Hi Marcel, On Sep 20, 2011, at 9:45 AM, Marcel Holtmann wrote: > Hi Andre, > >> Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx> >> --- >> net/bluetooth/hci_event.c | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c >> index c9d641b..47abba4 100644 >> --- a/net/bluetooth/hci_event.c >> +++ b/net/bluetooth/hci_event.c >> @@ -897,12 +897,16 @@ static void hci_cc_le_set_scan_enable(struct hci_dev *hdev, >> return; >> >> if (cp->enable == 0x01) { >> + mgmt_discovering(hdev->id, 1); >> + >> del_timer(&hdev->adv_timer); >> >> hci_dev_lock(hdev); >> hci_adv_entries_clear(hdev); >> hci_dev_unlock(hdev); >> } else if (cp->enable == 0x00) { >> + mgmt_discovering(hdev->id, 0); >> + > > do we wanna send discovering on/off for BR/EDR and then again for LE. > Don't we wanna have something like this: > > discovering on > BR/EDR inquiry > LE scan > discovering off Yes, we have exactly this. Notice that at this point (patch 09/14) we don't have BR/EDR/LE discovery supported yet. We are preparing for support LE-Only discovery. In a further patch we implement BR/EDR/LE discovery support and do the proper handling for mgmt_discovering event (see patch 14/14). BR, 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