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

Resolve erroneous data on enumeration.

Browse Source
This commit is contained in:
Clifroy Henry
2023-01-03 11:58:22 -05:00
committed by hathach
parent 87905d30ab
commit 5a9b3eab6f
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/portable/ti/msp430x5xx/dcd_msp430x5xx.c
View File

@@ -616,11 +616,20 @@ static void handle_setup_packet(void)
_setup_packet[i] = setup_buf[i]; _setup_packet[i] = setup_buf[i];
} }
// Clearing SETUPIFG by reading USBVECINT does not set NAK, so now that we // Force NAKs until tinyusb can handle the SETUP packet and prepare
// have a SETUP packet, force NAKs until tinyusb can handle the SETUP // for a new xfer.
// packet and prepare for a new xfer.
USBIEPCNT_0 |= NAK; USBIEPCNT_0 |= NAK;
USBOEPCNT_0 |= NAK; USBOEPCNT_0 |= NAK;
// Clear SETUPIFG to avoid handling in the USBVECINT switch statement.
// When handled there the NAKs applied to the endpoints above are
// cleared by hardware and the host will receive stale/duplicate data.
//
// Excerpt from MSP430x5xx and MSP430x6xx Family User's Guide:
//
// "...the SETUPIFG is cleared upon reading USBIV. In addition, the NAK on
// input endpoint 0 and output endpoint 0 is also cleared."
USBIFG &= ~SETUPIFG;
dcd_event_setup_received(0, (uint8_t*) &_setup_packet[0], true); dcd_event_setup_received(0, (uint8_t*) &_setup_packet[0], true);
} }