diff options
author | Nico Huber <nico.h@gmx.de> | 2019-06-18 23:39:56 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2019-06-21 11:54:19 +0000 |
commit | d8b2e808cd46986f945ba9cf3b90c70fe58de9c6 (patch) | |
tree | 4094be4996c4ae32a78e0e13c558ee78bcdd85dc /spi25.c | |
parent | 0373ce31fe5b11dcf23b27fbc221ba019a1cf7f1 (diff) | |
download | flashrom-d8b2e808cd46986f945ba9cf3b90c70fe58de9c6.tar.gz flashrom-d8b2e808cd46986f945ba9cf3b90c70fe58de9c6.tar.bz2 flashrom-d8b2e808cd46986f945ba9cf3b90c70fe58de9c6.zip |
spi: Move 16MiB partitioning up into spi_chip_read()
We enforced a 16MiB limit in spi_read_chunked() for multi-die flash
chips that can't be fully read at once. The same limit can be useful
for dediprog programmers. So move it into a more generic place.
Change-Id: Iab1fd5b2ea550b4b3ef3e8402e0b6ca218485a51
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/33613
Reviewed-by: Ryan O'Leary
Reviewed-by: ron minnich <rminnich@gmail.com>
Reviewed-by: David Hendricks <david.hendricks@gmail.com>
Diffstat (limited to 'spi25.c')
-rw-r--r-- | spi25.c | 41 |
1 files changed, 9 insertions, 32 deletions
@@ -657,43 +657,20 @@ int spi_nbyte_read(struct flashctx *flash, unsigned int address, uint8_t *bytes, /* * Read a part of the flash chip. - * FIXME: Use the chunk code from Michael Karcher instead. - * Each naturally aligned area is read separately in chunks with a maximum size of chunksize. + * Data is read in chunks with a maximum size of chunksize. */ int spi_read_chunked(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len, unsigned int chunksize) { - int rc = 0; - unsigned int i, j, starthere, lenhere, toread; - /* Limit for multi-die 4-byte-addressing chips. */ - unsigned int area_size = min(flash->chip->total_size * 1024, 16 * 1024 * 1024); - - /* Warning: This loop has a very unusual condition and body. - * The loop needs to go through each area with at least one affected - * byte. The lowest area number is (start / area_size) since that - * division rounds down. The highest area number we want is the area - * where the last byte of the range lives. That last byte has the - * address (start + len - 1), thus the highest area number is - * (start + len - 1) / area_size. Since we want to include that last - * area as well, the loop condition uses <=. - */ - for (i = start / area_size; i <= (start + len - 1) / area_size; i++) { - /* Byte position of the first byte in the range in this area. */ - /* starthere is an offset to the base address of the chip. */ - starthere = max(start, i * area_size); - /* Length of bytes in the range in this area. */ - lenhere = min(start + len, (i + 1) * area_size) - starthere; - for (j = 0; j < lenhere; j += chunksize) { - toread = min(chunksize, lenhere - j); - rc = spi_nbyte_read(flash, starthere + j, buf + starthere - start + j, toread); - if (rc) - break; - } - if (rc) - break; + int ret; + size_t to_read; + for (; len; len -= to_read, buf += to_read, start += to_read) { + to_read = min(chunksize, len); + ret = spi_nbyte_read(flash, start, buf, to_read); + if (ret) + return ret; } - - return rc; + return 0; } /* |