This is an automated email from Gerrit. "Antonio Borneo <[email protected]>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/9072
-- gerrit commit a4d317714fc0e86d67a6696b75ab3a00434fa843 Author: Antonio Borneo <[email protected]> Date: Mon Aug 4 17:45:53 2025 +0200 target: riscv-011: don't change 'debug_level' during target polling In the riscv fork, [1] has disable the debug log during target polling, with message: Improve low-level logging. Now logging is consistent and more readable. I did remove most logging during riscv_poll() since it clutters up the log/screen and is not generally helpful. This is questionable, because if the user enables the debug log, the messages should all be logged. Drop the code that overwrites the 'debug_level'. Link: https://github.com/riscv-collab/riscv-openocd/commit/54c65a9a4b71 [1] Change-Id: Ia86b998cf654760f36c2f217d44bcb9ffd9c3a94 Signed-off-by: Antonio Borneo <[email protected]> diff --git a/src/target/riscv/riscv-011.c b/src/target/riscv/riscv-011.c index 15c1c307d7..26189bdc86 100644 --- a/src/target/riscv/riscv-011.c +++ b/src/target/riscv/riscv-011.c @@ -1931,11 +1931,6 @@ static int poll_target(struct target *target, bool announce) { jtag_add_ir_scan(target->tap, &select_dbus, TAP_IDLE); - /* Inhibit debug logging during poll(), which isn't usually interesting and - * just fills up the screen/logs with clutter. */ - int old_debug_level = debug_level; - if (LOG_LEVEL_IS(LOG_LVL_DEBUG)) - debug_level = LOG_LVL_INFO; bits_t bits = { .haltnot = 0, .interrupt = 0 @@ -1943,8 +1938,6 @@ static int poll_target(struct target *target, bool announce) if (read_bits(target, &bits) != ERROR_OK) return ERROR_FAIL; - debug_level = old_debug_level; - if (bits.haltnot && bits.interrupt) { target->state = TARGET_DEBUG_RUNNING; LOG_DEBUG("debug running"); --
