diff options
author | Tomas Vanek <vanekt@fbl.cz> | 2018-01-21 10:47:32 +0100 |
---|---|---|
committer | Tomas Vanek <vanekt@fbl.cz> | 2018-02-04 21:44:27 +0000 |
commit | 0056037d68d29c566ce9f527a60828af51c68329 (patch) | |
tree | 971fa49fef8abf7136799e7a0f406764de0d26bd /src/flash | |
parent | f2b228f5d1728b9972624c1a7ae681fc6df2cddc (diff) | |
download | riscv-openocd-0056037d68d29c566ce9f527a60828af51c68329.zip riscv-openocd-0056037d68d29c566ce9f527a60828af51c68329.tar.gz riscv-openocd-0056037d68d29c566ce9f527a60828af51c68329.tar.bz2 |
flash/nor/kinetis_ke: fix warning retval set but not used
I see no reason for not returning error from target_run_algorithm()
to higher level.
Reported by Clang static analyzer.
Change-Id: Iaaa8b66e487ecae88c0cf4ae2addba63341c032c
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: http://openocd.zylin.com/4391
Tested-by: jenkins
Reviewed-by: Ivan Meleca <ivan@artekit.eu>
Diffstat (limited to 'src/flash')
-rw-r--r-- | src/flash/nor/kinetis_ke.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/flash/nor/kinetis_ke.c b/src/flash/nor/kinetis_ke.c index 636fcc2..b7a6a1e 100644 --- a/src/flash/nor/kinetis_ke.c +++ b/src/flash/nor/kinetis_ke.c @@ -478,14 +478,13 @@ int kinetis_ke_stop_watchdog(struct target *target) watchdog_algorithm->address, 0, 100000, &armv7m_info); if (retval != ERROR_OK) { LOG_ERROR("Error executing Kinetis KE watchdog algorithm"); - retval = ERROR_FAIL; } else { LOG_INFO("Watchdog stopped"); } target_free_working_area(target, watchdog_algorithm); - return ERROR_OK; + return retval; } COMMAND_HANDLER(kinetis_ke_disable_wdog_handler) |