This is an automated email from Gerrit.

Christopher Head (ch...@zaber.com) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/4620

-- gerrit

commit 20708ffdd290b8914644d026874e8dd6528d8ddf
Author: Christopher Head <ch...@zaber.com>
Date:   Tue Jul 31 10:05:36 2018 -0700

    target/cortex_m: fix incorrect comment
    
    The code sets C_MASKINTS if that bit is not already set (correctly). Fix
    the comment to agree.
    
    Change-Id: If4543e2660a9fa2cdabb2d2698427a6c8d9a274c
    Signed-off-by: Christopher Head <ch...@zaber.com>

diff --git a/src/target/cortex_m.c b/src/target/cortex_m.c
index 03dc515..f186946 100644
--- a/src/target/cortex_m.c
+++ b/src/target/cortex_m.c
@@ -165,7 +165,7 @@ static int cortex_m_single_step_core(struct target *target)
        struct armv7m_common *armv7m = &cortex_m->armv7m;
        int retval;
 
-       /* Mask interrupts before clearing halt, if done already.  This avoids
+       /* Mask interrupts before clearing halt, if not done already.  This 
avoids
         * Erratum 377497 (fixed in r1p0) where setting MASKINTS while clearing
         * HALT can put the core into an unknown state.
         */

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to