drivers: usb: uhc: update usb host driver

1. Add interval for usb xfer.
2. Set right mps for during pipe init.
3. Set right direction for non-zero endpoint.
4. Check transferBuffer during init transfer.

Signed-off-by: Aiden Hu <[email protected]>

Author: AidenHu

drivers/usb/uhc/uhc_common.c

@@ -101,6 +101,7 @@ struct uhc_transfer *uhc_xfer_alloc(const struct device *dev,
 	const struct uhc_api *api = dev->api;
 	struct uhc_transfer *xfer = NULL;
 	uint16_t mps;
+	uint8_t interval;
 
 	api->lock(dev);
 
@@ -125,6 +126,7 @@ struct uhc_transfer *uhc_xfer_alloc(const struct device *dev,
 		}
 
 		mps = ep_desc->wMaxPacketSize;
+		interval = ep_desc->bInterval;
 	}
 
 	LOG_DBG("Allocate xfer, ep 0x%02x mps %u cb %p", ep, mps, cb);
@@ -137,6 +139,7 @@ struct uhc_transfer *uhc_xfer_alloc(const struct device *dev,
 	memset(xfer, 0, sizeof(struct uhc_transfer));
 	xfer->ep = ep;
 	xfer->mps = mps;
+	xfer->interval = interval;
 	xfer->udev = udev;
 	xfer->cb = cb;
 	xfer->priv = cb_priv;

drivers/usb/uhc/uhc_mcux_common.c

@@ -280,20 +280,21 @@ usb_host_pipe_t *uhc_mcux_init_hal_ep(const struct device *dev, struct uhc_trans
 	/* USB_HostHelperGetPeripheralInformation uses this value as first parameter */
 	pipe_init.devInstance = xfer->udev;
 	pipe_init.nakCount = USB_HOST_CONFIG_MAX_NAK;
-	pipe_init.maxPacketSize = xfer->mps;
+	pipe_init.maxPacketSize = USB_MPS_EP_SIZE(xfer->mps);
 	pipe_init.endpointAddress = USB_EP_GET_IDX(xfer->ep);
 	pipe_init.direction = USB_EP_GET_IDX(xfer->ep) == 0 ? USB_OUT :
 			      USB_EP_GET_DIR(xfer->ep) ? USB_IN : USB_OUT;
 	/* Current Zephyr Host stack is experimental, the endpoint's interval,
 	 * 'number per uframe' and the endpoint type cannot be got yet.
 	 */
 	pipe_init.numberPerUframe = 0; /* TODO: need right way to implement it. */
+	pipe_init.numberPerUframe = USB_MPS_ADDITIONAL_TRANSACTIONS(xfer->mps);
 	pipe_init.interval = xfer->interval;
 	/* TODO: need right way to implement it. */
 	if (pipe_init.endpointAddress == 0) {
 		pipe_init.pipeType = USB_ENDPOINT_CONTROL;
 	} else {
-		pipe_init.pipeType = USB_ENDPOINT_BULK;
+		pipe_init.pipeType = USB_ENDPOINT_ISOCHRONOUS;
 	}
 
 	status = priv->mcux_if->controllerOpenPipe(priv->mcux_host.controllerHandle,
@@ -353,6 +354,10 @@ int uhc_mcux_hal_init_transfer_common(const struct device *dev, usb_host_transfe
 					       ? USB_IN
 					       : USB_OUT;
 	}
+	else
+	{
+		mcux_xfer->direction = USB_EP_DIR_IS_IN(xfer->ep) ? USB_IN : USB_OUT;
+	}
 
 	return 0;
 }

drivers/usb/uhc/uhc_mcux_ehci.c

@@ -202,6 +202,10 @@ static usb_host_transfer_t *uhc_mcux_hal_init_transfer(const struct device *dev,
 	memcpy(mcux_xfer->setupPacket, xfer->setup_pkt, 8u);
 	if (xfer->buf != NULL) {
 		mcux_xfer->transferBuffer = uhc_mcux_nocache_alloc(mcux_xfer->transferLength);
+		if (mcux_xfer->transferBuffer == NULL)
+		{
+			LOG_ERR("Failed to allocate non-cached transfer buffer");
+		}
 	}
 #endif