Bluetooth: Remove magic numbers from le scan cmd
Make code readable by removing magic numbers. Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com> Acked-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
This commit is contained in:
parent
8a154a8feb
commit
68a8aea459
|
@ -804,6 +804,9 @@ struct hci_cp_le_set_scan_param {
|
||||||
__u8 filter_policy;
|
__u8 filter_policy;
|
||||||
} __packed;
|
} __packed;
|
||||||
|
|
||||||
|
#define LE_SCANNING_DISABLED 0x00
|
||||||
|
#define LE_SCANNING_ENABLED 0x01
|
||||||
|
|
||||||
#define HCI_OP_LE_SET_SCAN_ENABLE 0x200c
|
#define HCI_OP_LE_SET_SCAN_ENABLE 0x200c
|
||||||
struct hci_cp_le_set_scan_enable {
|
struct hci_cp_le_set_scan_enable {
|
||||||
__u8 enable;
|
__u8 enable;
|
||||||
|
|
|
@ -1033,7 +1033,8 @@ static void hci_cc_le_set_scan_enable(struct hci_dev *hdev,
|
||||||
if (!cp)
|
if (!cp)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
if (cp->enable == 0x01) {
|
switch (cp->enable) {
|
||||||
|
case LE_SCANNING_ENABLED:
|
||||||
set_bit(HCI_LE_SCAN, &hdev->dev_flags);
|
set_bit(HCI_LE_SCAN, &hdev->dev_flags);
|
||||||
|
|
||||||
cancel_delayed_work_sync(&hdev->adv_work);
|
cancel_delayed_work_sync(&hdev->adv_work);
|
||||||
|
@ -1041,12 +1042,19 @@ static void hci_cc_le_set_scan_enable(struct hci_dev *hdev,
|
||||||
hci_dev_lock(hdev);
|
hci_dev_lock(hdev);
|
||||||
hci_adv_entries_clear(hdev);
|
hci_adv_entries_clear(hdev);
|
||||||
hci_dev_unlock(hdev);
|
hci_dev_unlock(hdev);
|
||||||
} else if (cp->enable == 0x00) {
|
break;
|
||||||
|
|
||||||
|
case LE_SCANNING_DISABLED:
|
||||||
clear_bit(HCI_LE_SCAN, &hdev->dev_flags);
|
clear_bit(HCI_LE_SCAN, &hdev->dev_flags);
|
||||||
|
|
||||||
cancel_delayed_work_sync(&hdev->adv_work);
|
cancel_delayed_work_sync(&hdev->adv_work);
|
||||||
queue_delayed_work(hdev->workqueue, &hdev->adv_work,
|
queue_delayed_work(hdev->workqueue, &hdev->adv_work,
|
||||||
jiffies + ADV_CLEAR_TIMEOUT);
|
jiffies + ADV_CLEAR_TIMEOUT);
|
||||||
|
break;
|
||||||
|
|
||||||
|
default:
|
||||||
|
BT_ERR("Used reserved LE_Scan_Enable param %d", cp->enable);
|
||||||
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue