diff options
| author | Marc Schink <dev@zapb.de> | 2020-06-07 17:00:13 +0200 |
|---|---|---|
| committer | Tomas Vanek <vanekt@fbl.cz> | 2020-07-07 05:23:54 +0100 |
| commit | ef14384b681af4f731f768bb866457832af6925f (patch) | |
| tree | a8cb953d765484d4cd627eaa7ea71d9be6699866 /src/flash/nor/w600.c | |
| parent | a2e6982a1816a0229bd5644156f3025a0e8cb6ce (diff) | |
| download | riscv-openocd-ef14384b681af4f731f768bb866457832af6925f.zip riscv-openocd-ef14384b681af4f731f768bb866457832af6925f.tar.gz riscv-openocd-ef14384b681af4f731f768bb866457832af6925f.tar.bz2 | |
flash/nor: Use proper data types in driver API
Use 'unsigned int' and 'bool' instead of 'int' where appropriate.
While at it, fix some coding style issues.
No new Clang analyzer warnings.
Change-Id: I700802c9ee81c3c7ae73108f0f8f06b15a4345f8
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: http://openocd.zylin.com/4929
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Diffstat (limited to 'src/flash/nor/w600.c')
| -rw-r--r-- | src/flash/nor/w600.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/flash/nor/w600.c b/src/flash/nor/w600.c index 3a6f3ff..ce3bc6e 100644 --- a/src/flash/nor/w600.c +++ b/src/flash/nor/w600.c @@ -204,7 +204,8 @@ static int w600_start(struct flash_bank *bank, uint32_t cmd, uint32_t addr, return retval; } -static int w600_erase(struct flash_bank *bank, int first, int last) +static int w600_erase(struct flash_bank *bank, unsigned int first, + unsigned int last) { int retval = ERROR_OK; @@ -217,7 +218,7 @@ static int w600_erase(struct flash_bank *bank, int first, int last) return ERROR_FAIL; } - for (int i = first; i <= last; i++) { + for (unsigned int i = first; i <= last; i++) { retval = w600_start(bank, QFLASH_CMD_SE, QFLASH_ADDR(bank->sectors[i].offset), 0); if (retval != ERROR_OK) |