projects / thirdparty / kernel / stable-queue.git / blame
| Commit | Line | Data |
|---|---|---|
| e3010154 GKH |
1 | From 2d20c72c021d96f8b9230396c8e3782f204214ec Mon Sep 17 00:00:00 2001 |
| 2 | From: Valentin Longchamp <valentin.longchamp@epfl.ch> | |
| 3 | Date: Fri, 26 Mar 2010 11:44:33 +0100 | |
| 4 | Subject: setup correct int pipe type in ar9170_usb_exec_cmd | |
| 5 | ||
| 6 | From: Valentin Longchamp <valentin.longchamp@epfl.ch> | |
| 7 | ||
| 8 | commit 2d20c72c021d96f8b9230396c8e3782f204214ec upstream. | |
| 9 | ||
| 10 | An int urb is constructed but we fill it in with a bulk pipe type. | |
| 11 | ||
| 12 | Commit f661c6f8c67bd55e93348f160d590ff9edf08904 implemented a pipe type | |
| 13 | check when CONFIG_USB_DEBUG is enabled. The check failed for all the ar9170 | |
| 14 | usb transfers and the driver could not configure the wifi dongle. | |
| 15 | ||
| 16 | This went unnoticed until now because most people don't have | |
| 17 | CONFIG_USB_DEBUG enabled. | |
| 18 | ||
| 19 | Signed-off-by: Valentin Longchamp <valentin.longchamp@epfl.ch> | |
| 20 | Acked-by: Christian Lamparter <chunkeey@googlemail.com> | |
| 21 | Signed-off-by: John W. Linville <linville@tuxdriver.com> | |
| 22 | Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> | |
| 23 | ||
| 24 | --- | |
| 25 | drivers/net/wireless/ath/ar9170/usb.c | 2 +- | |
| 26 | 1 file changed, 1 insertion(+), 1 deletion(-) | |
| 27 | ||
| 28 | --- a/drivers/net/wireless/ath/ar9170/usb.c | |
| 29 | +++ b/drivers/net/wireless/ath/ar9170/usb.c | |
| 30 | @@ -414,7 +414,7 @@ static int ar9170_usb_exec_cmd(struct ar | |
| 31 | spin_unlock_irqrestore(&aru->common.cmdlock, flags); | |
| 32 | ||
| 33 | usb_fill_int_urb(urb, aru->udev, | |
| 34 | - usb_sndbulkpipe(aru->udev, AR9170_EP_CMD), | |
| 35 | + usb_sndintpipe(aru->udev, AR9170_EP_CMD), | |
| 36 | aru->common.cmdbuf, plen + 4, | |
| 37 | ar9170_usb_tx_urb_complete, NULL, 1); | |
| 38 |