aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Newsome <tim@sifive.com>2022-12-27 10:54:26 -0800
committerGitHub <noreply@github.com>2022-12-27 10:54:26 -0800
commiteb8bb957d4c34adfe40fb02f57dfafff14e871a8 (patch)
tree7cca14fd37fa37d169e16abc72da183b018f1dcf
parentb337b0cfb4612c0474c109abeb511189bfd5559c (diff)
downloadriscv-openocd-eb8bb957d4c34adfe40fb02f57dfafff14e871a8.zip
riscv-openocd-eb8bb957d4c34adfe40fb02f57dfafff14e871a8.tar.gz
riscv-openocd-eb8bb957d4c34adfe40fb02f57dfafff14e871a8.tar.bz2
flash: fix clang static analyzer build errors (#778)
Fixes "variable set but not used" errors. Tested with Homebrew clang version 13.0.1 Signed-off-by: Erhan Kurubas <erhan.kurubas@espressif.com> Change-Id: Ia90baf5b4857db2b5569ebe6adbbb832de772aad Reviewed-on: https://review.openocd.org/c/openocd/+/6971 Tested-by: jenkins Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com> Signed-off-by: Erhan Kurubas <erhan.kurubas@espressif.com> Co-authored-by: Erhan Kurubas <erhan.kurubas@espressif.com>
-rw-r--r--src/flash/nand/davinci.c2
-rw-r--r--src/flash/nor/dsp5680xx_flash.c3
2 files changed, 0 insertions, 5 deletions
diff --git a/src/flash/nand/davinci.c b/src/flash/nand/davinci.c
index 84f8e34..1aa7ffc 100644
--- a/src/flash/nand/davinci.c
+++ b/src/flash/nand/davinci.c
@@ -605,8 +605,6 @@ static int davinci_write_page_ecc4infix(struct nand_device *nand, uint32_t page,
/* write this "out-of-band" data -- infix */
davinci_write_block_data(nand, oob, 16);
oob += 16;
- oob_size -= 16;
-
} while (data_size);
/* the last data and OOB writes included the spare area */
diff --git a/src/flash/nor/dsp5680xx_flash.c b/src/flash/nor/dsp5680xx_flash.c
index 5e8eec3..858b669 100644
--- a/src/flash/nor/dsp5680xx_flash.c
+++ b/src/flash/nor/dsp5680xx_flash.c
@@ -45,14 +45,11 @@
static int dsp5680xx_build_sector_list(struct flash_bank *bank)
{
- uint32_t offset = HFM_FLASH_BASE_ADDR;
-
bank->sectors = malloc(sizeof(struct flash_sector) * bank->num_sectors);
for (unsigned int i = 0; i < bank->num_sectors; ++i) {
bank->sectors[i].offset = i * HFM_SECTOR_SIZE;
bank->sectors[i].size = HFM_SECTOR_SIZE;
- offset += bank->sectors[i].size;
bank->sectors[i].is_erased = -1;
bank->sectors[i].is_protected = -1;
}