kingkevin
/
esp32-s2_dfu
1
0
Fork 0
Code Issues Releases Activity

valentyusb: eptri: don't double-advance read buffer 

Due to an error, we were double-advancing the FIFO buffer.  The end
result was that the second half of most reads were getting ignored.

This wasn't found during earlier testing because only 64-byte buffers
were tested.

This corrects this error by avoiding double-advancing the buffer.

Signed-off-by: Sean Cross <sean@xobs.io>
This commit is contained in:
Sean Cross 2019-11-29 18:14:23 +08:00
parent 2e0dcd50ea
commit 26dcc19b18
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/portable/valentyusb/eptri/dcd_eptri.c
View File

@ -226,7 +226,7 @@ static void process_rx(void) {
test_buffer[total_read] = c;
#endif
total_read++;
if ((rx_buffer_offset[rx_ep] + current_offset) < rx_buffer_max[rx_ep]) {
if (current_offset < rx_buffer_max[rx_ep]) {
#if LOG_USB
usb_log[usb_log_offset].data[usb_log[usb_log_offset].size++] = c;
#endif