summaryrefslogtreecommitdiffstats
path: root/net/bluetooth/hci_core.c
diff options
context:
space:
mode:
authorGustavo Padovan2013-06-13 13:34:31 +0200
committerGustavo Padovan2013-06-23 04:05:47 +0200
commitb8f4e068004859eefac7b1ced59ddb67ca6d2d80 (patch)
tree9d88a1aea22e6220011d6de72bee82885ae7697e /net/bluetooth/hci_core.c
parentBluetooth: Add missing reset_resume dev_pm_ops (diff)
downloadkernel-qcow2-linux-b8f4e068004859eefac7b1ced59ddb67ca6d2d80.tar.gz
kernel-qcow2-linux-b8f4e068004859eefac7b1ced59ddb67ca6d2d80.tar.xz
kernel-qcow2-linux-b8f4e068004859eefac7b1ced59ddb67ca6d2d80.zip
Bluetooth: Improve comments on the HCI_Delete_Store_Link_Key issue
Some Bluetooth controllers doesn't support this command so we first need to check for its support before sending it. This patch adds a lengthful commentary about this. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Diffstat (limited to 'net/bluetooth/hci_core.c')
-rw-r--r--net/bluetooth/hci_core.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index e2e9d409d0f6..061523eb52a1 100644
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -597,7 +597,15 @@ static void hci_init3_req(struct hci_request *req, unsigned long opt)
struct hci_dev *hdev = req->hdev;
u8 p;
- /* Only send HCI_Delete_Stored_Link_Key if it is supported */
+ /* Some Broadcom based Bluetooth controllers do not support the
+ * Delete Stored Link Key command. They are clearly indicating its
+ * absence in the bit mask of supported commands.
+ *
+ * Check the supported commands and only if the the command is marked
+ * as supported send it. If not supported assume that the controller
+ * does not have actual support for stored link keys which makes this
+ * command redundant anyway.
+ */
if (hdev->commands[6] & 0x80) {
struct hci_cp_delete_stored_link_key cp;