diff options
| author | Johan Hedberg <johan.hedberg@intel.com> | 2015-01-23 15:42:46 +0200 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2015-01-23 19:07:03 +0100 |
| commit | a1443f5a273713d4bfda360e45aa6e1d14fe7324 (patch) | |
| tree | 1c1ff41e6bac931e44c1d8da3b5ccb747fc6a6a1 /net/bluetooth/hci_event.c | |
| parent | 484aabc1c4e869879e614a621e2dfabc9f98fb00 (diff) | |
| download | op-kernel-dev-a1443f5a273713d4bfda360e45aa6e1d14fe7324.zip op-kernel-dev-a1443f5a273713d4bfda360e45aa6e1d14fe7324.tar.gz | |
Bluetooth: Convert Set SC to use HCI Request
This patch converts the Set Secure Connection HCI handling to use a HCI
request instead of using a hard-coded callback in hci_event.c. This e.g.
ensures that we don't clear the flags incorrectly if something goes
wrong with the power up process (not related to a mgmt Set SC command).
The code can also be simplified a bit since only one pending Set SC
command is allowed, i.e. mgmt_pending_foreach usage is not needed.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth/hci_event.c')
| -rw-r--r-- | net/bluetooth/hci_event.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c index a58845e..e2b81ad 100644 --- a/net/bluetooth/hci_event.c +++ b/net/bluetooth/hci_event.c @@ -525,9 +525,7 @@ static void hci_cc_write_sc_support(struct hci_dev *hdev, struct sk_buff *skb) hdev->features[1][0] &= ~LMP_HOST_SC; } - if (test_bit(HCI_MGMT, &hdev->dev_flags)) - mgmt_sc_enable_complete(hdev, sent->support, status); - else if (!status) { + if (!test_bit(HCI_MGMT, &hdev->dev_flags) && !status) { if (sent->support) set_bit(HCI_SC_ENABLED, &hdev->dev_flags); else |
