flatten/simplify completion event logic

Signed-off-by: Josuah Demangeon <[email protected]>

Author: Josuah Demangeon

drivers/usb/uhc/uhc_dwc2.c

@@ -819,8 +819,10 @@ static void uhc_dwc2_isr_chan_handler(const struct device *dev, struct uhc_dwc2_
 		if (chan->halt_requested) {
 			chan->halt_requested = 0;
 			chan_event |= BIT(DWC2_CHAN_EVENT_HALT_REQ);
-		} else {
+		} else if (uhc_dwc2_buffer_is_done(chan)) {
 			chan_event |= BIT(DWC2_CHAN_EVENT_CPLT);
+		} else {
+			uhc_dwc2_buffer_exec_proceed(dev, chan);
 		}
 
 	} else if (hcint & USB_DWC2_HCINT_XFERCOMPL) {
@@ -840,12 +842,6 @@ static void uhc_dwc2_isr_chan_handler(const struct device *dev, struct uhc_dwc2_
 		__ASSERT(false, "Unknown channel interrupt, HCINT=%08Xh", hcint);
 	}
 
-	if ((chan_event & BIT(DWC2_CHAN_EVENT_CPLT)) && !uhc_dwc2_buffer_is_done(chan)) {
-		/* No completion event until the buffer is complete software-side too */
-		chan_event &= ~BIT(DWC2_CHAN_EVENT_CPLT);
-		uhc_dwc2_buffer_exec_proceed(dev, chan);
-	}
-
 	if (chan_event != 0) {
 		atomic_or(&chan->event, chan_event);
 		k_event_set(&priv->event, BIT(UHC_DWC2_EVENT_CHAN0 + chan->chan_idx));