openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-27 23:22:37 +02:00
Code Issues Releases Wiki Activity

dediprog: wait for spi bulk read xfers to finish

Browse Source 
dediprog_bulk_read_poll()'s finish argument allows it to be used in two
distinct cases: where dediprog_bulk_read_poll will be called as part of
a loop (finish=0) and where dediprog_bulk_read_poll should wait for all
outstanding transfers to finish (finish=1).  In both cases,
dediprog_bulk_read_poll() calls libusb to process events with a 10
second timeout.

After dediprog_spi_bulk_read() has queued the last transfers, it calls
dediprog_bulk_read_poll() with finish=0 when it should be finish=1.
finish=0 just happens to work because frequently the transfers finish in
the 10 second timeout.

Signed-off-by: Rick Altherr <rick@oxidecomputer.com>
Change-Id: If7cb541742c8620358c8e04275d8316131b2d1ab
Reviewed-on: https://review.coreboot.org/c/flashrom/+/60087
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
Rick Altherr 2021-12-13 17:10:00 -08:00 committed by Nico Huber
parent 89c73b5a74
commit 3ae04a1204
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dediprog.c
View File

@ -526,7 +526,7 @@ static int dediprog_spi_bulk_read(struct flashctx *flash, uint8_t *buf, unsigned
goto err_free; goto err_free;
} }
/* Wait for transfers to finish. */ /* Wait for transfers to finish. */
if (dediprog_bulk_read_poll(dp_data->usb_ctx, &status, 0)) if (dediprog_bulk_read_poll(dp_data->usb_ctx, &status, 1))
goto err_free; goto err_free;
/* Check if everything has been transmitted. */ /* Check if everything has been transmitted. */
if ((status.finished_idx < count) || status.error) if ((status.finished_idx < count) || status.error)