diff options
author | Tomas Vanek <vanekt@fbl.cz> | 2022-09-11 12:11:31 +0200 |
---|---|---|
committer | Tomas Vanek <vanekt@fbl.cz> | 2022-09-27 08:39:58 +0000 |
commit | 84d73d0225407594ed9cf646fa3dd8a5752df7ad (patch) | |
tree | 8c046f84301e6720af48ef872a9cefa6faae980c /src/flash | |
parent | 47ed1c1eab82ee329915e04ea61f857dae89fc6a (diff) | |
download | riscv-openocd-84d73d0225407594ed9cf646fa3dd8a5752df7ad.zip riscv-openocd-84d73d0225407594ed9cf646fa3dd8a5752df7ad.tar.gz riscv-openocd-84d73d0225407594ed9cf646fa3dd8a5752df7ad.tar.bz2 |
flash/nor/rp2040: fix size of flash write buffer
The size of the flash write buffer should be rounded
down to the multiply of flash page size.
Using write chunks of unadjusted size results in write of chunks
unaligned to flash pages.
Change-Id: If7931362ee193dff4dc2df7ec78f13530658cf08
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: https://review.openocd.org/c/openocd/+/7187
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Diffstat (limited to 'src/flash')
-rw-r--r-- | src/flash/nor/rp2040.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/flash/nor/rp2040.c b/src/flash/nor/rp2040.c index 1c57424..ce09fea 100644 --- a/src/flash/nor/rp2040.c +++ b/src/flash/nor/rp2040.c @@ -217,7 +217,11 @@ static int rp2040_flash_write(struct flash_bank *bank, const uint8_t *buffer, ui if (err != ERROR_OK) goto cleanup; - const unsigned int chunk_size = target_get_working_area_avail(target); + unsigned int avail_pages = target_get_working_area_avail(target) / priv->dev->pagesize; + /* We try to allocate working area rounded down to device page size, + * al least 1 page, at most the write data size + */ + unsigned int chunk_size = MIN(MAX(avail_pages, 1) * priv->dev->pagesize, count); err = target_alloc_working_area(target, chunk_size, &bounce); if (err != ERROR_OK) { LOG_ERROR("Could not allocate bounce buffer for flash programming. Can't continue"); |