Commit 406ef2a6 authored by Johan Hedberg's avatar Johan Hedberg Committed by Marcel Holtmann
Browse files

Bluetooth: Make Fast Connectable available while powered off 



To maximize the usability of the Fast Connectable feature we should make
it possible to set (or unset) it at any given moment. This means
removing the dependency on the 'connectable' setting as well as the
'powered' setting. The former makes also sense since page scan may get
enabled through add_device even if 'connectable' is false. To keep the
setting available over power cycles its flag also needs to be removed
from the flags that are cleared upon HCI_Reset.

Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
parent 0402d9f2

include/net/bluetooth/hci.h

+1 −1
Original line number Diff line number Diff line
@@ -231,7 +231,7 @@ enum { 
 * or the HCI device is closed.

 */

#define HCI_PERSISTENT_MASK (BIT(HCI_LE_SCAN) | BIT(HCI_PERIODIC_INQ) | \

			      BIT(HCI_FAST_CONNECTABLE) | BIT(HCI_LE_ADV))

			     BIT(HCI_LE_ADV))



/* HCI timeouts */

#define HCI_DISCONN_TIMEOUT	msecs_to_jiffies(2000)	/* 2 seconds */

net/bluetooth/mgmt.c

+12 −18
Original line number Diff line number Diff line
@@ -1968,15 +1968,6 @@ static int set_connectable(struct sock *sk, struct hci_dev *hdev, void *data, 
	}



no_scan_update:

	/* If we're going from non-connectable to connectable or

	 * vice-versa when fast connectable is enabled ensure that fast

	 * connectable gets disabled. write_fast_connectable won't do

	 * anything if the page scan parameters are already what they

	 * should be.

	 */

	if (cp->val || test_bit(HCI_FAST_CONNECTABLE, &hdev->dev_flags))

		write_fast_connectable(&req, false);



	/* Update the advertising parameters if necessary */

	if (test_bit(HCI_ADVERTISING, &hdev->dev_flags))

		enable_advertising(&req);
@@ -4660,14 +4651,6 @@ static int set_fast_connectable(struct sock *sk, struct hci_dev *hdev, 
		return mgmt_cmd_status(sk, hdev->id, MGMT_OP_SET_FAST_CONNECTABLE,

				       MGMT_STATUS_INVALID_PARAMS);



	if (!hdev_is_powered(hdev))

		return mgmt_cmd_status(sk, hdev->id, MGMT_OP_SET_FAST_CONNECTABLE,

				       MGMT_STATUS_NOT_POWERED);



	if (!test_bit(HCI_CONNECTABLE, &hdev->dev_flags))

		return mgmt_cmd_status(sk, hdev->id, MGMT_OP_SET_FAST_CONNECTABLE,

				       MGMT_STATUS_REJECTED);



	hci_dev_lock(hdev);



	if (mgmt_pending_find(MGMT_OP_SET_FAST_CONNECTABLE, hdev)) {
@@ -4682,6 +4665,14 @@ static int set_fast_connectable(struct sock *sk, struct hci_dev *hdev, 
		goto unlock;

	}



	if (!hdev_is_powered(hdev)) {

		change_bit(HCI_FAST_CONNECTABLE, &hdev->dev_flags);

		err = send_settings_rsp(sk, MGMT_OP_SET_FAST_CONNECTABLE,

					hdev);

		new_settings(hdev, sk);

		goto unlock;

	}



	cmd = mgmt_pending_add(sk, MGMT_OP_SET_FAST_CONNECTABLE, hdev,

			       data, len);

	if (!cmd) {
@@ -6481,6 +6472,9 @@ static int powered_update_hci(struct hci_dev *hdev) 
			    sizeof(link_sec), &link_sec);



	if (lmp_bredr_capable(hdev)) {

		if (test_bit(HCI_FAST_CONNECTABLE, &hdev->dev_flags))

			write_fast_connectable(&req, true);

		else

			write_fast_connectable(&req, false);

		__hci_update_page_scan(&req);

		update_class(&req);

CRA Git | Maintained and supported by SUSTech CRA and CCSE