diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2016-12-17 13:16:07 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2016-12-19 22:34:52 +0100 |
| commit | 41f669023953b3d032078ffc17d80944880d1db7 (patch) | |
| tree | ed5243a97c69a8d55c5fb7bd748fa158d36008a4 /src/drivers/spi/winbond.c | |
| parent | 06cd903566b57af4698089de4a67dd49bf7e56ed (diff) | |
| download | coreboot-41f669023953b3d032078ffc17d80944880d1db7.tar.gz coreboot-41f669023953b3d032078ffc17d80944880d1db7.tar.bz2 coreboot-41f669023953b3d032078ffc17d80944880d1db7.zip | |
drivers/spi: fix flash writes at page boundaries
There was an assumption that all SPI controllers could
consume a full page of data to write. However, that
assumption doesn't hold when spi_crop_chunk() indicates
sizes smaller than page size. If the requested offset isn't
page aligned from the start then writes will fail corrupting
data since a page boundary isn't honored.
The spansion driver needed quite a bit more work to honor
the spi_crop_chunk() result. It now mimics the other
driver's code. Also, needed to add spi_crop_chunk() to
marvell/bg4cd SoC to make google/cosmos build. SPI obviously
doesn't work on that platform, but it fixes the build error.
Change-Id: I93e24a5a717adcee45a017c164bd960f4592ad50
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/17910
Tested-by: build bot (Jenkins)
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'src/drivers/spi/winbond.c')
| -rw-r--r-- | src/drivers/spi/winbond.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/drivers/spi/winbond.c b/src/drivers/spi/winbond.c index 74c6e81185b3..d071b9f28d3c 100644 --- a/src/drivers/spi/winbond.c +++ b/src/drivers/spi/winbond.c @@ -151,9 +151,9 @@ static int winbond_write(const struct spi_flash *flash, u32 offset, size_t len, u8 cmd[4]; page_size = 1 << stm->params->l2_page_size; - byte_addr = offset % page_size; for (actual = 0; actual < len; actual += chunk_len) { + byte_addr = offset % page_size; chunk_len = min(len - actual, page_size - byte_addr); chunk_len = spi_crop_chunk(sizeof(cmd), chunk_len); @@ -185,7 +185,6 @@ static int winbond_write(const struct spi_flash *flash, u32 offset, size_t len, goto out; offset += chunk_len; - byte_addr = 0; } #if CONFIG_DEBUG_SPI_FLASH |