Bluetooth: Allow ACL packets over USB in HCI_RAW mode
authorPeter Hurley <peter@hurleysoftware.com>
Thu, 14 Jul 2011 12:48:32 +0000 (08:48 -0400)
committerGustavo F. Padovan <gustavo@padovan.org>
Tue, 27 Sep 2011 21:15:45 +0000 (18:15 -0300)
Removed tests which prevent transmission of ACL packets
when the device is in HCI_RAW mode. These tests verified that
there are ACL or LE links currently tracked by the HCI
connection manager. However, a HCI_RAW mode device does not
use the connection manager. In these circumstances, the connection
counts will be zero, and thus, transmitted ACL packets dropped.

The acl_num test is actually a vestige of a previous bulk URB
scheme that is no longer used by this driver (bulk URBs were not
started until at least one ACL connection was created). This was
incompatible with some endpoint implementations and was dropped -
see commit 43c2e57f94.

The utility of these tests is marginal - currently, the hci tx
scheduler cannot send an ACL or LE packet for an untracked connection
(except if the device is in HCI_RAW mode).

Lastly, no other transport layer driver enforces these same tests.

Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Acked-by: Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
drivers/bluetooth/btusb.c

index 9cbac6b..2755c1a 100644 (file)
@@ -708,8 +708,7 @@ static int btusb_send_frame(struct sk_buff *skb)
                break;
 
        case HCI_ACLDATA_PKT:
-               if (!data->bulk_tx_ep || (hdev->conn_hash.acl_num < 1 &&
-                                               hdev->conn_hash.le_num < 1))
+               if (!data->bulk_tx_ep)
                        return -ENODEV;
 
                urb = usb_alloc_urb(0, GFP_ATOMIC);