Bluetooth: Introduce hci_dev_clear_flag helper macro

Instead of manually coding clear_bit on hdev->dev_flags all the time,
use hci_dev_clear_flag helper macro.

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

6 files changed, 51 insertions, 52 deletions

diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
index bc2a7e918da7..535b23c73c92 100644
--- a/include/net/bluetooth/hci_core.h
+++ b/include/net/bluetooth/hci_core.h
@@ -503,6 +503,7 @@ extern rwlock_t hci_dev_list_lock;
 extern struct mutex hci_cb_list_lock;
 
 #define hci_dev_set_flag(hdev, nr)    set_bit((nr), &(hdev)->dev_flags)
+#define hci_dev_clear_flag(hdev, nr)  clear_bit((nr), &(hdev)->dev_flags)
 #define hci_dev_test_flag(hdev, nr)   test_bit((nr), &(hdev)->dev_flags)
 
 /* ----- HCI interface to upper protocols ----- */
diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
index c0fd42ceff61..ee5e59839b02 100644
--- a/net/bluetooth/hci_conn.c
+++ b/net/bluetooth/hci_conn.c
@@ -700,7 +700,7 @@ static void hci_req_directed_advertising(struct hci_request *req,
          * and write a new random address. The flag will be set back on
          * as soon as the SET_ADV_ENABLE HCI command completes.
          */
-        clear_bit(HCI_LE_ADV, &hdev->dev_flags);
+        hci_dev_clear_flag(hdev, HCI_LE_ADV);
 
         /* Set require_privacy to false so that the remote device has a
          * chance of identifying us.
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index 252b597362eb..01710698e547 100644
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -591,7 +591,7 @@ static void hci_init2_req(struct hci_request *req, unsigned long opt)
         if (lmp_bredr_capable(hdev))
                 bredr_setup(req);
         else
-                clear_bit(HCI_BREDR_ENABLED, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_BREDR_ENABLED);
 
         if (lmp_le_capable(hdev))
                 le_setup(req);
@@ -1625,8 +1625,8 @@ static int hci_dev_do_close(struct hci_dev *hdev)
         if (hdev->discov_timeout > 0) {
                 cancel_delayed_work(&hdev->discov_off);
                 hdev->discov_timeout = 0;
-                clear_bit(HCI_DISCOVERABLE, &hdev->dev_flags);
-                clear_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_DISCOVERABLE);
+                hci_dev_clear_flag(hdev, HCI_LIMITED_DISCOVERABLE);
         }
 
         if (test_and_clear_bit(HCI_SERVICE_CACHE, &hdev->dev_flags))
@@ -1846,7 +1846,7 @@ static void hci_update_scan_state(struct hci_dev *hdev, u8 scan)
                 discov_changed = !test_and_set_bit(HCI_DISCOVERABLE,
                                                    &hdev->dev_flags);
         } else {
-                clear_bit(HCI_LIMITED_DISCOVERABLE, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_LIMITED_DISCOVERABLE);
                 discov_changed = test_and_clear_bit(HCI_DISCOVERABLE,
                                                     &hdev->dev_flags);
         }
@@ -2087,7 +2087,7 @@ static int hci_rfkill_set_block(void *data, bool blocked)
                     !hci_dev_test_flag(hdev, HCI_CONFIG))
                         hci_dev_do_close(hdev);
         } else {
-                clear_bit(HCI_RFKILLED, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_RFKILLED);
         }
 
         return 0;
@@ -2121,7 +2121,7 @@ static void hci_power_on(struct work_struct *work)
             (hdev->dev_type == HCI_BREDR &&
              !bacmp(&hdev->bdaddr, BDADDR_ANY) &&
              !bacmp(&hdev->static_addr, BDADDR_ANY))) {
-                clear_bit(HCI_AUTO_OFF, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_AUTO_OFF);
                 hci_dev_do_close(hdev);
         } else if (hci_dev_test_flag(hdev, HCI_AUTO_OFF)) {
                 queue_delayed_work(hdev->req_workqueue, &hdev->power_off,
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
index b01a93efada8..808b78cf8ad0 100644
--- a/net/bluetooth/hci_event.c
+++ b/net/bluetooth/hci_event.c
@@ -82,7 +82,7 @@ static void hci_cc_exit_periodic_inq(struct hci_dev *hdev, struct sk_buff *skb)
         if (status)
                 return;
 
-        clear_bit(HCI_PERIODIC_INQ, &hdev->dev_flags);
+        hci_dev_clear_flag(hdev, HCI_PERIODIC_INQ);
 
         hci_conn_check_pending(hdev);
 }
@@ -503,7 +503,7 @@ static void hci_cc_write_ssp_mode(struct hci_dev *hdev, struct sk_buff *skb)
                 if (sent->mode)
                         hci_dev_set_flag(hdev, HCI_SSP_ENABLED);
                 else
-                        clear_bit(HCI_SSP_ENABLED, &hdev->dev_flags);
+                        hci_dev_clear_flag(hdev, HCI_SSP_ENABLED);
         }
 
         hci_dev_unlock(hdev);
@@ -533,7 +533,7 @@ static void hci_cc_write_sc_support(struct hci_dev *hdev, struct sk_buff *skb)
                 if (sent->support)
                         hci_dev_set_flag(hdev, HCI_SC_ENABLED);
                 else
-                        clear_bit(HCI_SC_ENABLED, &hdev->dev_flags);
+                        hci_dev_clear_flag(hdev, HCI_SC_ENABLED);
         }
 
         hci_dev_unlock(hdev);
@@ -1117,7 +1117,7 @@ static void hci_cc_le_set_adv_enable(struct hci_dev *hdev, struct sk_buff *skb)
                                            &conn->le_conn_timeout,
                                            conn->conn_timeout);
         } else {
-                clear_bit(HCI_LE_ADV, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_LE_ADV);
         }
 
         hci_dev_unlock(hdev);
@@ -1217,7 +1217,7 @@ static void hci_cc_le_set_scan_enable(struct hci_dev *hdev,
                  */
                 cancel_delayed_work(&hdev->le_scan_disable);
 
-                clear_bit(HCI_LE_SCAN, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_LE_SCAN);
 
                 /* The HCI_LE_SCAN_INTERRUPTED flag indicates that we
                  * interrupted scanning due to a connect request. Mark
@@ -1391,8 +1391,8 @@ static void hci_cc_write_le_host_supported(struct hci_dev *hdev,
                 hci_dev_set_flag(hdev, HCI_LE_ENABLED);
         } else {
                 hdev->features[1][0] &= ~LMP_HOST_LE;
-                clear_bit(HCI_LE_ENABLED, &hdev->dev_flags);
-                clear_bit(HCI_ADVERTISING, &hdev->dev_flags);
+                hci_dev_clear_flag(hdev, HCI_LE_ENABLED);
+                hci_dev_clear_flag(hdev, HCI_ADVERTISING);
         }
 
         if (sent->simul)
@@ -4409,7 +4409,7 @@ static void hci_le_conn_complete_evt(struct hci_dev *hdev, struct sk_buff *skb)
         /* All controllers implicitly stop advertising in the event of a
          * connection, so ensure that the state bit is cleared.
          */
-        clear_bit(HCI_LE_ADV, &hdev->dev_flags);
+        hci_dev_clear_flag(hdev, HCI_LE_ADV);
 
         conn = hci_conn_hash_lookup_state(hdev, LE_LINK, BT_CONNECT);
         if (!conn) {
diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c
index b297709d82bf..ca402a0a6740 100644
--- a/net/bluetooth/hci_sock.c
+++ b/net/bluetooth/hci_sock.c
@@ -474,7 +474,7 @@ static int hci_sock_release(struct socket *sock)
         if (hdev) {