diff options
| author |   Thinh Nguyen <Thinh.Nguyen@synopsys.com> | 2020-05-05 19:47:03 -0700 |
|---|---|---|
| committer |   Felipe Balbi <balbi@kernel.org> | 2020-05-25 11:09:42 +0300 |
| commit | aefe3d232b6629cb0ac92cc3d4238a5133dd9fec (patch) | |
| tree | db858fe41780d3d38bb72a3c6c718059d11c037a /drivers/usb | |
| parent | usb: dwc3: gadget: Wait for transfer completion (diff) | |
| download | wireguard-linux-aefe3d232b6629cb0ac92cc3d4238a5133dd9fec.tar.xz wireguard-linux-aefe3d232b6629cb0ac92cc3d4238a5133dd9fec.zip | |
usb: dwc3: gadget: Don't prepare beyond a transfer
Don't prepare TRBs beyond a transfer. In DWC_usb32, its transfer burst
capability may try to read and use TRBs beyond the active transfer. For
other controllers, they don't process the next transfer TRBs until the
current transfer is completed. Explicitly prevent preparing TRBs ahead
for all controllers.
Signed-off-by: Thinh Nguyen <thinhn@synopsys.com>
Signed-off-by: Felipe Balbi <balbi@kernel.org>
Diffstat (limited to 'drivers/usb')
| -rw-r--r-- | drivers/usb/dwc3/gadget.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index 97c6a5785725..07824b670440 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -1231,6 +1231,14 @@ static void dwc3_prepare_trbs(struct dwc3_ep *dep) if (!dwc3_calc_trbs_left(dep)) return; + + /* + * Don't prepare beyond a transfer. In DWC_usb32, its transfer + * burst capability may try to read and use TRBs beyond the + * active transfer instead of stopping. + */ + if (dep->stream_capable && req->request.is_last) + return; } } |