Bluetooth: Move HCI_LIMITED_DISCOVERABLE changes to a general place
authorJohan Hedberg <johan.hedberg@intel.com>
Sat, 19 Oct 2013 20:38:22 +0000 (23:38 +0300)
committerMarcel Holtmann <marcel@holtmann.org>
Sun, 20 Oct 2013 16:05:41 +0000 (09:05 -0700)
We'll soon be introducing also LE support for the Set Discoverable
management command, so move the HCI_LIMITED_DISCOVERABLE flag clearing
and setting out from the if-branch that is only used for a BR/EDR
specific HCI command.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/mgmt.c

index 67b95a8..796db58 100644 (file)
@@ -1220,6 +1220,12 @@ static int set_discoverable(struct sock *sk, struct hci_dev *hdev, void *data,
        cancel_delayed_work(&hdev->discov_off);
        hdev->discov_timeout = timeout;
 
+       /* Limited discoverable mode */
+       if (cp->val == 0x02)
+               set_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
+       else
+               clear_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
+
        hci_req_init(&req, hdev);
 
        scan = SCAN_PAGE;
@@ -1229,8 +1235,6 @@ static int set_discoverable(struct sock *sk, struct hci_dev *hdev, void *data,
 
                if (cp->val == 0x02) {
                        /* Limited discoverable mode */
-                       set_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
-
                        hci_cp.num_iac = 2;
                        hci_cp.iac_lap[0] = 0x00;       /* LIAC */
                        hci_cp.iac_lap[1] = 0x8b;
@@ -1240,8 +1244,6 @@ static int set_discoverable(struct sock *sk, struct hci_dev *hdev, void *data,
                        hci_cp.iac_lap[5] = 0x9e;
                } else {
                        /* General discoverable mode */
-                       clear_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
-
                        hci_cp.num_iac = 1;
                        hci_cp.iac_lap[0] = 0x33;       /* GIAC */
                        hci_cp.iac_lap[1] = 0x8b;