diff options
| author | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-01-03 13:30:28 +0100 |
|---|---|---|
| committer | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-01-03 13:33:27 +0100 |
| commit | c69553cbc51770f61cf3b9225d46d058fa2544d0 (patch) | |
| tree | 07aa7f6d5bc1861cc926e2359034ef42f17dc46a /src/target/etm.c | |
| parent | 1795239cfda77315ea2f4fbc028e7a411d13a7d0 (diff) | |
| download | riscv-openocd-c69553cbc51770f61cf3b9225d46d058fa2544d0.zip riscv-openocd-c69553cbc51770f61cf3b9225d46d058fa2544d0.tar.gz riscv-openocd-c69553cbc51770f61cf3b9225d46d058fa2544d0.tar.bz2 | |
error handling: the error number is not part of the user interface
Do not propagate error number to user. This is for internal
programming purposes only. Error messages to the user is
reported as text via LOG_ERROR().
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Diffstat (limited to 'src/target/etm.c')
| -rw-r--r-- | src/target/etm.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/target/etm.c b/src/target/etm.c index 9f7bc83..7384764 100644 --- a/src/target/etm.c +++ b/src/target/etm.c @@ -669,7 +669,7 @@ static int etm_read_instruction(struct etm_context *ctx, struct arm_instruction ctx->current_pc - ctx->image->sections[section].base_address, 4, buf, &size_read)) != ERROR_OK) { - LOG_ERROR("error while reading instruction: %i", retval); + LOG_ERROR("error while reading instruction"); return ERROR_TRACE_INSTRUCTION_UNAVAILABLE; } opcode = target_buffer_get_u32(ctx->target, buf); @@ -682,7 +682,7 @@ static int etm_read_instruction(struct etm_context *ctx, struct arm_instruction ctx->current_pc - ctx->image->sections[section].base_address, 2, buf, &size_read)) != ERROR_OK) { - LOG_ERROR("error while reading instruction: %i", retval); + LOG_ERROR("error while reading instruction"); return ERROR_TRACE_INSTRUCTION_UNAVAILABLE; } opcode = target_buffer_get_u16(ctx->target, buf); @@ -2109,6 +2109,7 @@ COMMAND_HANDLER(handle_etm_analyze_command) if ((retval = etmv1_analyze_trace(etm_ctx, CMD_CTX)) != ERROR_OK) { + /* FIX! error should be reported inside etmv1_analyze_trace() */ switch (retval) { case ERROR_ETM_ANALYSIS_FAILED: @@ -2121,7 +2122,7 @@ COMMAND_HANDLER(handle_etm_analyze_command) command_print(CMD_CTX, "no image available for trace analysis"); break; default: - command_print(CMD_CTX, "unknown error: %i", retval); + command_print(CMD_CTX, "unknown error"); } } |