Attention is currently required from: Nico Huber, Angel Pons. Peter Farley has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63798 )
Change subject: ft2232_spi: Reduce read size for FT232H ......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/flashrom/+/63798/comment/0450bb6d_606759ea PS1, Line 9: FT232H-based devices are unable to read over 256 bytes of flash in one
ftdi_read_data: -1, usb bulk read failed […]
I get the same result for wMaxPacketSize.
Decreasing the read chunk size below 512 (if I have not also decreased the max_data_read) causes a segfault when writing to the output file. (I assume it's trying to read beyond the buffer size, but I haven't looked too deeply into it.)
Looking at Wireshark captures of the USB communication, it does look like the device is transmitting all the data. I'm not familiar enough with the FTDI protocols to see if something looks wrong there though.
I don't see any way to upload files here, so here are the verbose outputs from tshark, if they might be helpful at all:
64K: https://pastebin.com/er7mv3tT 256B: https://pastebin.com/jfb2yv3d (Cut short due to its length)
The 64K reads start around frame 43. And frame 45 for the 256B reads.
-
Peter Farley (Code Review)