From b7811b76795aaeacfea0473bdca2c44826f20501 Mon Sep 17 00:00:00 2001 From: Øyvind Harboe Date: Sun, 21 Mar 2010 19:20:26 +0100 Subject: arm breakpoints: amended fix comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit the handling of caches, should be moved into the breakpoint specific callbacks rather than being plonked into generic memory write fn's. Signed-off-by: Øyvind Harboe --- src/target/arm926ejs.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'src/target/arm926ejs.c') diff --git a/src/target/arm926ejs.c b/src/target/arm926ejs.c index ff18bae8..1f753a6e 100644 --- a/src/target/arm926ejs.c +++ b/src/target/arm926ejs.c @@ -579,7 +579,12 @@ int arm926ejs_write_memory(struct target *target, uint32_t address, /* FIX!!!! this should be cleaned up and made much more general. The * plan is to write up and test on arm926ejs specifically and - * then generalize and clean up afterwards. */ + * then generalize and clean up afterwards. + * + * + * Also it should be moved to the callbacks that handle breakpoints + * specifically and not the generic memory write fn's. See XScale code. + **/ if (arm926ejs->armv4_5_mmu.mmu_enabled && (count == 1) && ((size==2) || (size==4))) { /* special case the handling of single word writes to bypass MMU -- cgit v1.2.3