Committed.

The mailing list has been out so there are various other things
that have been fixed as well.



On Wed, Aug 13, 2008 at 5:29 PM, Øyvind Harboe <[EMAIL PROTECTED]> wrote:
>
> These days GDB will have a much better idea of what sort
> of breakpoints to ask OpenOCD to set.
>
> E.g. if the memory map is set up, then GDB will ask GDB to
> use hardware breakpoints.
>
> In ARM7/9 there is currently some code that promotes hardware/software
> breakpoints to hardware or software breakpoints according to
> some more or less nebulous rules.
>
> This promotion belongs in gdb_server.c and not in the target.
>
> Additionally there is a problem with breakpoints not being set up
> correctly because it is unclear when the embedded ice watch
> registers should be set up.
>
> The attached patch will:
>
> - clear the watchpoint registers before the first breakpoint is
> added(gdb_server does this upon every step/resume) and
> after the last breakpoint has been removed.
> - retires the arm7_9 sw/hw breakpoint configuration commands. If
> breakpoint type promotion is to be added back, it belongs in gdb_server.c.
> - there is much less conditional code w.r.t. assumptions about what the
> state of the watch registers is on the target
>
>
> The patch is not done yet, but I'd appreciate any feedback.


-- 
Øyvind Harboe
http://www.zylin.com/zy1000.html
ARM7 ARM9 XScale Cortex
JTAG debugger and flash programmer
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to