clean up virtual com example

2 files changed, 22 insertions, 99 deletions

diff --git a/tinyusb/class/cdc/cdc_device.c b/tinyusb/class/cdc/cdc_device.c
index 0b83457b5..9ca71bddb 100644
--- a/tinyusb/class/cdc/cdc_device.c
+++ b/tinyusb/class/cdc/cdc_device.c
@@ -75,38 +75,43 @@ FIFO_DEF(_tx_ff, CFG_TUD_CDC_BUFSIZE, uint8_t, true);
 //--------------------------------------------------------------------+

 STATIC_VAR cdcd_data_t cdcd_data[CONTROLLER_DEVICE_NUMBER];

 

-static tusb_error_t cdcd_xfer(uint8_t coreid,  cdc_pipeid_t pipeid, void * p_buffer, uint32_t length, bool is_notify)

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

+// APPLICATION API

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

+bool tud_cdc_connected(uint8_t coreid)

 {

-  ASSERT(tud_mounted(coreid), TUSB_ERROR_USBD_DEVICE_NOT_CONFIGURED);

-

-  cdcd_data_t* p_cdc = &cdcd_data[coreid];

+  return cdcd_data[coreid].connected;

+}

 

-  ASSERT_FALSE ( dcd_pipe_is_busy(p_cdc->edpt_hdl[pipeid]), TUSB_ERROR_INTERFACE_IS_BUSY);

-  ASSERT_STATUS( hal_dcd_pipe_xfer(p_cdc->edpt_hdl[pipeid], p_buffer, length, is_notify) );

+uint32_t tud_cdc_available(uint8_t coreid)

+{

+  return fifo_count(&_rx_ff);

+}

 

-  return TUSB_ERROR_NONE;

+int tud_cdc_read_char(uint8_t coreid)

+{

+  uint8_t ch;

+  return fifo_read(&_rx_ff, &ch) ? ch : (-1);

 }

 

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

-// APPLICATION API (Parameters requires validation)

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

-bool tud_cdc_busy(uint8_t coreid, cdc_pipeid_t pipeid)

+uint32_t tud_cdc_read(uint8_t coreid, void* buffer, uint32_t bufsize)

 {

-  return dcd_pipe_is_busy( cdcd_data[coreid].edpt_hdl[pipeid] );

+  return fifo_read_n(&_rx_ff, buffer, bufsize);

 }

 

-tusb_error_t tud_cdc_receive(uint8_t coreid, void * p_buffer, uint32_t length, bool is_notify)

+uint32_t tud_cdc_write_char(uint8_t coreid, char ch)

 {

-  return cdcd_xfer(coreid, CDC_PIPE_DATA_OUT, p_buffer, length, is_notify);

+  return fifo_write(&_tx_ff, &ch);

 }

 

-tusb_error_t tud_cdc_send(uint8_t coreid, void * p_data, uint32_t length, bool is_notify)

+uint32_t tud_cdc_write(uint8_t coreid, void const* buffer, uint32_t bufsize)

 {

-  return cdcd_xfer(coreid, CDC_PIPE_DATA_IN, p_data, length, is_notify);

+  return fifo_write_n(&_tx_ff, buffer, bufsize);

 }

 

+

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

-// USBD-CLASS API

+// USBD Driver API

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

 void cdcd_init(void)

 {

@@ -282,36 +287,4 @@ void cdcd_sof(uint8_t coreid)
   }

 }

 

-bool tud_cdc_connected(uint8_t coreid)

-{

-  return cdcd_data[coreid].connected;

-}

-

-uint32_t tud_cdc_available(uint8_t coreid)

-{

-  return fifo_count(&_rx_ff);

-}

-

-int tud_cdc_read_char(uint8_t coreid)

-{

-  uint8_t ch;

-  return fifo_read(&_rx_ff, &ch) ? ch : (-1);

-}

-

-uint32_t tud_cdc_read(uint8_t coreid, void* buffer, uint32_t bufsize)

-{

-  return fifo_read_n(&_rx_ff, buffer, bufsize);

-}

-

-uint32_t tud_cdc_write_char(uint8_t coreid, char ch)

-{

-  return fifo_write(&_tx_ff, &ch);

-}

-

-uint32_t tud_cdc_write(uint8_t coreid, void const* buffer, uint32_t bufsize)

-{

-  return fifo_write_n(&_tx_ff, buffer, bufsize);

-}

-

-

 #endif

