mirror/tinyusb
1
0
Fork 0
mirror of https://github.com/hathach/tinyusb.git synced 2025-04-03 16:20:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

stm32f4: Handle only one setup packet at a time as a workaround. Device

Browse Source 
enumerates.
This commit is contained in:
William D. Jones 2019-01-30 05:28:48 -05:00
parent a9e188a3d2
commit 7aadaa3ef2
1 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/portable/stm/stm32f4/dcd_stm32f4.c
View File

@ -95,7 +95,7 @@ static void bus_reset(void) {
USB_OTG_FS->GRXFSIZ = 40; USB_OTG_FS->GRXFSIZ = 40;
USB_OTG_FS->DIEPTXF0_HNPTXFSIZ |= (16 << USB_OTG_TX0FD_Pos); // 16 32-bit words = 64 bytes USB_OTG_FS->DIEPTXF0_HNPTXFSIZ |= (16 << USB_OTG_TX0FD_Pos); // 16 32-bit words = 64 bytes
out_ep[0].DOEPTSIZ |= (3 << USB_OTG_DOEPTSIZ_STUPCNT_Pos); out_ep[0].DOEPTSIZ |= (1 << USB_OTG_DOEPTSIZ_STUPCNT_Pos);
USB_OTG_FS->GINTMSK |= USB_OTG_GINTMSK_OEPINT | USB_OTG_GINTMSK_IEPINT; USB_OTG_FS->GINTMSK |= USB_OTG_GINTMSK_OEPINT | USB_OTG_GINTMSK_IEPINT;
} }
@ -475,16 +475,18 @@ void OTG_FS_IRQHandler(void) {
case 0x03: // Out packet done (Interrupt) case 0x03: // Out packet done (Interrupt)
break; break;
case 0x04: // Setup packet done (Interrupt) case 0x04: // Setup packet done (Interrupt)
_setup_offs = 2 - ((out_ep[epnum].DOEPTSIZ & USB_OTG_DOEPTSIZ_STUPCNT_Msk) >> USB_OTG_DOEPTSIZ_STUPCNT_Pos); out_ep[epnum].DOEPTSIZ |= (1 << USB_OTG_DOEPTSIZ_STUPCNT_Pos);
out_ep[epnum].DOEPTSIZ |= (3 << USB_OTG_DOEPTSIZ_STUPCNT_Pos);
break; break;
case 0x06: // Setup packet recvd case 0x06: // Setup packet recvd
{ {
uint8_t setup_left = ((out_ep[epnum].DOEPTSIZ & USB_OTG_DOEPTSIZ_STUPCNT_Msk) >> USB_OTG_DOEPTSIZ_STUPCNT_Pos); // For some reason, it's possible to get a mismatch between
// We can receive up to three setup packets in succession, but // how many setup packets were received versus the location
// only the last one is valid. // of the Setup packet done word. This leads to situations
_setup_packet[4 - 2*setup_left] = (* rx_fifo); // where stale setup packets are in the RX FIFO that were received
_setup_packet[5 - 2*setup_left] = (* rx_fifo); // after the core loaded the Setup packet done word. Workaround by
// only accepting one setup packet at a time for now.
_setup_packet[0] = (* rx_fifo);
_setup_packet[1] = (* rx_fifo);
} }
break; break;
default: // Invalid, do something here? default: // Invalid, do something here?
@ -505,7 +507,7 @@ void OTG_FS_IRQHandler(void) {
// SETUP packet Setup Phase done. // SETUP packet Setup Phase done.
if(out_ep[n].DOEPINT & USB_OTG_DOEPINT_STUP) { if(out_ep[n].DOEPINT & USB_OTG_DOEPINT_STUP) {
out_ep[n].DOEPINT = USB_OTG_DOEPINT_STUP; out_ep[n].DOEPINT = USB_OTG_DOEPINT_STUP;
dcd_event_setup_received(0, (uint8_t*) &_setup_packet[2*_setup_offs], true); dcd_event_setup_received(0, (uint8_t*) &_setup_packet[0], true);
_setup_offs = 0; _setup_offs = 0;
} }