A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-08 04:42:04 -04:00
Code Issues Projects Releases Packages Wiki Activity

usb: dwc3: gadget: Check controller status with endpoint enable/disable

Browse source 
Some USB function driver (e.g. mass storage) performs USB endpoint enable
and disable functionality from worker thread context instead of composite/
controller work context. This may result into endpoint enable/disable
racing against USB controller low power mode functionality with USB cable
disconnect. This race may result into unclock register access. Hence fix
this issue by checking USB controller status and perform resume/autosuspend
with endpoint disable whereas return -EINVAL with endpoint enable if USB
controller is suspended (in low power mode).

Change-Id: I76cd75fc90b52ceefe099f48be6c880c04136301
Signed-off-by: Mayank Rana <mrana@codeaurora.org>
This commit is contained in:
Mayank Rana 2020-07-23 12:17:30 -07:00
commit ad8d260c33
1 changed files with 9 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -914,6 +914,12 @@ static int dwc3_gadget_ep_enable(struct usb_ep *ep,
dep->name))
return 0;
if (pm_runtime_suspended(dwc->sysdev)) {
dev_err(dwc->dev, "fail ep_enable %s device is into LPM\n",
dep->name);
return -EINVAL;
}
spin_lock_irqsave(&dwc->lock, flags);
ret = __dwc3_gadget_ep_enable(dep, DWC3_DEPCFG_ACTION_INIT);
dbg_event(dep->number, "ENABLE", ret);
@ -942,10 +948,13 @@ static int dwc3_gadget_ep_disable(struct usb_ep *ep)
dep->name))
return 0;
pm_runtime_get_sync(dwc->dev);
spin_lock_irqsave(&dwc->lock, flags);
ret = __dwc3_gadget_ep_disable(dep);
dbg_event(dep->number, "DISABLE", ret);
spin_unlock_irqrestore(&dwc->lock, flags);
pm_runtime_mark_last_busy(dwc->dev);
pm_runtime_put_sync_autosuspend(dwc->dev);
return ret;
}