andy/tinyUSB
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Changed comments, added define to fail if ISOCHRONOUS endpoint is not available but requested

Browse Source
This commit is contained in:
Simon Kueppers
2022-10-25 18:39:09 +02:00
committed by Mengsk
parent ec826732fc
commit 9a84390878
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/portable/st/stm32_fsdev/dcd_stm32_fsdev.c
View File

@@ -64,10 +64,9 @@
* - STALL handled, but not tested. * - STALL handled, but not tested.
* - Does it work? No clue. * - Does it work? No clue.
* - All EP BTABLE buffers are created based on max packet size of first EP opened with that address. * - All EP BTABLE buffers are created based on max packet size of first EP opened with that address.
* - No isochronous endpoints
* - Endpoint index is the ID of the endpoint * - Endpoint index is the ID of the endpoint
* - This means that priority is given to endpoints with lower ID numbers * - This means that priority is given to endpoints with lower ID numbers
* - Code is mixing up EP IX with EP ID. Everywhere. * - Manual override of this mapping is possible through callback
* - Packet buffer memory is copied in the interrupt. * - Packet buffer memory is copied in the interrupt.
* - This is better for performance, but means interrupts are disabled for longer * - This is better for performance, but means interrupts are disabled for longer
* - DMA may be the best choice, but it could also be pushed to the USBD task. * - DMA may be the best choice, but it could also be pushed to the USBD task.
@@ -776,17 +775,19 @@ bool dcd_edpt_open (uint8_t rhport, tusb_desc_endpoint_t const * p_endpoint_desc
uint16_t pma_addr; uint16_t pma_addr;
uint32_t wType; uint32_t wType;
// Isochronous not supported (yet), and some other driver assumptions.
TU_ASSERT(epnum < MAX_EP_COUNT); TU_ASSERT(epnum < MAX_EP_COUNT);
TU_ASSERT(buffer_size <= 1024);
// Set type // Set type
switch(p_endpoint_desc->bmAttributes.xfer) { switch(p_endpoint_desc->bmAttributes.xfer) {
case TUSB_XFER_CONTROL: case TUSB_XFER_CONTROL:
wType = USB_EP_CONTROL; wType = USB_EP_CONTROL;
break; break;
#if defined(ISOCHRONOUS_DOUBLEBUFFER)
case TUSB_XFER_ISOCHRONOUS: case TUSB_XFER_ISOCHRONOUS:
wType = USB_EP_ISOCHRONOUS; wType = USB_EP_ISOCHRONOUS;
break; break;
#endif
case TUSB_XFER_BULK: case TUSB_XFER_BULK:
wType = USB_EP_CONTROL; wType = USB_EP_CONTROL;
break; break;