arm920t: fix write memory operations with caches enabled
authorPaul Fertser <fercerpav@gmail.com>
Tue, 26 Nov 2013 15:46:09 +0000 (19:46 +0400)
committerSpencer Oliver <spen@spen-soft.co.uk>
Fri, 24 Jan 2014 12:48:43 +0000 (12:48 +0000)
Commit ff5ec942d80a34e20b5a3ca3328f7e6a55fb309b made this target
always use generic arm7_9 memory write routines for software
breakpoints which resulted in inability to debug and single-step
sources in Gdb when icache is active as generic routine doesn't
invalidate it. This should fix it (and is real-life tested against
Samsung S3C2442). I expect other arm7-9 targets to be affected as
well.

Change-Id: Id7980e370ae4db47ac6b1490321d81ffe85711c0
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1817
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
src/target/arm920t.c

index fbfa170369e7514bf3d4ae7dd1365a4b810ad84f..f6a0f5bf81d511137ae13356e435e4656deb87d2 100644 (file)
@@ -823,6 +823,7 @@ static int arm920t_init_arch_info(struct target *target,
 
        arm7_9->post_debug_entry = arm920t_post_debug_entry;
        arm7_9->pre_restore_context = arm920t_pre_restore_context;
+       arm7_9->write_memory = arm920t_write_memory;
 
        arm920t->armv4_5_mmu.armv4_5_cache.ctype = -1;
        arm920t->armv4_5_mmu.get_ttb = arm920t_get_ttb;