Bluetooth: Make mgmt_auth_enable_complete() return void
The return value of mgmt_auth_enable_complete() function is not used and so just change it to return void. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
This commit is contained in:
parent
e546099c31
commit
464996aea4
|
@ -1110,7 +1110,7 @@ int mgmt_user_passkey_notify(struct hci_dev *hdev, bdaddr_t *bdaddr,
|
||||||
u8 entered);
|
u8 entered);
|
||||||
void mgmt_auth_failed(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 link_type,
|
void mgmt_auth_failed(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 link_type,
|
||||||
u8 addr_type, u8 status);
|
u8 addr_type, u8 status);
|
||||||
int mgmt_auth_enable_complete(struct hci_dev *hdev, u8 status);
|
void mgmt_auth_enable_complete(struct hci_dev *hdev, u8 status);
|
||||||
int mgmt_ssp_enable_complete(struct hci_dev *hdev, u8 enable, u8 status);
|
int mgmt_ssp_enable_complete(struct hci_dev *hdev, u8 enable, u8 status);
|
||||||
int mgmt_set_class_of_dev_complete(struct hci_dev *hdev, u8 *dev_class,
|
int mgmt_set_class_of_dev_complete(struct hci_dev *hdev, u8 *dev_class,
|
||||||
u8 status);
|
u8 status);
|
||||||
|
|
|
@ -4693,37 +4693,33 @@ void mgmt_auth_failed(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 link_type,
|
||||||
mgmt_event(MGMT_EV_AUTH_FAILED, hdev, &ev, sizeof(ev), NULL);
|
mgmt_event(MGMT_EV_AUTH_FAILED, hdev, &ev, sizeof(ev), NULL);
|
||||||
}
|
}
|
||||||
|
|
||||||
int mgmt_auth_enable_complete(struct hci_dev *hdev, u8 status)
|
void mgmt_auth_enable_complete(struct hci_dev *hdev, u8 status)
|
||||||
{
|
{
|
||||||
struct cmd_lookup match = { NULL, hdev };
|
struct cmd_lookup match = { NULL, hdev };
|
||||||
bool changed = false;
|
bool changed;
|
||||||
int err = 0;
|
|
||||||
|
|
||||||
if (status) {
|
if (status) {
|
||||||
u8 mgmt_err = mgmt_status(status);
|
u8 mgmt_err = mgmt_status(status);
|
||||||
mgmt_pending_foreach(MGMT_OP_SET_LINK_SECURITY, hdev,
|
mgmt_pending_foreach(MGMT_OP_SET_LINK_SECURITY, hdev,
|
||||||
cmd_status_rsp, &mgmt_err);
|
cmd_status_rsp, &mgmt_err);
|
||||||
return 0;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (test_bit(HCI_AUTH, &hdev->flags)) {
|
if (test_bit(HCI_AUTH, &hdev->flags))
|
||||||
if (!test_and_set_bit(HCI_LINK_SECURITY, &hdev->dev_flags))
|
changed = !test_and_set_bit(HCI_LINK_SECURITY,
|
||||||
changed = true;
|
&hdev->dev_flags);
|
||||||
} else {
|
else
|
||||||
if (test_and_clear_bit(HCI_LINK_SECURITY, &hdev->dev_flags))
|
changed = test_and_clear_bit(HCI_LINK_SECURITY,
|
||||||
changed = true;
|
&hdev->dev_flags);
|
||||||
}
|
|
||||||
|
|
||||||
mgmt_pending_foreach(MGMT_OP_SET_LINK_SECURITY, hdev, settings_rsp,
|
mgmt_pending_foreach(MGMT_OP_SET_LINK_SECURITY, hdev, settings_rsp,
|
||||||
&match);
|
&match);
|
||||||
|
|
||||||
if (changed)
|
if (changed)
|
||||||
err = new_settings(hdev, match.sk);
|
new_settings(hdev, match.sk);
|
||||||
|
|
||||||
if (match.sk)
|
if (match.sk)
|
||||||
sock_put(match.sk);
|
sock_put(match.sk);
|
||||||
|
|
||||||
return err;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void clear_eir(struct hci_request *req)
|
static void clear_eir(struct hci_request *req)
|
||||||
|
|
Loading…
Reference in New Issue