Bluetooth: Don't bother doing anything if auto_connect doesn't change

When hci_conn_params_set() is called if the new auto_connect value is
the same as the old one we don't need to take any action. Simply return
success from the function in this case.

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

diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index dd8aa5f86810988e9a0aa519455b6036d7e21035..0601fcbd21eb88acd720b928645c53d0b8a25067 100644 (file)
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -3525,6 +3525,9 @@ int hci_conn_params_set(struct hci_dev *hdev, bdaddr_t *addr, u8 addr_type,
        if (!params)
                return -EIO;
 
+       if (params->auto_connect == auto_connect)
+               return 0;
+
        if (params->auto_connect == HCI_AUTO_CONN_REPORT &&
            auto_connect != HCI_AUTO_CONN_REPORT)
                list_del_init(&params->action);