drivers: uhc: use correct endpoint type and interval

AidenHu · AidenHu · commit 6a9e38de35cb · 2025-11-27T10:51:44.000+08:00
For usb xfer, set endpoint type and interval by the
selected endpoint desc.

Signed-off-by: Aiden Hu &lt;weiwei.hu@nxp.com&gt;
diff --git a/drivers/usb/uhc/uhc_common.c b/drivers/usb/uhc/uhc_common.c
@@ -111,6 +111,8 @@ struct uhc_transfer *uhc_xfer_alloc(const struct device *dev,
 	}
 
 	if (ep_idx == 0) {
+		interval = 0;
+		type = USB_EP_TYPE_CONTROL;
 		mps = udev->dev_desc.bMaxPacketSize0;
 	} else {
 		struct usb_ep_descriptor *ep_desc;
diff --git a/drivers/usb/uhc/uhc_mcux_common.c b/drivers/usb/uhc/uhc_mcux_common.c
@@ -244,8 +244,7 @@ static usb_host_pipe_handle uhc_mcux_check_hal_ep(const struct device *dev,
 		}
 	}
 
-	/* TODO: need to check endpoint type too */
-	if (mcux_ep != NULL &&
+	if (mcux_ep != NULL && mcux_ep->pipeType == xfer->type &&
 	    (mcux_ep->maxPacketSize != xfer->mps ||
 	     mcux_ep->interval != xfer->interval)) {
 		/* re-initialize the ep */

Original file line number	Diff line number	Diff line change
`@@ -111,6 +111,8 @@ struct uhc_transfer uhc_xfer_alloc(const struct device dev,`
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`if (ep_idx == 0) {`
	`114`	`+ interval = 0;`
	`115`	`+ type = USB_EP_TYPE_CONTROL;`
`114`	`116`	`mps = udev->dev_desc.bMaxPacketSize0;`
`115`	`117`	`} else {`
`116`	`118`	`struct usb_ep_descriptor *ep_desc;`
Original file line number	Diff line number	Diff line change
`@@ -244,8 +244,7 @@ static usb_host_pipe_handle uhc_mcux_check_hal_ep(const struct device *dev,`
`244`	`244`	`}`
`245`	`245`	`}`
`246`	`246`
`247`		`- /* TODO: need to check endpoint type too */`
`248`		`- if (mcux_ep != NULL &&`
	`247`	`+ if (mcux_ep != NULL && mcux_ep->pipeType == xfer->type &&`
`249`	`248`	`(mcux_ep->maxPacketSize != xfer->mps \|\|`
`250`	`249`	`mcux_ep->interval != xfer->interval)) {`
`251`	`250`	`/* re-initialize the ep */`