drivers: usb: udc_dwc2: Synchronize thread with UDC API

UDC API handlers and DWC2 driver thread share not only software constructs, but also the underlying hardware. Ensure that any UDC API call is not preempted by DWC2 driver thread (and vice versa) by acquiring the lock in thread handler. Signed-off-by: Tomasz Moń <[email protected]>
frkv · Sep 25, 2024 · 5fc0405 · 5fc0405
1 parent c2f2d8c
commit 5fc0405
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/drivers/usb/udc/udc_dwc2.c b/drivers/usb/udc/udc_dwc2.c
@@ -2722,6 +2722,8 @@ static ALWAYS_INLINE void dwc2_thread_handler(void *const arg)
 	 */
 	evt = k_event_wait(&priv->drv_evt, UINT32_MAX, false, K_FOREVER);
 
+	udc_lock_internal(dev, K_FOREVER);
+
 	if (evt & BIT(DWC2_DRV_EVT_XFER)) {
 		k_event_clear(&priv->drv_evt, BIT(DWC2_DRV_EVT_XFER));
 
@@ -2794,6 +2796,8 @@ static ALWAYS_INLINE void dwc2_thread_handler(void *const arg)
 
 		config->irq_enable_func(dev);
 	}
+
+	udc_unlock_internal(dev);
 }
 
 static const struct udc_api udc_dwc2_api = {