usb: dwc3: gadget: only unmap requests from DMA if mapped
authorJack Pham <jackp@codeaurora.org>
Thu, 29 Jun 2017 07:53:31 +0000 (00:53 -0700)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Mon, 17 Jul 2017 08:01:27 +0000 (11:01 +0300)
commit4a71fcb8ac5f94c07bf47a43b13258a52e4fe3ad
treeb270a4868052eb0bd271a56001afb09aad3ff234
parenta9ef5c47d065e3c8a6eb0e5911e95809811e6c6b
usb: dwc3: gadget: only unmap requests from DMA if mapped

A recent optimization was made so that a request put on the
pending_list wouldn't get mapped for DMA until just before
preparing a TRB for it. However, this poses a problem in case
the request is dequeued or the endpoint is disabled before the
mapping is done as that would lead to dwc3_gadget_giveback()
unconditionally calling usb_gadget_unmap_request_for_dev() with
an invalid request->dma handle. Depending on the platform's DMA
implementation the unmap operation could result in a panic.

Since we know a successful mapping is a prerequisite for getting
a TRB, the unmap can be conditionally called only when req->trb
is non-NULL.

Fixes: cdb55b39fab8 ("usb: dwc3: gadget: lazily map requests for DMA")
Signed-off-by: Jack Pham <jackp@codeaurora.org>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/dwc3/gadget.c