This is an automated email from Gerrit.

"Tim Newsome <t...@sifive.com>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/7401

-- gerrit

commit 9c0e516552083370f71795a4c079f17db3b506f9
Author: Tim Newsome <t...@sifive.com>
Date:   Wed Dec 14 15:15:45 2022 -0800

    flash/nor/dsp5680xx: Fix build.
    
    Newer compilers are smart enough to realize offset is never used, and
    warn about it.
    
    Change-Id: I330377db6b71b6c07b9de7ae4e0e7dd7efdeb8c1
    Signed-off-by: Tim Newsome <t...@sifive.com>

diff --git a/src/flash/nor/dsp5680xx_flash.c b/src/flash/nor/dsp5680xx_flash.c
index b1625f11f0..3df2db0172 100644
--- a/src/flash/nor/dsp5680xx_flash.c
+++ b/src/flash/nor/dsp5680xx_flash.c
@@ -34,11 +34,14 @@
 
 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;
        }

-- 

Reply via email to