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/+/9484

-- gerrit

commit 05db71a9e2178bd34200212ff4bd9b1271958c59
Author: Evgeniy Naydanov <[email protected]>
Date:   Wed Feb 25 19:16:38 2026 +0300

    target/xtensa: 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.
    * `xtensa_examine` only calls `xtensa_smpbreak_write()` after setting
      the flag, which in turn does not include `target_was_examined()` in
      it's call graph (see the review comment for details).
    
    Change-Id: Ie5895849a3c69ad8ee75d7b94b059ce532eb99d2
    Signed-off-by: Evgeniy Naydanov <[email protected]>

diff --git a/src/target/xtensa/xtensa.c b/src/target/xtensa/xtensa.c
index f8c36b01d7..350fb2cd15 100644
--- a/src/target/xtensa/xtensa.c
+++ b/src/target/xtensa/xtensa.c
@@ -907,7 +907,6 @@ int xtensa_examine(struct target *target)
                return ERROR_TARGET_FAILURE;
        }
        LOG_DEBUG("OCD_ID = %08" PRIx32, xtensa->dbg_mod.device_id);
-       target_set_examined(target);
        xtensa_smpbreak_write(xtensa, xtensa->smp_break);
        return ERROR_OK;
 }

-- 

Reply via email to