diff --git a/tinyusb/class/cdc/cdc_device.h b/tinyusb/class/cdc/cdc_device.h
index d842b8022..97b15e82b 100644
--- a/tinyusb/class/cdc/cdc_device.h
+++ b/tinyusb/class/cdc/cdc_device.h
@@ -55,46 +55,6 @@
 //--------------------------------------------------------------------+

 // APPLICATION API

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

-/** \brief      Check if the interface is currently busy or not

- * \param[in]   coreid USB Controller ID

- * \retval      true if the interface is busy meaning the stack is still transferring/waiting data from/to host

- * \retval      false if the interface is not busy meaning the stack successfully transferred data from/to host

- * \note        This function is primarily used for polling/waiting result after \ref tusbd_hid_keyboard_send.

- */

-bool tud_cdc_busy(uint8_t coreid, cdc_pipeid_t pipeid)  ATTR_PURE ATTR_WARN_UNUSED_RESULT;

-

-/** \brief        Submit USB transfer

- * \param[in]		  coreid USB Controller ID

- * \param[in]     p_data  buffer containing data from application. Must be accessible by USB controller (see \ref TUSB_CFG_ATTR_USBRAM)

- * \param[in]     length number of bytes in \a p_data.

- * \param[in]     is_notify indicates whether the hardware completion (data transferred through USB bus) will be notified

- *                to Application (via \ref tusbd_cdc_xfer_cb)

- * \returns       \ref tusb_error_t type to indicate success or error condition.

- * \retval        TUSB_ERROR_NONE on success

- * \retval        TUSB_ERROR_INTERFACE_IS_BUSY if the interface is busy transferring previous data.

- * \retval        TUSB_ERROR_DEVICE_NOT_READY if device is not yet configured (by SET CONFIGURED request)

- * \retval        TUSB_ERROR_INVALID_PARA if input parameters are not correct

- * \note          This function is non-blocking and returns immediately. Data will be transferred when USB Host work with this interface.

- *                The result of usb transfer will be reported by the interface's callback function if \a is_notify is true

- */

-tusb_error_t tud_cdc_send(uint8_t coreid, void * p_data, uint32_t length, bool is_notify);

-

-/** \brief        Submit USB transfer

- * \param[in]		  coreid USB Controller ID

- * \param[in]     p_buffer  application's buffer to receive data. Must be accessible by USB controller (see \ref TUSB_CFG_ATTR_USBRAM)

- * \param[in]     length number of bytes in \a p_buffer.

- * \param[in]     is_notify indicates whether the hardware completion (data transferred through USB bus) will be notified

- *                to Application (via \ref tusbd_cdc_xfer_cb)

- * \returns       \ref tusb_error_t type to indicate success or error condition.

- * \retval        TUSB_ERROR_NONE on success

- * \retval        TUSB_ERROR_INTERFACE_IS_BUSY if the interface is busy transferring previous data.

- * \retval        TUSB_ERROR_DEVICE_NOT_READY if device is not yet configured (by SET CONFIGURED request)

- * \retval        TUSB_ERROR_INVALID_PARA if input parameters are not correct

- * \note          This function is non-blocking and returns immediately. Data will be transferred when USB Host work with this interface.

- *                The result of usb transfer will be reported by the interface's callback function if \a is_notify is true

- */

-tusb_error_t tud_cdc_receive(uint8_t coreid, void * p_buffer, uint32_t length, bool is_notify);

-

 bool     tud_cdc_connected(uint8_t coreid);

 uint32_t tud_cdc_available(uint8_t coreid);

 

@@ -108,17 +68,7 @@ uint32_t tud_cdc_write(uint8_t coreid, void const* buffer, uint32_t bufsize);
 //--------------------------------------------------------------------+

 // APPLICATION CALLBACK API

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

-/** \brief      Callback function that is invoked when an completion (error or success) of an USB transfer previously submitted

- *              by application (e.g \ref tusbd_cdc_send or \ref tusbd_cdc_send) with \a is_notify set to true.

- * \param[in]		coreid	USB Controller ID

- * \param[in]   event an value from \ref tusb_event_t

- * \param[in]   pipe_id indicates which pipe of this interface the event occured.

- * \param[in]   xferred_bytes is actual number of bytes transferred via USB bus. This value in general can be different to

- *              the one that previously submitted by application.

- */

-void tud_cdc_xfer_cb(uint8_t coreid, tusb_event_t event, cdc_pipeid_t pipe_id, uint32_t xferred_bytes);

 //void tud_cdc_line_coding_changed_cb(uint8_t coreid, cdc_line_coding_t* p_line_coding);

-

 void tud_cdc_rx_cb(uint8_t coreid);

 

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