From: Christian Lamparter Date: Mon, 27 Sep 2010 23:53:42 +0000 (+0200) Subject: carl9170: interrupt urbs must not set URB_ZERO_PACKET X-Git-Tag: upstream/snapshot3+hdmi~12689^2~265^2^2~3 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dc4769f0bb949e312ad8d9b652047ff6709978c2;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git carl9170: interrupt urbs must not set URB_ZERO_PACKET This patch fixes a bug in the driver which was exposed by CONFIG_USB_DEBUG: "usb 1-1.6.3: BOGUS urb flags, 40 --> 0" The transfer flag "URB_ZERO_PACKET" is only valid for bulk urbs. Reported-by: André Erdmann Signed-off-by: Christian Lamparter Signed-off-by: John W. Linville --- diff --git a/drivers/net/wireless/ath/carl9170/usb.c b/drivers/net/wireless/ath/carl9170/usb.c index eb789a9..c7f6193 100644 --- a/drivers/net/wireless/ath/carl9170/usb.c +++ b/drivers/net/wireless/ath/carl9170/usb.c @@ -606,8 +606,6 @@ int __carl9170_exec_cmd(struct ar9170 *ar, struct carl9170_cmd *cmd, AR9170_USB_EP_CMD), cmd, cmd->hdr.len + 4, carl9170_usb_cmd_complete, ar, 1); - urb->transfer_flags |= URB_ZERO_PACKET; - if (free_buf) urb->transfer_flags |= URB_FREE_BUFFER;