Commit 2b78e5f5 authored by Piotr Figiel's avatar Piotr Figiel Committed by Kalle Valo

brcmfmac: remove pending parameter from brcmf_usb_free_q

brcmf_usb_free_q is no longer called with pending=true thus this boolean
parameter is no longer needed.
Signed-off-by: default avatarPiotr Figiel <p.figiel@camlintechnologies.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent db3b9e2e
...@@ -445,9 +445,10 @@ brcmf_usbdev_qinit(struct list_head *q, int qsize) ...@@ -445,9 +445,10 @@ brcmf_usbdev_qinit(struct list_head *q, int qsize)
} }
static void brcmf_usb_free_q(struct list_head *q, bool pending) static void brcmf_usb_free_q(struct list_head *q)
{ {
struct brcmf_usbreq *req, *next; struct brcmf_usbreq *req, *next;
int i = 0; int i = 0;
list_for_each_entry_safe(req, next, q, list) { list_for_each_entry_safe(req, next, q, list) {
if (!req->urb) { if (!req->urb) {
...@@ -455,12 +456,8 @@ static void brcmf_usb_free_q(struct list_head *q, bool pending) ...@@ -455,12 +456,8 @@ static void brcmf_usb_free_q(struct list_head *q, bool pending)
break; break;
} }
i++; i++;
if (pending) { usb_free_urb(req->urb);
usb_kill_urb(req->urb); list_del_init(&req->list);
} else {
usb_free_urb(req->urb);
list_del_init(&req->list);
}
} }
} }
...@@ -1029,8 +1026,8 @@ static void brcmf_usb_detach(struct brcmf_usbdev_info *devinfo) ...@@ -1029,8 +1026,8 @@ static void brcmf_usb_detach(struct brcmf_usbdev_info *devinfo)
brcmf_dbg(USB, "Enter, devinfo %p\n", devinfo); brcmf_dbg(USB, "Enter, devinfo %p\n", devinfo);
/* free the URBS */ /* free the URBS */
brcmf_usb_free_q(&devinfo->rx_freeq, false); brcmf_usb_free_q(&devinfo->rx_freeq);
brcmf_usb_free_q(&devinfo->tx_freeq, false); brcmf_usb_free_q(&devinfo->tx_freeq);
usb_free_urb(devinfo->ctl_urb); usb_free_urb(devinfo->ctl_urb);
usb_free_urb(devinfo->bulk_urb); usb_free_urb(devinfo->bulk_urb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment