Bluetooth: btintel: Fix calls to __hci_cmd_sync()
authorFrederic Danis <frederic.danis@linux.intel.com>
Fri, 15 May 2015 09:58:41 +0000 (11:58 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 15 May 2015 14:04:49 +0000 (16:04 +0200)
Remove test of command reply status as it is already performed by
__hci_cmd_sync().

__hci_cmd_sync_ev() function already returns an error if it got a
non-zero status either through a Command Complete or a Command
Status event.

For both of these events the status is collected up in the event
handlers called by hci_event_packet() and then passed as the second
parameter to req_complete_skb(). The req_complete_skb() callback in
turn is hci_req_sync_complete() for __hci_cmd_sync_ev() which stores
the status in hdev->req_result. The hdev->req_result is then further
converted through bt_to_errno() back in __hci_cmd_sync_ev().

Signed-off-by: Frederic Danis <frederic.danis@linux.intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/btintel.c

index 2d43d4279b0092d8cfe5f225e6574e7d4ccea67e..828f2f8d1568c8c50962dee7d8e77fcfcd669972 100644 (file)
@@ -53,12 +53,6 @@ int btintel_check_bdaddr(struct hci_dev *hdev)
        }
 
        bda = (struct hci_rp_read_bd_addr *)skb->data;
-       if (bda->status) {
-               BT_ERR("%s: Intel device address result failed (%02x)",
-                      hdev->name, bda->status);
-               kfree_skb(skb);
-               return -bt_to_errno(bda->status);
-       }
 
        /* For some Intel based controllers, the default Bluetooth device
         * address 00:03:19:9E:8B:00 can be found. These controllers are