Bluetooth: btusb: Fixed too many in-token issue for Mediatek Chip.
authormark-yw.chen <mark-yw.chen@mediatek.com>
Mon, 12 Apr 2021 15:06:26 +0000 (23:06 +0800)
committerMarcel Holtmann <marcel@holtmann.org>
Sat, 26 Jun 2021 05:12:24 +0000 (07:12 +0200)
This patch reduce in-token during download patch procedure.
Don't submit urb for polling event before sending hci command.

Signed-off-by: mark-yw.chen <mark-yw.chen@mediatek.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/btusb.c

index 7f6ba2c975ed4e8e1c5e767cc2bb1dce7e594399..99fd88f7653d4fa82df075101992696517b7c474 100644 (file)
@@ -3312,11 +3312,6 @@ static int btusb_mtk_hci_wmt_sync(struct hci_dev *hdev,
        struct btmtk_wmt_hdr *hdr;
        int err;
 
-       /* Submit control IN URB on demand to process the WMT event */
-       err = btusb_mtk_submit_wmt_recv_urb(hdev);
-       if (err < 0)
-               return err;
-
        /* Send the WMT command and wait until the WMT event returns */
        hlen = sizeof(*hdr) + wmt_params->dlen;
        if (hlen > 255)
@@ -3342,6 +3337,11 @@ static int btusb_mtk_hci_wmt_sync(struct hci_dev *hdev,
                goto err_free_wc;
        }
 
+       /* Submit control IN URB on demand to process the WMT event */
+       err = btusb_mtk_submit_wmt_recv_urb(hdev);
+       if (err < 0)
+               return err;
+
        /* The vendor specific WMT commands are all answered by a vendor
         * specific event and will have the Command Status or Command
         * Complete as with usual HCI command flow control.