|
From: openocd-gerrit <ope...@us...> - 2025-11-12 20:28:00
|
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".
The branch, master has been updated
via f354d259ffe613ed2e5c4369b8c25727baab32c5 (commit)
from 6d764f5b099b40151de4f02e39231533cf14eae5 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit f354d259ffe613ed2e5c4369b8c25727baab32c5
Author: Tomas Vanek <va...@fb...>
Date: Tue Oct 14 15:29:55 2025 +0200
target/riscv: return ERROR_TARGET_NOT_HALTED
instead of ERROR_FAIL where appropriate.
Change-Id: I1881c0c6c437355007c3844556489162666023dc
Signed-off-by: Tomas Vanek <va...@fb...>
Reviewed-on: https://review.openocd.org/c/openocd/+/9171
Tested-by: jenkins
Reviewed-by: Evgeniy Naydanov <evg...@sy...>
diff --git a/src/target/riscv/riscv-013.c b/src/target/riscv/riscv-013.c
index 9da9684e4..d2afdba2d 100644
--- a/src/target/riscv/riscv-013.c
+++ b/src/target/riscv/riscv-013.c
@@ -2323,7 +2323,7 @@ static int prep_for_vector_access(struct target *target,
if (target->state != TARGET_HALTED) {
LOG_TARGET_ERROR(target,
"Unable to access vector register: target not halted");
- return ERROR_FAIL;
+ return ERROR_TARGET_NOT_HALTED;
}
if (prep_for_register_access(target, orig_mstatus, GDB_REGNO_VL) != ERROR_OK)
return ERROR_FAIL;
@@ -5476,7 +5476,7 @@ static int riscv013_step_or_resume_current_hart(struct target *target,
{
if (target->state != TARGET_HALTED) {
LOG_TARGET_ERROR(target, "Hart is not halted!");
- return ERROR_FAIL;
+ return ERROR_TARGET_NOT_HALTED;
}
LOG_TARGET_DEBUG(target, "resuming (operation=%s)",
diff --git a/src/target/riscv/riscv.c b/src/target/riscv/riscv.c
index 7ef875f80..92daffa49 100644
--- a/src/target/riscv/riscv.c
+++ b/src/target/riscv/riscv.c
@@ -5425,7 +5425,7 @@ COMMAND_HANDLER(riscv_exec_progbuf)
if (target->state != TARGET_HALTED) {
LOG_TARGET_ERROR(target, "exec_progbuf: Can't execute "
"program buffer, target not halted.");
- return ERROR_FAIL;
+ return ERROR_TARGET_NOT_HALTED;
}
if (riscv_progbuf_size(target) == 0) {
@@ -6039,7 +6039,7 @@ static int riscv_step_rtos_hart(struct target *target)
if (target->state != TARGET_HALTED) {
LOG_TARGET_ERROR(target, "Hart isn't halted before single step!");
- return ERROR_FAIL;
+ return ERROR_TARGET_NOT_HALTED;
}
r->on_step(target);
if (r->step_current_hart(target) != ERROR_OK)
@@ -6229,7 +6229,7 @@ int riscv_enumerate_triggers(struct target *target)
if (target->state != TARGET_HALTED) {
LOG_TARGET_ERROR(target, "Unable to enumerate triggers: target not halted.");
- return ERROR_FAIL;
+ return ERROR_TARGET_NOT_HALTED;
}
riscv_reg_t orig_tselect;
-----------------------------------------------------------------------
Summary of changes:
src/target/riscv/riscv-013.c | 4 ++--
src/target/riscv/riscv.c | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
hooks/post-receive
--
Main OpenOCD repository
|