aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarek Vrbka <marek.vrbka@codasip.com>2023-09-06 13:04:04 +0200
committerMarek Vrbka <marek.vrbka@codasip.com>2023-09-07 08:08:06 +0200
commit1936dbd6cffd130f71b18ea38585c03ab9418b18 (patch)
tree603de8f35079b729ac1621d7c077d3c4be9f7091
parent699eecaab434337dc3915171606b0548c48c6d51 (diff)
downloadriscv-openocd-1936dbd6cffd130f71b18ea38585c03ab9418b18.zip
riscv-openocd-1936dbd6cffd130f71b18ea38585c03ab9418b18.tar.gz
riscv-openocd-1936dbd6cffd130f71b18ea38585c03ab9418b18.tar.bz2
target/riscv_semihosting: Make the unknown operation number an error
Previously, an unknown semihosting operation number was logged as debug. This patch changes it and few other places to be logged as error instead. Change-Id: I83cae5ca1e3daed440f92b08bd372bfffbbad63c Signed-off-by: Marek Vrbka <marek.vrbka@codasip.com>
-rw-r--r--src/target/riscv/riscv_semihosting.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/target/riscv/riscv_semihosting.c b/src/target/riscv/riscv_semihosting.c
index 15c3d60..9c708c8 100644
--- a/src/target/riscv/riscv_semihosting.c
+++ b/src/target/riscv/riscv_semihosting.c
@@ -109,13 +109,13 @@ enum semihosting_result riscv_semihosting(struct target *target, int *retval)
result = riscv_get_register(target, &r0, GDB_REGNO_A0);
if (result != ERROR_OK) {
- LOG_TARGET_DEBUG(target, " -> ERROR (couldn't read a0)");
+ LOG_TARGET_ERROR(target, "Could not read semihosting operation code (register a0)");
return SEMIHOSTING_ERROR;
}
result = riscv_get_register(target, &r1, GDB_REGNO_A1);
if (result != ERROR_OK) {
- LOG_TARGET_DEBUG(target, " -> ERROR (couldn't read a1)");
+ LOG_TARGET_ERROR(target, "Could not read semihosting operation code (register a1)");
return SEMIHOSTING_ERROR;
}
@@ -134,7 +134,7 @@ enum semihosting_result riscv_semihosting(struct target *target, int *retval)
}
} else {
/* Unknown operation number, not a semihosting call. */
- LOG_TARGET_DEBUG(target, " -> NONE (unknown operation number)");
+ LOG_TARGET_ERROR(target, "Unknown semihosting operation requested (op = 0x%x)", semihosting->op);
return SEMIHOSTING_NONE;
}
}