From: Aloisio Almeida Jr <aloisio.almeida@xxxxxxxxxxxxx> It was added a parameter called 'reason' to le scan helpers (hci_do_le_scan and hci_cancel_le_scan). Distinguishing the reason to enbale/disable le scan will be important when both discovery and observer are implemented. 'reason' was also added do hdev structure (hdev.le_scan_req_reason) in order to make hci_cc_le_set_scan_enable to behave correctly. Signed-off-by: Aloisio Almeida Jr <aloisio.almeida@xxxxxxxxxxxxx> --- include/net/bluetooth/hci_core.h | 11 +++++-- net/bluetooth/hci_core.c | 63 +++++++++++++++++++++++++--------------- net/bluetooth/mgmt.c | 10 ++++--- 3 files changed, 54 insertions(+), 30 deletions(-) diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h index d5fd56a..8731add 100644 --- a/include/net/bluetooth/hci_core.h +++ b/include/net/bluetooth/hci_core.h @@ -121,6 +121,11 @@ struct le_scan_params { u16 interval; u16 window; int timeout; + u8 reason; +}; + +enum { + LE_SCAN_REQ_REASON_DISCOVERY, }; struct broadcast_data { @@ -289,6 +294,7 @@ struct hci_dev { struct work_struct le_scan; struct le_scan_params le_scan_params; + __u8 le_scan_req_reason; __s8 adv_tx_power; __u8 adv_data[HCI_MAX_AD_LENGTH]; @@ -1180,9 +1186,10 @@ void hci_le_start_enc(struct hci_conn *conn, __le16 ediv, __u8 rand[8], __u8 ltk[16]); int hci_do_inquiry(struct hci_dev *hdev, u8 length); int hci_cancel_inquiry(struct hci_dev *hdev); + int hci_le_scan(struct hci_dev *hdev, u8 type, u16 interval, u16 window, - int timeout); -int hci_cancel_le_scan(struct hci_dev *hdev); + int timeout, u8 reason); +int hci_cancel_le_scan(struct hci_dev *hdev, u8 reason); u8 bdaddr_to_le(u8 bdaddr_type); diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index 518c916..9b6c9e4 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -1654,20 +1654,20 @@ static void le_scan_param_req(struct hci_dev *hdev, unsigned long opt) static void le_scan_enable_req(struct hci_dev *hdev, unsigned long opt) { - struct hci_cp_le_set_scan_enable cp; + struct hci_cp_le_set_scan_enable *cp; - memset(&cp, 0, sizeof(cp)); - cp.enable = 1; - cp.filter_dup = 1; + cp = (struct hci_cp_le_set_scan_enable *) opt; - hci_send_cmd(hdev, HCI_OP_LE_SET_SCAN_ENABLE, sizeof(cp), &cp); + hci_send_cmd(hdev, HCI_OP_LE_SET_SCAN_ENABLE, + sizeof(struct hci_cp_le_set_scan_enable), cp); } static int hci_do_le_scan(struct hci_dev *hdev, u8 type, u16 interval, - u16 window, int timeout) + u16 window, int timeout, u8 reason) { long timeo = msecs_to_jiffies(3000); struct le_scan_params param; + struct hci_cp_le_set_scan_enable cp; int err; BT_DBG("%s", hdev->name); @@ -1679,38 +1679,52 @@ static int hci_do_le_scan(struct hci_dev *hdev, u8 type, u16 interval, param.interval = interval; param.window = window; + memset(&cp, 0, sizeof(cp)); + cp.enable = 1; + if (reason == LE_SCAN_REQ_REASON_DISCOVERY) + cp.filter_dup = 1; + + hci_dev_lock(hdev); + hdev->le_scan_req_reason = reason; + hci_dev_unlock(hdev); + hci_req_lock(hdev); err = __hci_request(hdev, le_scan_param_req, (unsigned long) ¶m, timeo); - if (!err) - err = __hci_request(hdev, le_scan_enable_req, 0, timeo); + if (err) + goto unlock; - hci_req_unlock(hdev); - - if (err < 0) - return err; + err = __hci_request(hdev, le_scan_enable_req, (unsigned long) &cp, + timeo); + if (err) + goto unlock; - schedule_delayed_work(&hdev->le_scan_disable, - msecs_to_jiffies(timeout)); + if (timeout > 0) + schedule_delayed_work(&hdev->le_scan_disable, + msecs_to_jiffies(timeout)); - return 0; +unlock: + hci_req_unlock(hdev); + return err; } -int hci_cancel_le_scan(struct hci_dev *hdev) +int hci_cancel_le_scan(struct hci_dev *hdev, u8 reason) { + struct hci_cp_le_set_scan_enable cp; + BT_DBG("%s", hdev->name); if (!test_bit(HCI_LE_SCAN, &hdev->dev_flags)) return -EALREADY; - if (cancel_delayed_work(&hdev->le_scan_disable)) { - struct hci_cp_le_set_scan_enable cp; + if (reason == LE_SCAN_REQ_REASON_DISCOVERY) + if (!cancel_delayed_work(&hdev->le_scan_disable)) + return 0; - /* Send HCI command to disable LE Scan */ - memset(&cp, 0, sizeof(cp)); - hci_send_cmd(hdev, HCI_OP_LE_SET_SCAN_ENABLE, sizeof(cp), &cp); - } + /* Send HCI command to disable LE Scan */ + memset(&cp, 0, sizeof(cp)); + hci_send_cmd(hdev, HCI_OP_LE_SET_SCAN_ENABLE, sizeof(cp), &cp); return 0; } @@ -1736,11 +1750,11 @@ static void le_scan_work(struct work_struct *work) BT_DBG("%s", hdev->name); hci_do_le_scan(hdev, param->type, param->interval, param->window, - param->timeout); + param->timeout, param->reason); } int hci_le_scan(struct hci_dev *hdev, u8 type, u16 interval, u16 window, - int timeout) + int timeout, u8 reason) { struct le_scan_params *param = &hdev->le_scan_params; @@ -1756,6 +1770,7 @@ int hci_le_scan(struct hci_dev *hdev, u8 type, u16 interval, u16 window, param->interval = interval; param->window = window; param->timeout = timeout; + param->reason = reason; queue_work(system_long_wq, &hdev->le_scan); diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 805ae3b..bed28e4 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -2377,7 +2377,8 @@ static int start_discovery(struct sock *sk, struct hci_dev *hdev, case DISCOV_TYPE_LE: if (lmp_host_le_capable(hdev)) err = hci_le_scan(hdev, LE_SCAN_TYPE, LE_SCAN_INT, - LE_SCAN_WIN, LE_SCAN_TIMEOUT_LE_ONLY); + LE_SCAN_WIN, LE_SCAN_TIMEOUT_LE_ONLY, + LE_SCAN_REQ_REASON_DISCOVERY); else err = -ENOTSUPP; break; @@ -2385,8 +2386,8 @@ static int start_discovery(struct sock *sk, struct hci_dev *hdev, case DISCOV_TYPE_INTERLEAVED: if (lmp_host_le_capable(hdev) && lmp_bredr_capable(hdev)) err = hci_le_scan(hdev, LE_SCAN_TYPE, LE_SCAN_INT, - LE_SCAN_WIN, - LE_SCAN_TIMEOUT_BREDR_LE); + LE_SCAN_WIN, LE_SCAN_TIMEOUT_BREDR_LE, + LE_SCAN_REQ_REASON_DISCOVERY); else err = -ENOTSUPP; break; @@ -2443,7 +2444,8 @@ static int stop_discovery(struct sock *sk, struct hci_dev *hdev, void *data, if (test_bit(HCI_INQUIRY, &hdev->flags)) err = hci_cancel_inquiry(hdev); else - err = hci_cancel_le_scan(hdev); + err = hci_cancel_le_scan(hdev, + LE_SCAN_REQ_REASON_DISCOVERY); break; -- 1.8.0.2 -- 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