subsys: usb: host: remove root member for usbh_context

add usbh_device_get_root and usbh_device_is_root
function to check root device

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

Author: AidenHu

include/zephyr/usb/usbh.h

@@ -45,8 +45,6 @@ struct usbh_context {
 	const struct device *dev;
 	/** USB device list */
 	sys_dlist_t udevs;
-	/** USB root device */
-	struct usb_device *root;
 	/** Allocated device addresses bit array */
 	struct sys_bitarray *addr_ba;
 };

subsys/usb/host/usbh_core.c

@@ -46,6 +46,7 @@ static int usbh_event_carrier(const struct device *dev,
 static void dev_connected_handler(struct usbh_context *const ctx,
 				  const struct uhc_event *const event)
 {
+	int err;
 	struct usb_device *udev;
 
 	LOG_DBG("Device connected event");
@@ -66,27 +67,28 @@ static void dev_connected_handler(struct usbh_context *const ctx,
 
 	k_mutex_lock(&ctx->mutex, K_FOREVER);
 	sys_dlist_append(&ctx->udevs, &udev->node);
-
-	if (ctx->root == NULL) {
-		ctx->root = udev;
-	}
 	k_mutex_unlock(&ctx->mutex);
 
-	if (usbh_device_init(udev)) {
+	err = usbh_device_init(udev);
+	if (ret != 0) {
 		LOG_ERR("Failed to reset new USB device");
 		sys_dlist_remove(&udev->node);
 	}
 }
 
 static void dev_removed_handler(struct usbh_context *const ctx)
 {
-	if (ctx->root != NULL) {
-		usbh_device_free(ctx->root);
-		ctx->root = NULL;
+	struct usb_device *udev = NULL;
+
+	udev = usbh_device_get_root(ctx);
+
+	if (NULL != udev) {
+		usbh_device_free(udev);
 		LOG_DBG("Device removed");
 	} else {
 		LOG_DBG("Spurious device removed event");
 	}
+	/* TODO: handle remove for all of classes for the unattached device */
 }
 
 static int discard_ep_request(struct usbh_context *const ctx,

subsys/usb/host/usbh_device.c

@@ -447,6 +447,26 @@ int usbh_device_set_configuration(struct usb_device *const udev, const uint8_t n
 	return err;
 }
 
+struct usb_device *usbh_device_get_root(struct usbh_context *const ctx)
+{
+	sys_dnode_t *node;
+
+	node = sys_dlist_peek_head(&ctx->udevs);
+	if (node == NULL) {
+		/* No devices in the list */
+		return NULL;
+	}
+
+	/* Get the usb_device structure from the node */
+	return CONTAINER_OF(node, struct usb_device, node);
+}
+
+bool usbh_device_is_root(struct usbh_context *const ctx,
+			struct usb_device *const udev)
+{
+	return sys_dlist_peek_head(&ctx->udevs) == &udev->node;
+}
+
 int usbh_device_init(struct usb_device *const udev)
 {
 	struct usbh_context *const uhs_ctx = udev->ctx;

subsys/usb/host/usbh_device.h

@@ -36,6 +36,13 @@ int usbh_device_interface_set(struct usb_device *const udev,
 			      const uint8_t iface, const uint8_t alt,
 			      const bool dry);
 
+/* Get root USB device */
+struct usb_device *usbh_device_get_root(struct usbh_context *const ctx);
+
+/* Check if USB device is root */
+bool usbh_device_is_root(struct usbh_context *const ctx,
+			struct usb_device *const udev);
+
 /* Wrappers around to avoid glue UHC calls. */
 static inline struct uhc_transfer *usbh_xfer_alloc(struct usb_device *udev,
 						   const uint8_t ep,