diff options
| author |   Anton Vorontsov <avorontsov@ru.mvista.com> | 2008-12-25 17:15:14 +0300 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@suse.de> | 2009-02-09 11:19:47 -0800 |
| commit | af3ddbd76304f9f602c970f9b09a0c9d8cf8336c (patch) | |
| tree | ffd737b63503469d328790577e5e27c657e327d2 /drivers/usb/gadget/fsl_qe_udc.c | |
| parent | USB: fsl_qe_udc: Fix muram corruption by disabled endpoints (diff) | |
| download | linux-dev-af3ddbd76304f9f602c970f9b09a0c9d8cf8336c.tar.xz linux-dev-af3ddbd76304f9f602c970f9b09a0c9d8cf8336c.zip | |
USB: fsl_qe_udc: Fix stalled TX requests bug
While disabling an endpoint the driver nuking any pending requests,
thus completing them with -ESHUTDOWN status. But the driver doesn't
clear the tx_req, which means that a next TX request (after
ep_enable), might get stalled, since the driver won't queue the new
reqests.
This patch fixes a bug I'm observing with ethernet gadget while
playing with ifconfig usb0 up/down (the up/down sequence disables
and enables `in' and `out' endpoints).
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
Acked-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to '')
| -rw-r--r-- | drivers/usb/gadget/fsl_qe_udc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/usb/gadget/fsl_qe_udc.c b/drivers/usb/gadget/fsl_qe_udc.c index 32f415ebd3df..d701bf4698d2 100644 --- a/drivers/usb/gadget/fsl_qe_udc.c +++ b/drivers/usb/gadget/fsl_qe_udc.c @@ -1622,6 +1622,7 @@ static int qe_ep_disable(struct usb_ep *_ep) nuke(ep, -ESHUTDOWN); ep->desc = NULL; ep->stopped = 1; + ep->tx_req = NULL; qe_ep_reset(udc, ep->epnum); spin_unlock_irqrestore(&udc->lock, flags); |