Bluetooth: Remove redundant check for req.cmd_q

The hci_req_run() function already checks for empty cmd_q and bails
out if necessary. Also, req.cmd_q should really be treated as private
data of the request and not accessed directly.

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

diff --git a/net/bluetooth/hci_request.c b/net/bluetooth/hci_request.c
index edf2199de4ff3c259c96ce570d70b070a49f745b..f1529d7740f63848f449fc8898607f5486858d1d 100644 (file)
--- a/net/bluetooth/hci_request.c
+++ b/net/bluetooth/hci_request.c
@@ -1187,8 +1187,7 @@ static void adv_timeout_expire(struct work_struct *work)
        if (list_empty(&hdev->adv_instances))
                __hci_req_disable_advertising(&req);
 
-       if (!skb_queue_empty(&req.cmd_q))
-               hci_req_run(&req, NULL);
+       hci_req_run(&req, NULL);
 
 unlock:
        hci_dev_unlock(hdev);