[PATCH]: 9a5dfb1ffc target/arc: allow reading memory while target runs

[gerrit]

This is an automated email from Gerrit.

"Daniel DeGrasse <ddegra...@tenstorrent.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8721

-- gerrit

commit 9a5dfb1ffc41fb38390cfe1439bd0fece98b9d0d
Author: Daniel DeGrasse <ddegra...@tenstorrent.com>
Date:   Thu Jan 16 22:33:48 2025 -0500

    target/arc: allow reading memory while target runs
    
    There is no reason that ARC can't support reading from memory over JTAG
    while the target is executing, and this is in fact required for RTT
    support. Remove the error return when the target is running, and reduce
    the log message to a debug print
    
    Change-Id: I2accfb4b99bf77c5473d133623e0eb0632cb45f6
    Signed-off-by: Daniel DeGrasse <ddegra...@tenstorrent.com>

diff --git a/src/target/arc_mem.c b/src/target/arc_mem.c
index 3daed1cbbf..901f9a5d62 100644
--- a/src/target/arc_mem.c
+++ b/src/target/arc_mem.c
@@ -162,8 +162,7 @@ int arc_mem_write(struct target *target, target_addr_t 
address, uint32_t size,
                address, size, count);
 
        if (target->state != TARGET_HALTED) {
-               LOG_TARGET_ERROR(target, "not halted");
-               return ERROR_TARGET_NOT_HALTED;
+               LOG_TARGET_DEBUG("target not halted");
        }
 
        /* sanitize arguments */
@@ -247,8 +246,7 @@ int arc_mem_read(struct target *target, target_addr_t 
address, uint32_t size,
                        ", count=%" PRIu32, address, size, count);
 
        if (target->state != TARGET_HALTED) {
-               LOG_TARGET_WARNING(target, "target not halted");
-               return ERROR_TARGET_NOT_HALTED;
+               LOG_TARGET_DEBUG("target not halted");
        }
 
        /* Sanitize arguments */

--