This is an automated email from Gerrit.

"Evgeniy Naydanov <[email protected]>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/9483

-- gerrit

commit 7ab4b4656a5f19295bac8555ecdf72f93aa208c4
Author: Evgeniy Naydanov <[email protected]>
Date:   Fri Feb 20 19:13:06 2026 +0300

    target/openrisc: drop the call to `target_set_examined()`
    
    This is a valid change, since:
    * The flag is set in the type-independent code on success and reset on
      failure.
    * `or1k_examine` does not include `target_was_examined()` in it's
      call graph (see the review comment for details).
    
    Change-Id: Ifdd79eb12e2a6337f8224469c8621484fa14c25d
    Signed-off-by: Evgeniy Naydanov <[email protected]>
    
    Change-Id: I1e4670d00df186d24ef59bde4826691607fda59f
    Signed-off-by: Evgeniy Naydanov <[email protected]>

diff --git a/src/target/openrisc/or1k.c b/src/target/openrisc/or1k.c
index 4aa2bd7347..eab064140e 100644
--- a/src/target/openrisc/or1k.c
+++ b/src/target/openrisc/or1k.c
@@ -1124,8 +1124,6 @@ static int or1k_examine(struct target *target)
 
        if (!target_was_examined(target)) {
 
-               target_set_examined(target);
-
                int running;
 
                int retval = du_core->or1k_is_cpu_running(&or1k->jtag, 
&running);

-- 

Reply via email to