Hi Andre, > This patch modifies hci_do_le_scan and hci_cancel_le_scan helpers so > we are able to start and stop LE scan with no timeout. This feature > will be used by the LE connection routine. > > Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx> > --- > net/bluetooth/hci_core.c | 17 +++++++++-------- > 1 file changed, 9 insertions(+), 8 deletions(-) > > diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c > index 22e77a7..3aa0345 100644 > --- a/net/bluetooth/hci_core.c > +++ b/net/bluetooth/hci_core.c > @@ -1618,26 +1618,27 @@ static int hci_do_le_scan(struct hci_dev *hdev, u8 type, u16 interval, > if (err < 0) > return err; > > - queue_delayed_work(hdev->workqueue, &hdev->le_scan_disable, > - msecs_to_jiffies(timeout)); > + if (timeout > 0) > + queue_delayed_work(hdev->workqueue, &hdev->le_scan_disable, > + msecs_to_jiffies(timeout)); > > return 0; > } I really do not like this magic handling of scan disable. Maybe you better remove the timeout handling completely and put it into the discovery functionality. Doing it like this seems pretty much hacked together. It no longer looks like the right place to do handle it. Regards Marcel -- 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
