aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/ath
diff options
context:
space:
mode:
authorValentin Longchamp <valentin.longchamp@epfl.ch>2010-03-26 06:44:33 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2010-04-26 10:47:48 -0400
commite0db64375f23e12bbb2c43f89abc099bb61fc2f1 (patch)
tree1d1ac9e279eb2a68759869c6139a8ffb9de20826 /drivers/net/wireless/ath
parent2596e91190508f987871ad58589adb6016668b87 (diff)
setup correct int pipe type in ar9170_usb_exec_cmd
commit 2d20c72c021d96f8b9230396c8e3782f204214ec upstream. An int urb is constructed but we fill it in with a bulk pipe type. Commit f661c6f8c67bd55e93348f160d590ff9edf08904 implemented a pipe type check when CONFIG_USB_DEBUG is enabled. The check failed for all the ar9170 usb transfers and the driver could not configure the wifi dongle. This went unnoticed until now because most people don't have CONFIG_USB_DEBUG enabled. Signed-off-by: Valentin Longchamp <valentin.longchamp@epfl.ch> Acked-by: Christian Lamparter <chunkeey@googlemail.com> Signed-off-by: John W. Linville <linville@tuxdriver.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/net/wireless/ath')
-rw-r--r--drivers/net/wireless/ath/ar9170/usb.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ar9170/usb.c b/drivers/net/wireless/ath/ar9170/usb.c
index e0799d924057..0387658d1e14 100644
--- a/drivers/net/wireless/ath/ar9170/usb.c
+++ b/drivers/net/wireless/ath/ar9170/usb.c
@@ -414,7 +414,7 @@ static int ar9170_usb_exec_cmd(struct ar9170 *ar, enum ar9170_cmd cmd,
414 spin_unlock_irqrestore(&aru->common.cmdlock, flags); 414 spin_unlock_irqrestore(&aru->common.cmdlock, flags);
415 415
416 usb_fill_int_urb(urb, aru->udev, 416 usb_fill_int_urb(urb, aru->udev,
417 usb_sndbulkpipe(aru->udev, AR9170_EP_CMD), 417 usb_sndintpipe(aru->udev, AR9170_EP_CMD),
418 aru->common.cmdbuf, plen + 4, 418 aru->common.cmdbuf, plen + 4,
419 ar9170_usb_tx_urb_complete, NULL, 1); 419 ar9170_usb_tx_urb_complete, NULL, 1);
420 420