This is an automated email from Gerrit.

Paul Fertser ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/2358

-- gerrit

commit 567379d723a9a0122bd752ff12cb7165154dacee
Author: Paul Fertser <[email protected]>
Date:   Mon Oct 27 16:36:32 2014 +0300

    hla_target: declare target as halted after asserting reset
    
    As the current implementation resumes the target on reset deassertion
    when reset_halt is not requested, it should be safe to declare the
    target halted unconditionally after any reset assertion. Unless this
    is done, it's impossible to e.g. add software breakpoints right after
    "reset halt".
    
    Change-Id: I5d3128945c99e4246987ec698161f9dd7f31fab7
    Signed-off-by: Paul Fertser <[email protected]>

diff --git a/src/target/hla_target.c b/src/target/hla_target.c
index efc0313..799e438 100644
--- a/src/target/hla_target.c
+++ b/src/target/hla_target.c
@@ -543,12 +543,8 @@ static int adapter_assert_reset(struct target *target)
        /* registers are now invalid */
        register_cache_invalidate(armv7m->arm.core_cache);
 
-       if (target->reset_halt) {
-               target->state = TARGET_RESET;
-               target->debug_reason = DBG_REASON_DBGRQ;
-       } else {
-               target->state = TARGET_HALTED;
-       }
+       target->state = TARGET_HALTED;
+       target->debug_reason = DBG_REASON_DBGRQ;
 
        return ERROR_OK;
 }

-- 

------------------------------------------------------------------------------
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to