This is an automated email from Gerrit. "Name of user not set <[email protected]>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/9574
-- gerrit commit 04acd548d56355967b0713f43ee1645a38464bb1 Author: ssampath <[email protected]> Date: Wed Apr 8 17:54:28 2026 -0700 target/riscv: Remove redundant calls to select_dmi() This is to fix issue #1092 Change-Id: If963ecf534d4a4fb5cb06e111b0023bbb9ab65c2 Signed-off-by: ssampath <[email protected]> BugLink: https://github.com/riscv-collab/riscv-openocd/issues/1092 diff --git a/src/target/riscv/riscv-013.c b/src/target/riscv/riscv-013.c index 50e0f83dd8..35ab136ed0 100644 --- a/src/target/riscv/riscv-013.c +++ b/src/target/riscv/riscv-013.c @@ -2909,8 +2909,6 @@ static int assert_reset(struct target *target) RISCV013_INFO(info); int result; - select_dmi(target->tap); - if (target_has_event_action(target, TARGET_EVENT_RESET_ASSERT)) { /* Run the user-supplied script if there is one. */ target_handle_event(target, TARGET_EVENT_RESET_ASSERT); @@ -2963,7 +2961,6 @@ static int deassert_reset(struct target *target) return ERROR_FAIL; int result; - select_dmi(target->tap); /* Clear the reset, but make sure haltreq is still set */ uint32_t control = 0; control = set_field(control, DM_DMCONTROL_DMACTIVE, 1); @@ -4450,7 +4447,6 @@ read_memory_progbuf(struct target *target, const struct riscv_mem_access_args ar { assert(riscv_mem_access_is_read(args)); - select_dmi(target->tap); memset(args.read_buffer, 0, args.count * args.size); if (execute_autofence(target) != ERROR_OK) --
