From 394302c8cb1b712d55929ae19805a96d560376db Mon Sep 17 00:00:00 2001 From: hathach Date: Sat, 3 Mar 2018 14:45:29 +0700 Subject: [PATCH] rename usbd_setup_received_isr to hal_dcd_setup_received --- hw/mcu/nxp/lpc43xx/tusb_port/dcd_lpc43xx.c | 3 +-- tinyusb/device/dcd_lpc175x_6x.c | 2 +- tinyusb/device/dcd_lpc_11uxx_13uxx.c | 2 +- tinyusb/device/usbd.c | 4 ++-- tinyusb/device/usbd_dcd.h | 7 ++----- 5 files changed, 7 insertions(+), 11 deletions(-) diff --git a/hw/mcu/nxp/lpc43xx/tusb_port/dcd_lpc43xx.c b/hw/mcu/nxp/lpc43xx/tusb_port/dcd_lpc43xx.c index b1869c48..93f27373 100644 --- a/hw/mcu/nxp/lpc43xx/tusb_port/dcd_lpc43xx.c +++ b/hw/mcu/nxp/lpc43xx/tusb_port/dcd_lpc43xx.c @@ -567,10 +567,9 @@ void dcd_isr(uint8_t coreid) //------------- Set up Received -------------// if (lpc_usb->ENDPTSETUPSTAT) { // 23.10.10.2 Operational model for setup transfers - tusb_control_request_t control_request = p_dcd->qhd[0].setup_request; lpc_usb->ENDPTSETUPSTAT = lpc_usb->ENDPTSETUPSTAT;// acknowledge - usbd_setup_received_isr(coreid, &control_request); + hal_dcd_setup_received(coreid, (uint8_t*) &p_dcd->qhd[0].setup_request); } //------------- Control Request Completed -------------// else if ( edpt_complete & 0x03 ) diff --git a/tinyusb/device/dcd_lpc175x_6x.c b/tinyusb/device/dcd_lpc175x_6x.c index 90509e29..f34da292 100644 --- a/tinyusb/device/dcd_lpc175x_6x.c +++ b/tinyusb/device/dcd_lpc175x_6x.c @@ -188,7 +188,7 @@ static void endpoint_control_isr(void) tusb_control_request_t control_request; pipe_control_read(&control_request, 8); // TODO read before clear setup above - usbd_setup_received_isr(0, &control_request); + hal_dcd_setup_received(0, (uint8_t*) &control_request); } else if (endpoint_int_status & 0x03) { diff --git a/tinyusb/device/dcd_lpc_11uxx_13uxx.c b/tinyusb/device/dcd_lpc_11uxx_13uxx.c index 4c95a487..34b0861e 100644 --- a/tinyusb/device/dcd_lpc_11uxx_13uxx.c +++ b/tinyusb/device/dcd_lpc_11uxx_13uxx.c @@ -349,7 +349,7 @@ void dcd_isr(uint8_t coreid) if ( BIT_TEST_(int_status, 0) && (dev_cmd_stat & CMDSTAT_SETUP_RECEIVED_MASK) ) { // received control request from host // copy setup request & acknowledge so that the next setup can be received by hw - usbd_setup_received_isr(coreid, &dcd_data.setup_request); + hal_dcd_setup_received(coreid, (uint8_t*)&dcd_data.setup_request); // NXP control flowchart clear Active & Stall on both Control IN/OUT endpoints dcd_data.qhd[0][0].stall = dcd_data.qhd[1][0].stall = 0; diff --git a/tinyusb/device/usbd.c b/tinyusb/device/usbd.c index 48dd57eb..5908404d 100644 --- a/tinyusb/device/usbd.c +++ b/tinyusb/device/usbd.c @@ -432,7 +432,7 @@ void usbd_dcd_bus_event_isr(uint8_t coreid, usbd_bus_event_type_t bus_event) } } -void usbd_setup_received_isr(uint8_t coreid, tusb_control_request_t * p_request) +void hal_dcd_setup_received(uint8_t coreid, uint8_t const* p_request) { usbd_task_event_t task_event = { @@ -440,7 +440,7 @@ void usbd_setup_received_isr(uint8_t coreid, tusb_control_request_t * p_request) .event_id = USBD_EVENTID_SETUP_RECEIVED, }; - task_event.setup_received = (*p_request); + memcpy(&task_event.setup_received, p_request, sizeof(tusb_control_request_t)); osal_queue_send(usbd_queue_hdl, &task_event); } diff --git a/tinyusb/device/usbd_dcd.h b/tinyusb/device/usbd_dcd.h index a82da839..8bc3a5fb 100644 --- a/tinyusb/device/usbd_dcd.h +++ b/tinyusb/device/usbd_dcd.h @@ -47,10 +47,6 @@ //--------------------------------------------------------------------+ #include "common/common.h" -#ifdef _TEST_ -#include "dcd.h" -#endif - #ifdef __cplusplus extern "C" { #endif @@ -79,7 +75,8 @@ extern usbd_device_info_t usbd_devices[CONTROLLER_DEVICE_NUMBER]; // callback from DCD ISR //--------------------------------------------------------------------+ void usbd_dcd_bus_event_isr(uint8_t coreid, usbd_bus_event_type_t bus_event); -void usbd_setup_received_isr(uint8_t coreid, tusb_control_request_t * p_request); +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);