remove class code in pipe open()

4 files changed, 15 insertions, 16 deletions

diff --git a/tinyusb/device/dcd.h b/tinyusb/device/dcd.h
index 2fa8f1b86..d9b339740 100644
--- a/tinyusb/device/dcd.h
+++ b/tinyusb/device/dcd.h
@@ -60,21 +60,18 @@ typedef enum
 // TODO move Hal

 typedef struct {

   uint8_t coreid;

-  uint8_t reserved; // TODO redundant, cannot be control as control uses separated API

-  uint8_t index;

-  uint8_t class_code;

+  uint8_t index; // must be zero to indicate control

 } endpoint_handle_t;

 

-static inline bool endpointhandle_is_valid(endpoint_handle_t edpt_hdl) ATTR_CONST ATTR_ALWAYS_INLINE ATTR_WARN_UNUSED_RESULT;

 static inline bool endpointhandle_is_valid(endpoint_handle_t edpt_hdl)

 {

-  return (edpt_hdl.class_code != 0);

+  // Control does not use this to check

+  return edpt_hdl.index != 0;

 }

 

-static inline bool endpointhandle_is_equal(endpoint_handle_t x, endpoint_handle_t y) ATTR_CONST ATTR_ALWAYS_INLINE ATTR_WARN_UNUSED_RESULT;

 static inline bool endpointhandle_is_equal(endpoint_handle_t x, endpoint_handle_t y)

 {

-  return (x.coreid == y.coreid) && (x.index == y.index) && (x.class_code == y.class_code);

+  return (x.coreid == y.coreid) && (x.index == y.index);

 }

 

 

@@ -95,7 +92,7 @@ void hal_dcd_setup_received(uint8_t coreid, uint8_t const* p_request);
 bool hal_dcd_control_xfer(uint8_t coreid, tusb_direction_t dir, uint8_t * p_buffer, uint16_t length, bool int_on_complete);

 void hal_dcd_control_stall(uint8_t coreid);

 

-endpoint_handle_t dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc, uint8_t class_code);

+endpoint_handle_t hal_dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc);

 tusb_error_t dcd_pipe_queue_xfer(endpoint_handle_t edpt_hdl, uint8_t * buffer, uint16_t total_bytes); // only queue, not transferring yet

 tusb_error_t dcd_pipe_xfer(endpoint_handle_t edpt_hdl, uint8_t * buffer, uint16_t total_bytes, bool int_on_complete);

 tusb_error_t dcd_pipe_stall(endpoint_handle_t edpt_hdl);

diff --git a/tinyusb/device/dcd_lpc175x_6x.c b/tinyusb/device/dcd_lpc175x_6x.c
index dbd25106e..ac2e40172 100644
--- a/tinyusb/device/dcd_lpc175x_6x.c
+++ b/tinyusb/device/dcd_lpc175x_6x.c
@@ -412,7 +412,7 @@ bool hal_dcd_control_xfer(uint8_t coreid, tusb_direction_t dir, uint8_t * p_buff
 //--------------------------------------------------------------------+

 // BULK/INTERRUPT/ISO PIPE API

 //--------------------------------------------------------------------+

-endpoint_handle_t dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc, uint8_t class_code)

+endpoint_handle_t hal_dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc, uint8_t class_code)

 {

   (void) coreid;

 

diff --git a/tinyusb/device/dcd_lpc_11uxx_13uxx.c b/tinyusb/device/dcd_lpc_11uxx_13uxx.c
index 66fabec2c..b0e73ac98 100644
--- a/tinyusb/device/dcd_lpc_11uxx_13uxx.c
+++ b/tinyusb/device/dcd_lpc_11uxx_13uxx.c
@@ -460,7 +460,7 @@ tusb_error_t dcd_pipe_clear_stall(uint8_t coreid, uint8_t edpt_addr)
   return TUSB_ERROR_NONE;

 }

 

-endpoint_handle_t dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc, uint8_t class_code)

+endpoint_handle_t hal_dcd_pipe_open(uint8_t coreid, tusb_descriptor_endpoint_t const * p_endpoint_desc, uint8_t class_code)

 {

   (void) coreid;

   endpoint_handle_t const null_handle = { 0 };

diff --git a/tinyusb/device/usbd.c b/tinyusb/device/usbd.c
index 4428948fe..3a3bb6220 100644
--- a/tinyusb/device/usbd.c
+++ b/tinyusb/device/usbd.c
@@ -221,11 +221,11 @@ static tusb_error_t usbd_body_subtask(void)
     OSAL_SUBTASK_INVOKED_AND_WAIT( usbd_control_request_subtask(event.coreid, &event.setup_received), error );

   }else if (USBD_EVENTID_XFER_DONE == event.event_id)

   {

-    uint8_t class_index;

-    class_index = std_class_code_to_index( event.xfer_done.edpt_hdl.class_code );

-

-    SUBTASK_ASSERT(usbd_class_drivers[class_index].xfer_cb != NULL);

-    usbd_class_drivers[class_index].xfer_cb( event.xfer_done.edpt_hdl, (tusb_event_t) event.sub_event_id, event.xfer_done.xferred_byte);

+    // Call class handling function, Class that endpoint not belong to should check and return

+    for (uint8_t class_code = TUSB_CLASS_AUDIO; class_code < USBD_CLASS_DRIVER_COUNT; class_code++)

+    {

+      if ( usbd_class_drivers[class_code].xfer_cb ) usbd_class_drivers[class_code].xfer_cb( event.xfer_done.edpt_hdl, (tusb_event_t) event.sub_event_id, event.xfer_done.xferred_byte);

+    }

   }else

   {

     SUBTASK_ASSERT(false);

@@ -445,8 +445,10 @@ void hal_dcd_setup_received(uint8_t coreid, uint8_t const* p_request)
 

 void usbd_xfer_isr(endpoint_handle_t edpt_hdl, tusb_event_t event, uint32_t xferred_bytes)

 {

-  if (edpt_hdl.class_code == 0 ) // Control Transfer

+//  if (edpt_hdl.class_code == 0 )

+  if (edpt_hdl.index == 0 )

   {

+    // Control Transfer

     osal_semaphore_post( usbd_control_xfer_sem_hdl );

   }else

   {