give up class register

class/audio/usbd_audio.c

@@ -375,13 +375,6 @@ static void audio_notify_handler(uint8_t event, void *arg)
 
 void usbd_audio_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = audio_class_request_handler;
 #if CONFIG_USBDEV_AUDIO_VERSION < 0x0200
     intf->custom_handler = audio_custom_request_handler;

class/cdc/usbd_cdc.c

@@ -92,13 +92,6 @@ static void cdc_notify_handler(uint8_t event, void *arg)
 
 void usbd_cdc_add_acm_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = cdc_acm_class_request_handler;
     intf->custom_handler = NULL;
     intf->vendor_handler = NULL;

class/dfu/usbd_dfu.c

@@ -53,13 +53,6 @@ static void dfu_notify_handler(uint8_t event, void *arg)
 
 void usbd_dfu_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = dfu_class_request_handler;
     intf->custom_handler = NULL;
     intf->vendor_handler = NULL;

class/hid/usbd_hid.c

@@ -175,13 +175,6 @@ int usbd_hid_alloc(uint8_t intf)
 
 void usbd_hid_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = hid_class_request_handler;
     intf->custom_handler = hid_custom_request_handler;
     intf->vendor_handler = NULL;

class/hub/usbd_hub.c

@@ -74,13 +74,6 @@ static void hub_notify_handler(uint8_t event, void *arg)
 
 void usbd_hub_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = NULL;
     intf->custom_handler = hub_custom_request_handler;
     intf->vendor_handler = NULL;

class/msc/usbd_msc.c

@@ -911,8 +911,6 @@ static void usbd_msc_thread(void *argument)
 }
 #endif
 
