This is an automated email from Gerrit.

Spencer Oliver (s...@spen-soft.co.uk) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/1634

-- gerrit

commit 9490a7baec3f5dca481a97fc01b0ee92b5c8a7fe
Author: Spencer Oliver <s...@spen-soft.co.uk>
Date:   Thu Sep 19 10:19:57 2013 +0100

    cortex_m: set fpb_enabled on enabling fpb
    
    If the fpb_enabled is not set then as part of cortex_m3_set_breakpoint we
    enable the fpb, however we do not signal the fpb as being enabled.
    
    This issue only effects the hla target as the current cortex_m code enables
    the fpb during cortex_m3_endreset_event.
    
    Change-Id: I44d3fc65916c131b7a226869dd16aed5afb441b4
    Signed-off-by: Spencer Oliver <s...@spen-soft.co.uk>

diff --git a/src/target/cortex_m.c b/src/target/cortex_m.c
index fbe635b..3109910 100644
--- a/src/target/cortex_m.c
+++ b/src/target/cortex_m.c
@@ -1147,6 +1147,7 @@ int cortex_m3_set_breakpoint(struct target *target, 
struct breakpoint *breakpoin
                if (!cortex_m3->fpb_enabled) {
                        LOG_DEBUG("FPB wasn't enabled, do it now");
                        target_write_u32(target, FP_CTRL, 3);
+                       cortex_m3->fpb_enabled = 1;
                }
        } else if (breakpoint->type == BKPT_SOFT) {
                uint8_t code[4];

-- 

------------------------------------------------------------------------------
LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint
2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes
Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13. 
http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to