diff options
| author | Tomas Vanek <vanekt@fbl.cz> | 2023-07-20 14:52:54 +0200 |
|---|---|---|
| committer | Tomas Vanek <vanekt@fbl.cz> | 2023-07-29 05:17:44 +0000 |
| commit | a5108240f9e12633fea400f92d96cc75e03c86ca (patch) | |
| tree | ad263d29852d7646998e34bd14ecc914e361f72a /src/target/armv7m.c | |
| parent | 7023deb06a8cac77fe50d341c9e34b80d4fc68fd (diff) | |
| download | riscv-openocd-a5108240f9e12633fea400f92d96cc75e03c86ca.zip riscv-openocd-a5108240f9e12633fea400f92d96cc75e03c86ca.tar.gz riscv-openocd-a5108240f9e12633fea400f92d96cc75e03c86ca.tar.bz2 | |
target: fix messages and return values of failed op because not halted
Lot of messages was logged as LOG_WARNING, but the operation failed
immediately. Sometimes no error message was logged at all.
Add missing messages, change warnings to errors.
Sometimes ERROR_TARGET_INVALID was returned. Some command handlers
returned ERROR_OK! Always return ERROR_TARGET_NOT_HALTED.
While on it use LOG_TARGET_ERROR() whenever possible.
Prefix command_print() message with 'Error:' to get closer
to LOG_TARGET_ERROR() variant.
Error message was not added to get() and set() methods of
struct xxx_reg_type - the return value is properly checked and a message
is logged by the caller in case of ERROR_TARGET_NOT_HALTED.
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Change-Id: I2fe4187c6025f0038956ab387edbf3f461c69398
Reviewed-on: https://review.openocd.org/c/openocd/+/7819
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Diffstat (limited to 'src/target/armv7m.c')
| -rw-r--r-- | src/target/armv7m.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/target/armv7m.c b/src/target/armv7m.c index 8c9ff90..d508af7 100644 --- a/src/target/armv7m.c +++ b/src/target/armv7m.c @@ -525,7 +525,7 @@ int armv7m_start_algorithm(struct target *target, } if (target->state != TARGET_HALTED) { - LOG_WARNING("target not halted"); + LOG_TARGET_ERROR(target, "not halted (start target algo)"); return ERROR_TARGET_NOT_HALTED; } |