Bluetooth: Simplify device type check for AMP discover response
authorMarcel Holtmann <marcel@holtmann.org>
Sun, 6 Oct 2013 09:08:35 +0000 (02:08 -0700)
committerJohan Hedberg <johan.hedberg@intel.com>
Sun, 6 Oct 2013 09:12:47 +0000 (11:12 +0200)
When counting the number for AMP controllers, a positive check is
used. To be consistent, use the same check when actually adding
the data for the AMP contollers.

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

index ad8bda3a6fd171df1dfd3a25a12710cd212d1447..9410e272d8b66fc11824d79ffb41d46081005b09 100644 (file)
@@ -78,24 +78,20 @@ u8 __next_ident(struct amp_mgr *mgr)
 /* hci_dev_list shall be locked */
 static void __a2mp_add_cl(struct amp_mgr *mgr, struct a2mp_cl *cl)
 {
-       int i = 0;
        struct hci_dev *hdev;
+       int i = 1;
 
        cl[0].id = AMP_ID_BREDR;
        cl[0].type = AMP_TYPE_BREDR;
        cl[0].status = AMP_STATUS_BLUETOOTH_ONLY;
 
        list_for_each_entry(hdev, &hci_dev_list, list) {
-               /* Iterate through AMP controllers */
-               if (hdev->dev_type != HCI_AMP)
-                       continue;
-
-               /* Starting from second entry */
-               ++i;
-
-               cl[i].id = hdev->id;
-               cl[i].type = hdev->amp_type;
-               cl[i].status = hdev->amp_status;
+               if (hdev->dev_type == HCI_AMP) {
+                       cl[i].id = hdev->id;
+                       cl[i].type = hdev->amp_type;
+                       cl[i].status = hdev->amp_status;
+                       i++;
+               }
        }
 }