evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

usb: dwc3: gadget: free trb pool only from epnum 2

Browse source 
we never allocate a TRB pool for physical endpoints
0 and 1 so trying to free it (a invalid TRB pool pointer)
will lead us in a warning while removing dwc3.ko module.

In order to fix the situation, all we have to do is skip
dwc3_free_trb_pool() for physical endpoints 0 and 1 just
as we while deleting endpoints from the endpoints list.

Cc: stable@vger.kernel.org
Signed-off-by: George Cherian <george.cherian@ti.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
This commit is contained in:
George Cherian 2013-05-27 14:35:49 +05:30 • committed by Felipe Balbi
parent 022d0547aa
commit 5bf8fae33d
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
drivers/usb/dwc3/gadget.c
View file

@ -1706,11 +1706,19 @@ static void dwc3_gadget_free_endpoints(struct dwc3 *dwc)
dep = dwc->eps[epnum]; dep = dwc->eps[epnum];
if (!dep) if (!dep)
continue; continue;
/*
dwc3_free_trb_pool(dep); * Physical endpoints 0 and 1 are special; they form the
* bi-directional USB endpoint 0.
if (epnum != 0 && epnum != 1) *
* For those two physical endpoints, we don't allocate a TRB
* pool nor do we add them the endpoints list. Due to that, we
* shouldn't do these two operations otherwise we would end up
* with all sorts of bugs when removing dwc3.ko.
*/
if (epnum != 0 && epnum != 1) {
dwc3_free_trb_pool(dep);
list_del(&dep->endpoint.ep_list); list_del(&dep->endpoint.ep_list);
}
kfree(dep); kfree(dep);
} }