-static usbd_class_t msc_class;
-
 static usbd_interface_t msc_intf = {
     .class_handler = msc_storage_class_request_handler,
     .vendor_handler = NULL,
@@ -921,10 +919,7 @@ static usbd_interface_t msc_intf = {
 
 void usbd_msc_class_init(uint8_t out_ep, uint8_t in_ep)
 {
-    msc_class.name = "usbd_msc";
+    usbd_class_add_interface(NULL, &msc_intf);
-
-    usbd_class_register(&msc_class);
-    usbd_class_add_interface(&msc_class, &msc_intf);
 
     mass_ep_data[0].ep_addr = out_ep;
     mass_ep_data[0].ep_cb = mass_storage_bulk_out;

class/printer/usbd_printer.c

@@ -48,13 +48,6 @@ static void printer_notify_handler(uint8_t event, void *arg)
 
 void usbd_printer_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = printer_class_request_handler;
     intf->custom_handler = NULL;
     intf->vendor_handler = NULL;

class/template/usbd_xxx.c

@@ -30,13 +30,6 @@ static void xxx_notify_handler(uint8_t event, void *arg)
 
 void usbd_xxx_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = xxx_class_request_handler;
     intf->custom_handler = NULL;
     intf->vendor_handler = NULL;

class/video/usbd_video.c

@@ -709,13 +709,6 @@ static void video_notify_handler(uint8_t event, void *arg)
 
 void usbd_video_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
-    static usbd_class_t *last_class = NULL;
-
-    if (last_class != devclass) {
-        last_class = devclass;
-        usbd_class_register(devclass);
-    }
-
     intf->class_handler = video_class_request_handler;
     intf->custom_handler = NULL;
     intf->vendor_handler = NULL;

core/usbd_core.c

@@ -51,7 +51,8 @@ USB_NOCACHE_RAM_SECTION struct usbd_core_cfg_priv {
 #endif
 } usbd_core_cfg;
 
-static usb_slist_t usbd_class_head = USB_SLIST_OBJECT_INIT(usbd_class_head);
+static usb_slist_t usbd_intf_head = USB_SLIST_OBJECT_INIT(usbd_intf_head);
+
 static struct usb_msosv1_descriptor *msosv1_desc;
 static struct usb_msosv2_descriptor *msosv2_desc;
 static struct usb_bos_descriptor *bos_desc;
@@ -582,34 +583,24 @@ static int usbd_class_request_handler(struct usb_setup_packet *setup, uint8_t **
 {
     usb_slist_t *i, *j;
     if ((setup->bmRequestType & USB_REQUEST_RECIPIENT_MASK) == USB_REQUEST_RECIPIENT_INTERFACE) {
-        usb_slist_for_each(i, &usbd_class_head)
+        usb_slist_for_each(i, &usbd_intf_head)
         {
-            usbd_class_t *devclass = usb_slist_entry(i, struct usbd_class, list);
+            usbd_interface_t *intf = usb_slist_entry(i, struct usbd_interface, list);
-
-            usb_slist_for_each(j, &devclass->intf_list)
-            {
-                usbd_interface_t *intf = usb_slist_entry(j, struct usbd_interface, list);
 
             if (intf->class_handler && (intf->intf_num == (setup->wIndex & 0xFF))) {
                 return intf->class_handler(setup, data, len);
             }
         }
-        }
     } else if ((setup->bmRequestType & USB_REQUEST_RECIPIENT_MASK) == USB_REQUEST_RECIPIENT_ENDPOINT) {
-        usb_slist_for_each(i, &usbd_class_head)
+        usb_slist_for_each(i, &usbd_intf_head)
         {
-            usbd_class_t *devclass = usb_slist_entry(i, struct usbd_class, list);
+            usbd_interface_t *intf = usb_slist_entry(i, struct usbd_interface, list);
-
-            usb_slist_for_each(j, &devclass->intf_list)
-            {
-                usbd_interface_t *intf = usb_slist_entry(j, struct usbd_interface, list);
 
             if (intf->custom_handler && (intf->intf_num == ((setup->wIndex >> 8) & 0xFF))) {
                 return intf->custom_handler(setup, data, len);
             }
         }
     }
-    }
     return -1;
 }
 
@@ -661,21 +652,16 @@ static int usbd_vendor_request_handler(struct usb_setup_packet *setup, uint8_t *
         }
     }
 
-    usb_slist_t *i, *j;
+    usb_slist_t *i;
 
-    usb_slist_for_each(i, &usbd_class_head)
+    usb_slist_for_each(i, &usbd_intf_head)
     {
-        usbd_class_t *devclass = usb_slist_entry(i, struct usbd_class, list);
+        usbd_interface_t *intf = usb_slist_entry(i, struct usbd_interface, list);
-
-        usb_slist_for_each(j, &devclass->intf_list)
-        {
-            usbd_interface_t *intf = usb_slist_entry(j, struct usbd_interface, list);
 
         if (intf->vendor_handler && !intf->vendor_handler(setup, data, len)) {
             return 0;
         }
     }
-    }
 
     return -1;
 }
@@ -695,20 +681,15 @@ static int usbd_custom_request_handler(struct usb_setup_packet *setup, uint8_t *
         return -1;
     }
 
-    usb_slist_t *i, *j;
+    usb_slist_t *i;
-    usb_slist_for_each(i, &usbd_class_head)
+    usb_slist_for_each(i, &usbd_intf_head)
     {
-        usbd_class_t *devclass = usb_slist_entry(i, struct usbd_class, list);
+        usbd_interface_t *intf = usb_slist_entry(i, struct usbd_interface, list);
-
-        usb_slist_for_each(j, &devclass->intf_list)
-        {
-            usbd_interface_t *intf = usb_slist_entry(j, struct usbd_interface, list);
 
         if (intf->custom_handler && (intf->intf_num == (setup->wIndex & 0xFF))) {
             return intf->custom_handler(setup, data, len);
         }
     }
-    }
 
     return -1;
 }
@@ -758,10 +739,9 @@ static bool usbd_setup_request_handler(struct usb_setup_packet *setup, uint8_t *
 static void usbd_class_event_notify_handler(uint8_t event, void *arg)
 {
     usb_slist_t *i;
-    usb_slist_for_each(i, &usbd_class_head)
+    usb_slist_for_each(i, &usbd_intf_head)
     {
-        usbd_class_t *devclass = usb_slist_entry(i, struct usbd_class, list);
+        usbd_interface_t *intf = usb_slist_entry(i, struct usbd_interface, list);
-        usbd_interface_t *intf = usb_slist_first_entry(&devclass->intf_list, struct usbd_interface, list);
 
         if (intf->notify_handler) {
             intf->notify_handler(event, arg);
@@ -974,15 +954,16 @@ void usbd_bos_desc_register(struct usb_bos_descriptor *desc)
 
 void usbd_class_register(usbd_class_t *devclass)
 {
-    usb_slist_add_tail(&usbd_class_head, &devclass->list);
+    // usb_slist_add_tail(&usbd_class_head, &devclass->list);
-    usb_slist_init(&devclass->intf_list);
+    // usb_slist_init(&devclass->intf_list);
 }
 
 void usbd_class_add_interface(usbd_class_t *devclass, usbd_interface_t *intf)
 {
     static uint8_t intf_offset = 0;
+
     intf->intf_num = intf_offset;
-    usb_slist_add_tail(&devclass->intf_list, &intf->list);
+    usb_slist_add_tail(&usbd_intf_head, &intf->list);
     usb_slist_init(&intf->ep_list);
     intf_offset++;
 }
@@ -1004,6 +985,5 @@ bool usb_device_is_configured(void)
 
 int usbd_initialize(void)
 {
-    usb_dc_init();
+    return usb_dc_init();
-    return 0;
 }