Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/55683 )
Change subject: ft2232_spi: Don't lower write data chunksize ......................................................................
ft2232_spi: Don't lower write data chunksize
This "chunk size" limits the amount of data that is passed to libusb at once. If we had exceeded the chunk size, libftdi would have split the data into individual, synchronous bulk transfers. But the chunk size was actually chosen to avoid this. So without any known effect, setting the chunk size is useless. Drop it.
Change-Id: I779e24dc3f3379a98ddce02c3765062ac3241884 Signed-off-by: Nico Huber nico.h@gmx.de Reviewed-on: https://review.coreboot.org/c/flashrom/+/55683 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Edward O'Callaghan quasisec@chromium.org --- M ft2232_spi.c 1 file changed, 0 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Edward O'Callaghan: Looks good to me, approved
diff --git a/ft2232_spi.c b/ft2232_spi.c index 3fe7ea4..5b5d3cd 100644 --- a/ft2232_spi.c +++ b/ft2232_spi.c @@ -651,10 +651,6 @@ msg_perr("Unable to set latency timer (%s).\n", ftdi_get_error_string(&ftdic)); }
- if (ftdi_write_data_set_chunksize(&ftdic, 270)) { - msg_perr("Unable to set chunk size (%s).\n", ftdi_get_error_string(&ftdic)); - } - if (ftdi_set_bitmode(&ftdic, 0x00, BITMODE_BITBANG_SPI) < 0) { msg_perr("Unable to set bitmode to SPI (%s).\n", ftdi_get_error_string(&ftdic)); }