cortex_a: fix FTBS on ARM due to alignment issues
[openocd.git] / src / target / cortex_a.c
old mode 100755 (executable)
new mode 100644 (file)
index c9401af..cc6ae99
@@ -17,6 +17,9 @@
  *   Copyright (C) ST-Ericsson SA 2011                                     *
  *   michel.jaouen@stericsson.com : smp minimum support                    *
  *                                                                         *
+ *   Copyright (C) Broadcom 2012                                           *
+ *   ehunter@broadcom.com : Cortex R4 support                              *
+ *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
  *   it under the terms of the GNU General Public License as published by  *
  *   the Free Software Foundation; either version 2 of the License, or     *
  *                                                                         *
  *   Cortex-A8(tm) TRM, ARM DDI 0344H                                      *
  *   Cortex-A9(tm) TRM, ARM DDI 0407F                                      *
+ *   Cortex-A4(tm) TRM, ARM DDI 0363E                                      *
  *                                                                         *
  ***************************************************************************/
+
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
@@ -52,44 +57,37 @@ static int cortex_a8_poll(struct target *target);
 static int cortex_a8_debug_entry(struct target *target);
 static int cortex_a8_restore_context(struct target *target, bool bpwp);
 static int cortex_a8_set_breakpoint(struct target *target,
-               struct breakpoint *breakpoint, uint8_t matchmode);
+       struct breakpoint *breakpoint, uint8_t matchmode);
 static int cortex_a8_set_context_breakpoint(struct target *target,
-               struct breakpoint *breakpoint, uint8_t matchmode);
+       struct breakpoint *breakpoint, uint8_t matchmode);
 static int cortex_a8_set_hybrid_breakpoint(struct target *target,
-               struct breakpoint *breakpoint);
+       struct breakpoint *breakpoint);
 static int cortex_a8_unset_breakpoint(struct target *target,
-               struct breakpoint *breakpoint);
+       struct breakpoint *breakpoint);
 static int cortex_a8_dap_read_coreregister_u32(struct target *target,
-               uint32_t *value, int regnum);
+       uint32_t *value, int regnum);
 static int cortex_a8_dap_write_coreregister_u32(struct target *target,
-               uint32_t value, int regnum);
+       uint32_t value, int regnum);
 static int cortex_a8_mmu(struct target *target, int *enabled);
 static int cortex_a8_virt2phys(struct target *target,
-                uint32_t virt, uint32_t *phys);
+       uint32_t virt, uint32_t *phys);
+static int cortex_a8_read_apb_ab_memory(struct target *target,
+       uint32_t address, uint32_t size, uint32_t count, uint8_t *buffer);
 
-/*
- * FIXME do topology discovery using the ROM; don't
- * assume this is an OMAP3.   Also, allow for multiple ARMv7-A
- * cores, with different AP numbering ... don't use a #define
- * for these numbers, use per-core armv7a state.
- */
-#define swjdp_memoryap 0
-#define swjdp_debugap 1
 
 /*  restore cp15_control_reg at resume */
-static int cortex_a8_restore_cp15_control_reg(struct targettarget)
+static int cortex_a8_restore_cp15_control_reg(struct target *target)
 {
        int retval = ERROR_OK;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
-    struct armv7a_common *armv7a = target_to_armv7a(target);
+       struct armv7a_common *armv7a = target_to_armv7a(target);
 
-       if (cortex_a8->cp15_control_reg !=cortex_a8->cp15_control_reg_curr)
-       {
+       if (cortex_a8->cp15_control_reg != cortex_a8->cp15_control_reg_curr) {
                cortex_a8->cp15_control_reg_curr = cortex_a8->cp15_control_reg;
-               //LOG_INFO("cp15_control_reg: %8.8" PRIx32, cortex_a8->cp15_control_reg);
-               retval = armv7a->armv4_5_common.mcr(target, 15,
-                               0, 0,   /* op1, op2 */
-                               1, 0,   /* CRn, CRm */
+               /* LOG_INFO("cp15_control_reg: %8.8" PRIx32, cortex_a8->cp15_control_reg); */
+               retval = armv7a->arm.mcr(target, 15,
+                               0, 0,   /* op1, op2 */
+                               1, 0,   /* CRn, CRm */
                                cortex_a8->cp15_control_reg);
        }
        return retval;
@@ -103,22 +101,22 @@ static int cortex_a8_check_address(struct target *target, uint32_t address)
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        uint32_t os_border = armv7a->armv7a_mmu.os_border;
        if ((address < os_border) &&
-                       (armv7a->armv4_5_common.core_mode == ARM_MODE_SVC)){
-               LOG_ERROR("%x access in userspace and target in supervisor",address);
+               (armv7a->arm.core_mode == ARM_MODE_SVC)) {
+               LOG_ERROR("%x access in userspace and target in supervisor", address);
                return ERROR_FAIL;
        }
        if ((address >= os_border) &&
-                       ( cortex_a8->curr_mode != ARM_MODE_SVC)){
-        dpm_modeswitch(&armv7a->dpm, ARM_MODE_SVC);
-        cortex_a8->curr_mode = ARM_MODE_SVC;
+               (cortex_a8->curr_mode != ARM_MODE_SVC)) {
+               dpm_modeswitch(&armv7a->dpm, ARM_MODE_SVC);
+               cortex_a8->curr_mode = ARM_MODE_SVC;
                LOG_INFO("%x access in kernel space and target not in supervisor",
-                               address);
+                       address);
                return ERROR_OK;
        }
        if ((address < os_border) &&
-                       (cortex_a8->curr_mode == ARM_MODE_SVC)){
-        dpm_modeswitch(&armv7a->dpm, ARM_MODE_ANY);
-     cortex_a8->curr_mode = ARM_MODE_ANY;
+               (cortex_a8->curr_mode == ARM_MODE_SVC)) {
+               dpm_modeswitch(&armv7a->dpm, ARM_MODE_ANY);
+               cortex_a8->curr_mode = ARM_MODE_ANY;
        }
        return ERROR_OK;
 }
@@ -130,39 +128,32 @@ static int cortex_a8_mmu_modify(struct target *target, int enable)
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = target_to_armv7a(target);
        int retval = ERROR_OK;
-       if (enable)
-       {
+       if (enable) {
                /*  if mmu enabled at target stop and mmu not enable */
-               if (!(cortex_a8->cp15_control_reg & 0x1U))
-               {
+               if (!(cortex_a8->cp15_control_reg & 0x1U)) {
                        LOG_ERROR("trying to enable mmu on target stopped with mmu disable");
                        return ERROR_FAIL;
                }
-               if (!(cortex_a8->cp15_control_reg_curr & 0x1U))
-               {
+               if (!(cortex_a8->cp15_control_reg_curr & 0x1U)) {
                        cortex_a8->cp15_control_reg_curr |= 0x1U;
-                       retval = armv7a->armv4_5_common.mcr(target, 15,
-                                       0, 0,   /* op1, op2 */
-                                       1, 0,   /* CRn, CRm */
+                       retval = armv7a->arm.mcr(target, 15,
+                                       0, 0,   /* op1, op2 */
+                                       1, 0,   /* CRn, CRm */
                                        cortex_a8->cp15_control_reg_curr);
                }
-       }
-       else
-       {
-               if (cortex_a8->cp15_control_reg_curr & 0x4U)
-               {
-                   /*  data cache is active */
+       } else {
+               if (cortex_a8->cp15_control_reg_curr & 0x4U) {
+                       /*  data cache is active */
                        cortex_a8->cp15_control_reg_curr &= ~0x4U;
                        /* flush data cache armv7 function to be called */
                        if (armv7a->armv7a_mmu.armv7a_cache.flush_all_data_cache)
-                       armv7a->armv7a_mmu.armv7a_cache.flush_all_data_cache(target);
+                               armv7a->armv7a_mmu.armv7a_cache.flush_all_data_cache(target);
                }
-               if ( (cortex_a8->cp15_control_reg_curr & 0x1U))
-               {
+               if ((cortex_a8->cp15_control_reg_curr & 0x1U)) {
                        cortex_a8->cp15_control_reg_curr &= ~0x1U;
-                       retval = armv7a->armv4_5_common.mcr(target, 15,
-                                       0, 0,   /* op1, op2 */
-                                       1, 0,   /* CRn, CRm */
+                       retval = armv7a->arm.mcr(target, 15,
+                                       0, 0,   /* op1, op2 */
+                                       1, 0,   /* CRn, CRm */
                                        cortex_a8->cp15_control_reg_curr);
                }
        }
@@ -175,32 +166,30 @@ static int cortex_a8_mmu_modify(struct target *target, int enable)
 static int cortex_a8_init_debug_access(struct target *target)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int retval;
        uint32_t dummy;
 
        LOG_DEBUG(" ");
 
-       /* Unlocking the debug registers for modification */
-       /* The debugport might be uninitialised so try twice */
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
+       /* Unlocking the debug registers for modification
+        * The debugport might be uninitialised so try twice */
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_LOCKACCESS, 0xC5ACCE55);
-       if (retval != ERROR_OK)
-       {
+       if (retval != ERROR_OK) {
                /* try again */
-               retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
+               retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
                                armv7a->debug_base + CPUDBG_LOCKACCESS, 0xC5ACCE55);
                if (retval == ERROR_OK)
-               {
-                       LOG_USER("Locking debug access failed on first, but succeeded on second try.");
-               }
+                       LOG_USER(
+                               "Locking debug access failed on first, but succeeded on second try.");
        }
        if (retval != ERROR_OK)
                return retval;
        /* Clear Sticky Power Down status Bit in PRSR to enable access to
           the registers in the Core Power Domain */
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                               armv7a->debug_base + CPUDBG_PRSR, &dummy);
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_PRSR, &dummy);
        if (retval != ERROR_OK)
                return retval;
 
@@ -218,12 +207,12 @@ static int cortex_a8_init_debug_access(struct target *target)
  * happen to know that no instruction is pending.
  */
 static int cortex_a8_exec_opcode(struct target *target,
-               uint32_t opcode, uint32_t *dscr_p)
+       uint32_t opcode, uint32_t *dscr_p)
 {
        uint32_t dscr;
        int retval;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
        dscr = dscr_p ? *dscr_p : 0;
 
@@ -231,44 +220,37 @@ static int cortex_a8_exec_opcode(struct target *target,
 
        /* Wait for InstrCompl bit to be set */
        long long then = timeval_ms();
-       while ((dscr & DSCR_INSTR_COMP) == 0)
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       while ((dscr & DSCR_INSTR_COMP) == 0) {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                                armv7a->debug_base + CPUDBG_DSCR, &dscr);
-               if (retval != ERROR_OK)
-               {
+               if (retval != ERROR_OK) {
                        LOG_ERROR("Could not read DSCR register, opcode = 0x%08" PRIx32, opcode);
                        return retval;
                }
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for cortex_a8_exec_opcode");
                        return ERROR_FAIL;
                }
        }
 
-       retval = mem_ap_sel_write_u32(swjdp, swjdp_debugap,
-                               armv7a->debug_base + CPUDBG_ITR, opcode);
+       retval = mem_ap_sel_write_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_ITR, opcode);
        if (retval != ERROR_OK)
                return retval;
 
        then = timeval_ms();
-       do
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       do {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                                armv7a->debug_base + CPUDBG_DSCR, &dscr);
-               if (retval != ERROR_OK)
-               {
+               if (retval != ERROR_OK) {
                        LOG_ERROR("Could not read DSCR register");
                        return retval;
                }
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for cortex_a8_exec_opcode");
                        return ERROR_FAIL;
                }
-       }
-       while ((dscr & DSCR_INSTR_COMP) == 0); /* Wait for InstrCompl bit to be set */
+       } while ((dscr & DSCR_INSTR_COMP) == 0);        /* Wait for InstrCompl bit to be set */
 
        if (dscr_p)
                *dscr_p = dscr;
@@ -281,11 +263,11 @@ Read core register with very few exec_opcode, fast but needs work_area.
 This can cause problems with MMU active.
 **************************************************************************/
 static int cortex_a8_read_regs_through_mem(struct target *target, uint32_t address,
-               uint32_t * regfile)
+       uint32_t *regfile)
 {
        int retval = ERROR_OK;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
        retval = cortex_a8_dap_read_coreregister_u32(target, regfile, 0);
        if (retval != ERROR_OK)
@@ -297,35 +279,32 @@ static int cortex_a8_read_regs_through_mem(struct target *target, uint32_t addre
        if (retval != ERROR_OK)
                return retval;
 
-       retval = mem_ap_sel_read_buf_u32(swjdp, swjdp_memoryap,
+       retval = mem_ap_sel_read_buf_u32(swjdp, armv7a->memory_ap,
                        (uint8_t *)(&regfile[1]), 4*15, address);
 
        return retval;
 }
 
 static int cortex_a8_dap_read_coreregister_u32(struct target *target,
-               uint32_t *value, int regnum)
+       uint32_t *value, int regnum)
 {
        int retval = ERROR_OK;
        uint8_t reg = regnum&0xFF;
        uint32_t dscr = 0;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
        if (reg > 17)
                return retval;
 
-       if (reg < 15)
-       {
+       if (reg < 15) {
                /* Rn to DCCTX, "MCR p14, 0, Rn, c0, c5, 0"  0xEE00nE15 */
                retval = cortex_a8_exec_opcode(target,
                                ARMV4_5_MCR(14, 0, reg, 0, 5, 0),
                                &dscr);
                if (retval != ERROR_OK)
                        return retval;
-       }
-       else if (reg == 15)
-       {
+       } else if (reg == 15) {
                /* "MOV r0, r15"; then move r0 to DCCTX */
                retval = cortex_a8_exec_opcode(target, 0xE1A0000F, &dscr);
                if (retval != ERROR_OK)
@@ -335,9 +314,7 @@ static int cortex_a8_dap_read_coreregister_u32(struct target *target,
                                &dscr);
                if (retval != ERROR_OK)
                        return retval;
-       }
-       else
-       {
+       } else {
                /* "MRS r0, CPSR" or "MRS r0, SPSR"
                 * then move r0 to DCCTX
                 */
@@ -353,20 +330,18 @@ static int cortex_a8_dap_read_coreregister_u32(struct target *target,
 
        /* Wait for DTRRXfull then read DTRRTX */
        long long then = timeval_ms();
-       while ((dscr & DSCR_DTR_TX_FULL) == 0)
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       while ((dscr & DSCR_DTR_TX_FULL) == 0) {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                                armv7a->debug_base + CPUDBG_DSCR, &dscr);
                if (retval != ERROR_OK)
                        return retval;
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for cortex_a8_exec_opcode");
                        return ERROR_FAIL;
                }
        }
 
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DTRTX, value);
        LOG_DEBUG("read DCC 0x%08" PRIx32, *value);
 
@@ -374,23 +349,22 @@ static int cortex_a8_dap_read_coreregister_u32(struct target *target,
 }
 
 static int cortex_a8_dap_write_coreregister_u32(struct target *target,
-               uint32_t value, int regnum)
+       uint32_t value, int regnum)
 {
        int retval = ERROR_OK;
        uint8_t Rd = regnum&0xFF;
        uint32_t dscr;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
        LOG_DEBUG("register %i, value 0x%08" PRIx32, regnum, value);
 
        /* Check that DCCRX is not full */
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
        if (retval != ERROR_OK)
                return retval;
-       if (dscr & DSCR_DTR_RX_FULL)
-       {
+       if (dscr & DSCR_DTR_RX_FULL) {
                LOG_ERROR("DSCR_DTR_RX_FULL, dscr 0x%08" PRIx32, dscr);
                /* Clear DCCRX with MRC(p14, 0, Rd, c0, c5, 0), opcode  0xEE100E15 */
                retval = cortex_a8_exec_opcode(target, ARMV4_5_MRC(14, 0, 0, 0, 5, 0),
@@ -404,22 +378,19 @@ static int cortex_a8_dap_write_coreregister_u32(struct target *target,
 
        /* Write DTRRX ... sets DSCR.DTRRXfull but exec_opcode() won't care */
        LOG_DEBUG("write DCC 0x%08" PRIx32, value);
-       retval = mem_ap_sel_write_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_write_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DTRRX, value);
        if (retval != ERROR_OK)
                return retval;
 
-       if (Rd < 15)
-       {
+       if (Rd < 15) {
                /* DCCRX to Rn, "MRC p14, 0, Rn, c0, c5, 0", 0xEE10nE15 */
                retval = cortex_a8_exec_opcode(target, ARMV4_5_MRC(14, 0, Rd, 0, 5, 0),
                                &dscr);
-       
+
                if (retval != ERROR_OK)
                        return retval;
-       }
-       else if (Rd == 15)
-       {
+       } else if (Rd == 15) {
                /* DCCRX to R0, "MRC p14, 0, R0, c0, c5, 0", 0xEE100E15
                 * then "mov r15, r0"
                 */
@@ -430,9 +401,7 @@ static int cortex_a8_dap_write_coreregister_u32(struct target *target,
                retval = cortex_a8_exec_opcode(target, 0xE1A0F000, &dscr);
                if (retval != ERROR_OK)
                        return retval;
-       }
-       else
-       {
+       } else {
                /* DCCRX to R0, "MRC p14, 0, R0, c0, c5, 0", 0xEE100E15
                 * then "MSR CPSR_cxsf, r0" or "MSR SPSR_cxsf, r0" (all fields)
                 */
@@ -446,8 +415,7 @@ static int cortex_a8_dap_write_coreregister_u32(struct target *target,
                        return retval;
 
                /* "Prefetch flush" after modifying execution status in CPSR */
-               if (Rd == 16)
-               {
+               if (Rd == 16) {
                        retval = cortex_a8_exec_opcode(target,
                                        ARMV4_5_MCR(15, 0, 0, 7, 5, 4),
                                        &dscr);
@@ -460,13 +428,15 @@ static int cortex_a8_dap_write_coreregister_u32(struct target *target,
 }
 
 /* Write to memory mapped registers directly with no cache or mmu handling */
-static int cortex_a8_dap_write_memap_register_u32(struct target *target, uint32_t address, uint32_t value)
+static int cortex_a8_dap_write_memap_register_u32(struct target *target,
+       uint32_t address,
+       uint32_t value)
 {
        int retval;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap, address, value);
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap, address, value);
 
        return retval;
 }
@@ -490,14 +460,14 @@ static inline struct cortex_a8_common *dpm_to_a8(struct arm_dpm *dpm)
 static int cortex_a8_write_dcc(struct cortex_a8_common *a8, uint32_t data)
 {
        LOG_DEBUG("write DCC 0x%08" PRIx32, data);
-       return mem_ap_sel_write_u32(a8->armv7a_common.armv4_5_common.dap,
-                       swjdp_debugap,a8->armv7a_common.debug_base + CPUDBG_DTRRX, data);
+       return mem_ap_sel_write_u32(a8->armv7a_common.arm.dap,
+               a8->armv7a_common.debug_ap, a8->armv7a_common.debug_base + CPUDBG_DTRRX, data);
 }
 
 static int cortex_a8_read_dcc(struct cortex_a8_common *a8, uint32_t *data,
-               uint32_t *dscr_p)
+       uint32_t *dscr_p)
 {
-       struct adiv5_dap *swjdp = a8->armv7a_common.armv4_5_common.dap;
+       struct adiv5_dap *swjdp = a8->armv7a_common.arm.dap;
        uint32_t dscr = DSCR_INSTR_COMP;
        int retval;
 
@@ -507,23 +477,22 @@ static int cortex_a8_read_dcc(struct cortex_a8_common *a8, uint32_t *data,
        /* Wait for DTRRXfull */
        long long then = timeval_ms();
        while ((dscr & DSCR_DTR_TX_FULL) == 0) {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+               retval = mem_ap_sel_read_atomic_u32(swjdp, a8->armv7a_common.debug_ap,
                                a8->armv7a_common.debug_base + CPUDBG_DSCR,
                                &dscr);
                if (retval != ERROR_OK)
                        return retval;
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for read dcc");
                        return ERROR_FAIL;
                }
        }
 
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_read_atomic_u32(swjdp, a8->armv7a_common.debug_ap,
                        a8->armv7a_common.debug_base + CPUDBG_DTRTX, data);
        if (retval != ERROR_OK)
                return retval;
-       //LOG_DEBUG("read DCC 0x%08" PRIx32, *data);
+       /* LOG_DEBUG("read DCC 0x%08" PRIx32, *data); */
 
        if (dscr_p)
                *dscr_p = dscr;
@@ -534,23 +503,21 @@ static int cortex_a8_read_dcc(struct cortex_a8_common *a8, uint32_t *data,
 static int cortex_a8_dpm_prepare(struct arm_dpm *dpm)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
-       struct adiv5_dap *swjdp = a8->armv7a_common.armv4_5_common.dap;
+       struct adiv5_dap *swjdp = a8->armv7a_common.arm.dap;
        uint32_t dscr;
        int retval;
 
        /* set up invariant:  INSTR_COMP is set after ever DPM operation */
        long long then = timeval_ms();
-       for (;;)
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       for (;; ) {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, a8->armv7a_common.debug_ap,
                                a8->armv7a_common.debug_base + CPUDBG_DSCR,
                                &dscr);
                if (retval != ERROR_OK)
                        return retval;
                if ((dscr & DSCR_INSTR_COMP) != 0)
                        break;
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for dpm prepare");
                        return ERROR_FAIL;
                }
@@ -561,7 +528,7 @@ static int cortex_a8_dpm_prepare(struct arm_dpm *dpm)
                LOG_ERROR("DSCR_DTR_RX_FULL, dscr 0x%08" PRIx32, dscr);
                /* Clear DCCRX */
                retval = cortex_a8_exec_opcode(
-                               a8->armv7a_common.armv4_5_common.target,
+                               a8->armv7a_common.arm.target,
                                ARMV4_5_MRC(14, 0, 0, 0, 5, 0),
                                &dscr);
                if (retval != ERROR_OK)
@@ -578,7 +545,7 @@ static int cortex_a8_dpm_finish(struct arm_dpm *dpm)
 }
 
 static int cortex_a8_instr_write_data_dcc(struct arm_dpm *dpm,
-               uint32_t opcode, uint32_t data)
+       uint32_t opcode, uint32_t data)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
        int retval;
@@ -589,13 +556,13 @@ static int cortex_a8_instr_write_data_dcc(struct arm_dpm *dpm,
                return retval;
 
        return cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        opcode,
                        &dscr);
 }
 
 static int cortex_a8_instr_write_data_r0(struct arm_dpm *dpm,
-               uint32_t opcode, uint32_t data)
+       uint32_t opcode, uint32_t data)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
        uint32_t dscr = DSCR_INSTR_COMP;
@@ -607,7 +574,7 @@ static int cortex_a8_instr_write_data_r0(struct arm_dpm *dpm,
 
        /* DCCRX to R0, "MCR p14, 0, R0, c0, c5, 0", 0xEE000E15 */
        retval = cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        ARMV4_5_MRC(14, 0, 0, 0, 5, 0),
                        &dscr);
        if (retval != ERROR_OK)
@@ -615,7 +582,7 @@ static int cortex_a8_instr_write_data_r0(struct arm_dpm *dpm,
 
        /* then the opcode, taking data from R0 */
        retval = cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        opcode,
                        &dscr);
 
@@ -634,7 +601,7 @@ static int cortex_a8_instr_cpsr_sync(struct arm_dpm *dpm)
 }
 
 static int cortex_a8_instr_read_data_dcc(struct arm_dpm *dpm,
-               uint32_t opcode, uint32_t *data)
+       uint32_t opcode, uint32_t *data)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
        int retval;
@@ -642,7 +609,7 @@ static int cortex_a8_instr_read_data_dcc(struct arm_dpm *dpm,
 
        /* the opcode, writing data to DCC */
        retval = cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        opcode,
                        &dscr);
        if (retval != ERROR_OK)
@@ -653,7 +620,7 @@ static int cortex_a8_instr_read_data_dcc(struct arm_dpm *dpm,
 
 
 static int cortex_a8_instr_read_data_r0(struct arm_dpm *dpm,
-               uint32_t opcode, uint32_t *data)
+       uint32_t opcode, uint32_t *data)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
        uint32_t dscr = DSCR_INSTR_COMP;
@@ -661,7 +628,7 @@ static int cortex_a8_instr_read_data_r0(struct arm_dpm *dpm,
 
        /* the opcode, writing data to R0 */
        retval = cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        opcode,
                        &dscr);
        if (retval != ERROR_OK)
@@ -669,7 +636,7 @@ static int cortex_a8_instr_read_data_r0(struct arm_dpm *dpm,
 
        /* write R0 to DCC */
        retval = cortex_a8_exec_opcode(
-                       a8->armv7a_common.armv4_5_common.target,
+                       a8->armv7a_common.arm.target,
                        ARMV4_5_MCR(14, 0, 0, 0, 5, 0),
                        &dscr);
        if (retval != ERROR_OK)
@@ -679,7 +646,7 @@ static int cortex_a8_instr_read_data_r0(struct arm_dpm *dpm,
 }
 
 static int cortex_a8_bpwp_enable(struct arm_dpm *dpm, unsigned index_t,
-               uint32_t addr, uint32_t control)
+       uint32_t addr, uint32_t control)
 {
        struct cortex_a8_common *a8 = dpm_to_a8(dpm);
        uint32_t vr = a8->armv7a_common.debug_base;
@@ -687,23 +654,23 @@ static int cortex_a8_bpwp_enable(struct arm_dpm *dpm, unsigned index_t,
        int retval;
 
        switch (index_t) {
-       case 0 ... 15:          /* breakpoints */
-               vr += CPUDBG_BVR_BASE;
-               cr += CPUDBG_BCR_BASE;
-               break;
-       case 16 ... 31:         /* watchpoints */
-               vr += CPUDBG_WVR_BASE;
-               cr += CPUDBG_WCR_BASE;
-               index_t -= 16;
-               break;
-       default:
-               return ERROR_FAIL;
+               case 0 ... 15:  /* breakpoints */
+                       vr += CPUDBG_BVR_BASE;
+                       cr += CPUDBG_BCR_BASE;
+                       break;
+               case 16 ... 31: /* watchpoints */
+                       vr += CPUDBG_WVR_BASE;
+                       cr += CPUDBG_WCR_BASE;
+                       index_t -= 16;
+                       break;
+               default:
+                       return ERROR_FAIL;
        }
        vr += 4 * index_t;
        cr += 4 * index_t;
 
        LOG_DEBUG("A8: bpwp enable, vr %08x cr %08x",
-                       (unsigned) vr, (unsigned) cr);
+               (unsigned) vr, (unsigned) cr);
 
        retval = cortex_a8_dap_write_memap_register_u32(dpm->arm->target,
                        vr, addr);
@@ -720,15 +687,15 @@ static int cortex_a8_bpwp_disable(struct arm_dpm *dpm, unsigned index_t)
        uint32_t cr;
 
        switch (index_t) {
-       case 0 ... 15:
-               cr = a8->armv7a_common.debug_base + CPUDBG_BCR_BASE;
-               break;
-       case 16 ... 31:
-               cr = a8->armv7a_common.debug_base + CPUDBG_WCR_BASE;
-               index_t -= 16;
-               break;
-       default:
-               return ERROR_FAIL;
+               case 0 ... 15:
+                       cr = a8->armv7a_common.debug_base + CPUDBG_BCR_BASE;
+                       break;
+               case 16 ... 31:
+                       cr = a8->armv7a_common.debug_base + CPUDBG_WCR_BASE;
+                       index_t -= 16;
+                       break;
+               default:
+                       return ERROR_FAIL;
        }
        cr += 4 * index_t;
 
@@ -743,7 +710,7 @@ static int cortex_a8_dpm_setup(struct cortex_a8_common *a8, uint32_t didr)
        struct arm_dpm *dpm = &a8->armv7a_common.dpm;
        int retval;
 
-       dpm->arm = &a8->armv7a_common.armv4_5_common;
+       dpm->arm = &a8->armv7a_common.arm;
        dpm->didr = didr;
 
        dpm->prepare = cortex_a8_dpm_prepare;
@@ -767,20 +734,17 @@ static int cortex_a8_dpm_setup(struct cortex_a8_common *a8, uint32_t didr)
 }
 static struct target *get_cortex_a8(struct target *target, int32_t coreid)
 {
-struct target_list *head;
-struct target *curr;
+       struct target_list *head;
+       struct target *curr;
 
        head = target->head;
-       while(head != (struct target_list*)NULL)
-       {
+       while (head != (struct target_list *)NULL) {
                curr = head->target;
                if ((curr->coreid == coreid) && (curr->state == TARGET_HALTED))
-               {
-        return curr;
-               }
+                       return curr;
                head = head->next;
        }
-   return target;
+       return target;
 }
 static int cortex_a8_halt(struct target *target);
 
@@ -790,13 +754,10 @@ static int cortex_a8_halt_smp(struct target *target)
        struct target_list *head;
        struct target *curr;
        head = target->head;
-       while(head != (struct target_list*)NULL)
-       {
+       while (head != (struct target_list *)NULL) {
                curr = head->target;
-               if ((curr != target) && (curr->state!= TARGET_HALTED))
-               {
+               if ((curr != target) && (curr->state != TARGET_HALTED))
                        retval += cortex_a8_halt(curr);
-               }
                head = head->next;
        }
        return retval;
@@ -805,8 +766,7 @@ static int cortex_a8_halt_smp(struct target *target)
 static int update_halt_gdb(struct target *target)
 {
        int retval = 0;
-       if (target->gdb_service->core[0]==-1)
-       {
+       if (target->gdb_service->core[0] == -1) {
                target->gdb_service->target = target;
                target->gdb_service->core[0] = target->coreid;
                retval += cortex_a8_halt_smp(target);
@@ -824,77 +784,64 @@ static int cortex_a8_poll(struct target *target)
        uint32_t dscr;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        enum target_state prev_target_state = target->state;
-       //  toggle to another core is done by gdb as follow
-       //  maint packet J core_id
-       //  continue
-       //  the next polling trigger an halt event sent to gdb
+       /*  toggle to another core is done by gdb as follow */
+       /*  maint packet J core_id */
+       /*  continue */
+       /*  the next polling trigger an halt event sent to gdb */
        if ((target->state == TARGET_HALTED) && (target->smp) &&
-                       (target->gdb_service) &&
-                       (target->gdb_service->target==NULL) )
-       {
+               (target->gdb_service) &&
+               (target->gdb_service->target == NULL)) {
                target->gdb_service->target =
                        get_cortex_a8(target, target->gdb_service->core[1]);
-               target_call_event_callbacks(target,
-                               TARGET_EVENT_HALTED);
+               target_call_event_callbacks(target, TARGET_EVENT_HALTED);
                return retval;
        }
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DSCR, &dscr);
        if (retval != ERROR_OK)
-       {
                return retval;
-       }
        cortex_a8->cpudbg_dscr = dscr;
 
-       if (DSCR_RUN_MODE(dscr) == (DSCR_CORE_HALTED | DSCR_CORE_RESTARTED))
-       {
-               if (prev_target_state != TARGET_HALTED)
-               {
+       if (DSCR_RUN_MODE(dscr) == (DSCR_CORE_HALTED | DSCR_CORE_RESTARTED)) {
+               if (prev_target_state != TARGET_HALTED) {
                        /* We have a halting debug event */
                        LOG_DEBUG("Target halted");
                        target->state = TARGET_HALTED;
                        if ((prev_target_state == TARGET_RUNNING)
-                                       || (prev_target_state == TARGET_RESET))
-                       {
+                               || (prev_target_state == TARGET_UNKNOWN)
+                               || (prev_target_state == TARGET_RESET)) {
                                retval = cortex_a8_debug_entry(target);
                                if (retval != ERROR_OK)
                                        return retval;
-                               if (target->smp)
-                               {
+                               if (target->smp) {
                                        retval = update_halt_gdb(target);
                                        if (retval != ERROR_OK)
                                                return retval;
                                }
                                target_call_event_callbacks(target,
-                                               TARGET_EVENT_HALTED);
+                                       TARGET_EVENT_HALTED);
                        }
-                       if (prev_target_state == TARGET_DEBUG_RUNNING)
-                       {
+                       if (prev_target_state == TARGET_DEBUG_RUNNING) {
                                LOG_DEBUG(" ");
 
                                retval = cortex_a8_debug_entry(target);
                                if (retval != ERROR_OK)
                                        return retval;
-                               if (target->smp)
-                               {
+                               if (target->smp) {
                                        retval = update_halt_gdb(target);
                                        if (retval != ERROR_OK)
                                                return retval;
                                }
 
                                target_call_event_callbacks(target,
-                                               TARGET_EVENT_DEBUG_HALTED);
+                                       TARGET_EVENT_DEBUG_HALTED);
                        }
                }
-       }
-       else if (DSCR_RUN_MODE(dscr) == DSCR_CORE_RESTARTED)
-       {
+       } else if (DSCR_RUN_MODE(dscr) == DSCR_CORE_RESTARTED)
                target->state = TARGET_RUNNING;
-       }
-       else
-       {
+       else {
                LOG_DEBUG("Unknown target state dscr = 0x%08" PRIx32, dscr);
                target->state = TARGET_UNKNOWN;
        }
@@ -907,13 +854,13 @@ static int cortex_a8_halt(struct target *target)
        int retval = ERROR_OK;
        uint32_t dscr;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
 
        /*
         * Tell the core to be halted by writing DRCR with 0x1
         * and then wait for the core to be halted.
         */
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DRCR, DRCR_HALT);
        if (retval != ERROR_OK)
                return retval;
@@ -921,29 +868,25 @@ static int cortex_a8_halt(struct target *target)
        /*
         * enter halting debug mode
         */
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
        if (retval != ERROR_OK)
                return retval;
 
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
-               armv7a->debug_base + CPUDBG_DSCR, dscr | DSCR_HALT_DBG_MODE);
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, dscr | DSCR_HALT_DBG_MODE);
        if (retval != ERROR_OK)
                return retval;
 
        long long then = timeval_ms();
-       for (;;)
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       for (;; ) {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
                if (retval != ERROR_OK)
                        return retval;
                if ((dscr & DSCR_CORE_HALTED) != 0)
-               {
                        break;
-               }
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for halt");
                        return ERROR_FAIL;
                }
@@ -955,10 +898,10 @@ static int cortex_a8_halt(struct target *target)
 }
 
 static int cortex_a8_internal_restore(struct target *target, int current,
-               uint32_t *address, int handle_breakpoints, int debug_execution)
+       uint32_t *address, int handle_breakpoints, int debug_execution)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
+       struct arm *arm = &armv7a->arm;
        int retval;
        uint32_t resume_pc;
 
@@ -966,8 +909,7 @@ static int cortex_a8_internal_restore(struct target *target, int current,
                target_free_all_working_areas(target);
 
 #if 0
-       if (debug_execution)
-       {
+       if (debug_execution) {
                /* Disable interrupts */
                /* We disable interrupts in the PRIMASK register instead of
                 * masking with C_MASKINTS,
@@ -980,14 +922,15 @@ static int cortex_a8_internal_restore(struct target *target, int current,
 
                /* Make sure we are in Thumb mode */
                buf_set_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0, 32,
-                       buf_get_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0, 32) | (1 << 24));
+                       buf_get_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0,
+                       32) | (1 << 24));
                armv7m->core_cache->reg_list[ARMV7M_xPSR].dirty = 1;
                armv7m->core_cache->reg_list[ARMV7M_xPSR].valid = 1;
        }
 #endif
 
        /* current = 1: continue on current pc, otherwise continue at <address> */
-       resume_pc = buf_get_u32(armv4_5->pc->value, 0, 32);
+       resume_pc = buf_get_u32(arm->pc->value, 0, 32);
        if (!current)
                resume_pc = *address;
        else
@@ -996,29 +939,28 @@ static int cortex_a8_internal_restore(struct target *target, int current,
        /* Make sure that the Armv7 gdb thumb fixups does not
         * kill the return address
         */
-       switch (armv4_5->core_state)
-       {
-       case ARM_STATE_ARM:
-               resume_pc &= 0xFFFFFFFC;
-               break;
-       case ARM_STATE_THUMB:
-       case ARM_STATE_THUMB_EE:
-               /* When the return address is loaded into PC
-                * bit 0 must be 1 to stay in Thumb state
-                */
-               resume_pc |= 0x1;
-               break;
-       case ARM_STATE_JAZELLE:
-               LOG_ERROR("How do I resume into Jazelle state??");
-               return ERROR_FAIL;
+       switch (arm->core_state) {
+               case ARM_STATE_ARM:
+                       resume_pc &= 0xFFFFFFFC;
+                       break;
+               case ARM_STATE_THUMB:
+               case ARM_STATE_THUMB_EE:
+                       /* When the return address is loaded into PC
+                        * bit 0 must be 1 to stay in Thumb state
+                        */
+                       resume_pc |= 0x1;
+                       break;
+               case ARM_STATE_JAZELLE:
+                       LOG_ERROR("How do I resume into Jazelle state??");
+                       return ERROR_FAIL;
        }
        LOG_DEBUG("resume pc = 0x%08" PRIx32, resume_pc);
-       buf_set_u32(armv4_5->pc->value, 0, 32, resume_pc);
-       armv4_5->pc->dirty = 1;
-       armv4_5->pc->valid = 1;
+       buf_set_u32(arm->pc->value, 0, 32, resume_pc);
+       arm->pc->dirty = 1;
+       arm->pc->valid = 1;
        /* restore dpm_mode at system halt */
-    dpm_modeswitch(&armv7a->dpm, ARM_MODE_ANY);
-    /* called it now before restoring context because it uses cpu
+       dpm_modeswitch(&armv7a->dpm, ARM_MODE_ANY);
+       /* called it now before restoring context because it uses cpu
         * register r0 for restoring cp15 control register */
        retval = cortex_a8_restore_cp15_control_reg(target);
        if (retval != ERROR_OK)
@@ -1026,19 +968,18 @@ static int cortex_a8_internal_restore(struct target *target, int current,
        retval = cortex_a8_restore_context(target, handle_breakpoints);
        if (retval != ERROR_OK)
                return retval;
-    target->debug_reason = DBG_REASON_NOTHALTED;
+       target->debug_reason = DBG_REASON_NOTHALTED;
        target->state = TARGET_RUNNING;
 
        /* registers are now invalid */
-       register_cache_invalidate(armv4_5->core_cache);
+       register_cache_invalidate(arm->core_cache);
 
 #if 0
        /* the front-end may request us not to handle breakpoints */
-       if (handle_breakpoints)
-       {
+       if (handle_breakpoints) {
                /* Single step past breakpoint at current address */
-               if ((breakpoint = breakpoint_find(target, resume_pc)))
-               {
+               breakpoint = breakpoint_find(target, resume_pc);
+               if (breakpoint) {
                        LOG_DEBUG("unset breakpoint at 0x%8.8x", breakpoint->address);
                        cortex_m3_unset_breakpoint(target, breakpoint);
                        cortex_m3_single_step_core(target);
@@ -1053,19 +994,19 @@ static int cortex_a8_internal_restore(struct target *target, int current,
 static int cortex_a8_internal_restart(struct target *target)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
-       struct adiv5_dap *swjdp = armv4_5->dap;
+       struct arm *arm = &armv7a->arm;
+       struct adiv5_dap *swjdp = arm->dap;
        int retval;
        uint32_t dscr;
-/*
-        * Restart core and wait for it to be started.  Clear ITRen and sticky
-        * exception flags: see ARMv7 ARM, C5.9.
+       /*
+        * Restart core and wait for it to be started.  Clear ITRen and sticky
+        * exception flags: see ARMv7 ARM, C5.9.
         *
         * REVISIT: for single stepping, we probably want to
         * disable IRQs by default, with optional override...
         */
 
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DSCR, &dscr);
        if (retval != ERROR_OK)
                return retval;
@@ -1073,28 +1014,26 @@ static int cortex_a8_internal_restart(struct target *target)
        if ((dscr & DSCR_INSTR_COMP) == 0)
                LOG_ERROR("DSCR InstrCompl must be set before leaving debug!");
 
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
-               armv7a->debug_base + CPUDBG_DSCR, dscr & ~DSCR_ITR_EN);
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, dscr & ~DSCR_ITR_EN);
        if (retval != ERROR_OK)
                return retval;
 
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DRCR, DRCR_RESTART |
                        DRCR_CLEAR_EXCEPTIONS);
        if (retval != ERROR_OK)
                return retval;
 
        long long then = timeval_ms();
-       for (;;)
-       {
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       for (;; ) {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
                if (retval != ERROR_OK)
                        return retval;
                if ((dscr & DSCR_CORE_RESTARTED) != 0)
                        break;
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("Timeout waiting for resume");
                        return ERROR_FAIL;
                }
@@ -1104,27 +1043,25 @@ static int cortex_a8_internal_restart(struct target *target)
        target->state = TARGET_RUNNING;
 
        /* registers are now invalid */
-       register_cache_invalidate(armv4_5->core_cache);
+       register_cache_invalidate(arm->core_cache);
 
        return ERROR_OK;
 }
 
-static int cortex_a8_restore_smp(struct target *target,int handle_breakpoints)
+static int cortex_a8_restore_smp(struct target *target, int handle_breakpoints)
 {
        int retval = 0;
        struct target_list *head;
        struct target *curr;
-    uint32_t address;
+       uint32_t address;
        head = target->head;
-       while(head != (struct target_list*)NULL)
-       {
+       while (head != (struct target_list *)NULL) {
                curr = head->target;
-               if ((curr != target) && (curr->state != TARGET_RUNNING))
-               {
-               /*  resume current address , not in step mode */
-               retval += cortex_a8_internal_restore(curr, 1, &address,
-                               handle_breakpoints, 0);
-                 retval += cortex_a8_internal_restart(curr);
+               if ((curr != target) && (curr->state != TARGET_RUNNING)) {
+                       /*  resume current address , not in step mode */
+                       retval += cortex_a8_internal_restore(curr, 1, &address,
+                                       handle_breakpoints, 0);
+                       retval += cortex_a8_internal_restart(curr);
                }
                head = head->next;
 
@@ -1133,12 +1070,11 @@ static int cortex_a8_restore_smp(struct target *target,int handle_breakpoints)
 }
 
 static int cortex_a8_resume(struct target *target, int current,
-               uint32_t address, int handle_breakpoints, int debug_execution)
+       uint32_t address, int handle_breakpoints, int debug_execution)
 {
        int retval = 0;
-       /*   dummy resume for smp toggle in order to reduce gdb impact  */
-       if ((target->smp) && (target->gdb_service->core[1]!=-1))
-       {
+       /* dummy resume for smp toggle in order to reduce gdb impact  */
+       if ((target->smp) && (target->gdb_service->core[1] != -1)) {
                /*   simulate a start and halt of target */
                target->gdb_service->target = NULL;
                target->gdb_service->core[0] = target->gdb_service->core[1];
@@ -1147,22 +1083,19 @@ static int cortex_a8_resume(struct target *target, int current,
                return 0;
        }
        cortex_a8_internal_restore(target, current, &address, handle_breakpoints, debug_execution);
-       if (target->smp)
-       {   target->gdb_service->core[0] = -1;
+       if (target->smp) {
+               target->gdb_service->core[0] = -1;
                retval = cortex_a8_restore_smp(target, handle_breakpoints);
                if (retval != ERROR_OK)
                        return retval;
        }
        cortex_a8_internal_restart(target);
 
-       if (!debug_execution)
-       {
+       if (!debug_execution) {
                target->state = TARGET_RUNNING;
                target_call_event_callbacks(target, TARGET_EVENT_RESUMED);
                LOG_DEBUG("target resumed at 0x%" PRIx32, address);
-       }
-       else
-       {
+       } else {
                target->state = TARGET_DEBUG_RUNNING;
                target_call_event_callbacks(target, TARGET_EVENT_DEBUG_RESUMED);
                LOG_DEBUG("target debug resumed at 0x%" PRIx32, address);
@@ -1179,15 +1112,15 @@ static int cortex_a8_debug_entry(struct target *target)
        struct working_area *regfile_working_area = NULL;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct arm *arm = &armv7a->arm;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        struct reg *reg;
 
        LOG_DEBUG("dscr = 0x%08" PRIx32, cortex_a8->cpudbg_dscr);
 
        /* REVISIT surely we should not re-read DSCR !! */
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
        if (retval != ERROR_OK)
                return retval;
 
@@ -1198,7 +1131,7 @@ static int cortex_a8_debug_entry(struct target *target)
 
        /* Enable the ITR execution once we are in debug mode */
        dscr |= DSCR_ITR_EN;
-       retval = mem_ap_sel_write_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_DSCR, dscr);
        if (retval != ERROR_OK)
                return retval;
@@ -1210,7 +1143,7 @@ static int cortex_a8_debug_entry(struct target *target)
        if (target->debug_reason == DBG_REASON_WATCHPOINT) {
                uint32_t wfar;
 
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                                armv7a->debug_base + CPUDBG_WFAR,
                                &wfar);
                if (retval != ERROR_OK)
@@ -1226,19 +1159,14 @@ static int cortex_a8_debug_entry(struct target *target)
 
        /* First load register acessible through core debug port*/
        if (!regfile_working_area)
-       {
                retval = arm_dpm_read_current_registers(&armv7a->dpm);
-       }
-       else
-       {
+       else {
                retval = cortex_a8_read_regs_through_mem(target,
                                regfile_working_area->address, regfile);
 
                target_free_working_area(target, regfile_working_area);
                if (retval != ERROR_OK)
-               {
                        return retval;
-               }
 
                /* read Current PSR */
                retval = cortex_a8_dap_read_coreregister_u32(target, &cpsr, 16);
@@ -1248,12 +1176,11 @@ static int cortex_a8_debug_entry(struct target *target)
 
                LOG_DEBUG("cpsr: %8.8" PRIx32, cpsr);
 
-               arm_set_cpsr(armv4_5, cpsr);
+               arm_set_cpsr(arm, cpsr);
 
                /* update cache */
-               for (i = 0; i <= ARM_PC; i++)
-               {
-                       reg = arm_reg_current(armv4_5, i);
+               for (i = 0; i <= ARM_PC; i++) {
+                       reg = arm_reg_current(arm, i);
 
                        buf_set_u32(reg->value, 0, 32, regfile[i]);
                        reg->valid = 1;
@@ -1261,18 +1188,15 @@ static int cortex_a8_debug_entry(struct target *target)
                }
 
                /* Fixup PC Resume Address */
-               if (cpsr & (1 << 5))
-               {
-                       // T bit set for Thumb or ThumbEE state
+               if (cpsr & (1 << 5)) {
+                       /* T bit set for Thumb or ThumbEE state */
                        regfile[ARM_PC] -= 4;
-               }
-               else
-               {
-                       // ARM state
+               } else {
+                       /* ARM state */
                        regfile[ARM_PC] -= 8;
                }
 
-               reg = armv4_5->pc;
+               reg = arm->pc;
                buf_set_u32(reg->value, 0, 32, regfile[ARM_PC]);
                reg->dirty = reg->valid;
        }
@@ -1291,9 +1215,8 @@ static int cortex_a8_debug_entry(struct target *target)
 #endif
 
        /* Are we in an exception handler */
-//     armv4_5->exception_number = 0;
-       if (armv7a->post_debug_entry)
-       {
+/*     armv4_5->exception_number = 0; */
+       if (armv7a->post_debug_entry) {
                retval = armv7a->post_debug_entry(target);
                if (retval != ERROR_OK)
                        return retval;
@@ -1309,57 +1232,54 @@ static int cortex_a8_post_debug_entry(struct target *target)
        int retval;
 
        /* MRC p15,0,<Rt>,c1,c0,0 ; Read CP15 System Control Register */
-       retval = armv7a->armv4_5_common.mrc(target, 15,
+       retval = armv7a->arm.mrc(target, 15,
                        0, 0,   /* op1, op2 */
                        1, 0,   /* CRn, CRm */
                        &cortex_a8->cp15_control_reg);
        if (retval != ERROR_OK)
                return retval;
        LOG_DEBUG("cp15_control_reg: %8.8" PRIx32, cortex_a8->cp15_control_reg);
-    cortex_a8->cp15_control_reg_curr = cortex_a8->cp15_control_reg;
+       cortex_a8->cp15_control_reg_curr = cortex_a8->cp15_control_reg;
 
        if (armv7a->armv7a_mmu.armv7a_cache.ctype == -1)
-       {
                armv7a_identify_cache(target);
-       }
 
-       armv7a->armv7a_mmu.mmu_enabled =
+       if (armv7a->is_armv7r) {
+               armv7a->armv7a_mmu.mmu_enabled = 0;
+       } else {
+               armv7a->armv7a_mmu.mmu_enabled =
                        (cortex_a8->cp15_control_reg & 0x1U) ? 1 : 0;
+       }
        armv7a->armv7a_mmu.armv7a_cache.d_u_cache_enabled =
-                       (cortex_a8->cp15_control_reg & 0x4U) ? 1 : 0;
+               (cortex_a8->cp15_control_reg & 0x4U) ? 1 : 0;
        armv7a->armv7a_mmu.armv7a_cache.i_cache_enabled =
-                       (cortex_a8->cp15_control_reg & 0x1000U) ? 1 : 0;
-    cortex_a8->curr_mode = armv7a->armv4_5_common.core_mode;
-    
+               (cortex_a8->cp15_control_reg & 0x1000U) ? 1 : 0;
+       cortex_a8->curr_mode = armv7a->arm.core_mode;
+
        return ERROR_OK;
 }
 
 static int cortex_a8_step(struct target *target, int current, uint32_t address,
-               int handle_breakpoints)
+       int handle_breakpoints)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
+       struct arm *arm = &armv7a->arm;
        struct breakpoint *breakpoint = NULL;
        struct breakpoint stepbreakpoint;
        struct reg *r;
        int retval;
 
-       if (target->state != TARGET_HALTED)
-       {
+       if (target->state != TARGET_HALTED) {
                LOG_WARNING("target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
 
        /* current = 1: continue on current pc, otherwise continue at <address> */
-       r = armv4_5->pc;
+       r = arm->pc;
        if (!current)
-       {
                buf_set_u32(r->value, 0, 32, address);
-       }
        else
-       {
                address = buf_get_u32(r->value, 0, 32);
-       }
 
        /* The front-end may request us not to handle breakpoints.
         * But since Cortex-A8 uses breakpoint for single step,
@@ -1374,8 +1294,8 @@ static int cortex_a8_step(struct target *target, int current, uint32_t address,
 
        /* Setup single step breakpoint */
        stepbreakpoint.address = address;
-       stepbreakpoint.length = (armv4_5->core_state == ARM_STATE_THUMB)
-                       ? 2 : 4;
+       stepbreakpoint.length = (arm->core_state == ARM_STATE_THUMB)
+               ? 2 : 4;
        stepbreakpoint.type = BKPT_HARD;
        stepbreakpoint.set = 0;
 
@@ -1389,13 +1309,11 @@ static int cortex_a8_step(struct target *target, int current, uint32_t address,
                return retval;
 
        long long then = timeval_ms();
-       while (target->state != TARGET_HALTED)
-       {
+       while (target->state != TARGET_HALTED) {
                retval = cortex_a8_poll(target);
                if (retval != ERROR_OK)
                        return retval;
-               if (timeval_ms() > then + 1000)
-               {
+               if (timeval_ms() > then + 1000) {
                        LOG_ERROR("timeout waiting for target halt");
                        return ERROR_FAIL;
                }
@@ -1426,46 +1344,40 @@ static int cortex_a8_restore_context(struct target *target, bool bpwp)
        return arm_dpm_write_dirty_registers(&armv7a->dpm, bpwp);
 }
 
-
 /*
  * Cortex-A8 Breakpoint and watchpoint functions
  */
 
 /* Setup hardware Breakpoint Register Pair */
 static int cortex_a8_set_breakpoint(struct target *target,
-               struct breakpoint *breakpoint, uint8_t matchmode)
+       struct breakpoint *breakpoint, uint8_t matchmode)
 {
        int retval;
-       int brp_i=0;
+       int brp_i = 0;
        uint32_t control;
        uint8_t byte_addr_select = 0x0F;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct cortex_a8_brp * brp_list = cortex_a8->brp_list;
+       struct cortex_a8_brp *brp_list = cortex_a8->brp_list;
 
-       if (breakpoint->set)
-       {
+       if (breakpoint->set) {
                LOG_WARNING("breakpoint already set");
                return ERROR_OK;
        }
 
-       if (breakpoint->type == BKPT_HARD)
-       {
+       if (breakpoint->type == BKPT_HARD) {
                while (brp_list[brp_i].used && (brp_i < cortex_a8->brp_num))
-                       brp_i++ ;
-               if (brp_i >= cortex_a8->brp_num)
-               {
+                       brp_i++;
+               if (brp_i >= cortex_a8->brp_num) {
                        LOG_ERROR("ERROR Can not find free Breakpoint Register Pair");
                        return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
                }
                breakpoint->set = brp_i + 1;
                if (breakpoint->length == 2)
-               {
                        byte_addr_select = (3 << (breakpoint->address & 0x02));
-               }
                control = ((matchmode & 0x7) << 20)
-                               | (byte_addr_select << 5)
-                               | (3 << 1) | 1;
+                       | (byte_addr_select << 5)
+                       | (3 << 1) | 1;
                brp_list[brp_i].used = 1;
                brp_list[brp_i].value = (breakpoint->address & 0xFFFFFFFC);
                brp_list[brp_i].control = control;
@@ -1480,92 +1392,85 @@ static int cortex_a8_set_breakpoint(struct target *target,
                if (retval != ERROR_OK)
                        return retval;
                LOG_DEBUG("brp %i control 0x%0" PRIx32 " value 0x%0" PRIx32, brp_i,
-                               brp_list[brp_i].control,
-                               brp_list[brp_i].value);
-       }
-       else if (breakpoint->type == BKPT_SOFT)
-       {
+                       brp_list[brp_i].control,
+                       brp_list[brp_i].value);
+       } else if (breakpoint->type == BKPT_SOFT) {
                uint8_t code[4];
                if (breakpoint->length == 2)
-               {
                        buf_set_u32(code, 0, 32, ARMV5_T_BKPT(0x11));
-               }
                else
-               {
                        buf_set_u32(code, 0, 32, ARMV5_BKPT(0x11));
-               }
-               retval = target->type->read_memory(target,
+               retval = target_read_memory(target,
                                breakpoint->address & 0xFFFFFFFE,
                                breakpoint->length, 1,
                                breakpoint->orig_instr);
                if (retval != ERROR_OK)
                        return retval;
-               retval = target->type->write_memory(target,
+               retval = target_write_memory(target,
                                breakpoint->address & 0xFFFFFFFE,
                                breakpoint->length, 1, code);
                if (retval != ERROR_OK)
                        return retval;
-               breakpoint->set = 0x11; /* Any nice value but 0 */
+               breakpoint->set = 0x11; /* Any nice value but 0 */
        }
 
        return ERROR_OK;
 }
 
 static int cortex_a8_set_context_breakpoint(struct target *target,
-               struct breakpoint *breakpoint, uint8_t matchmode)
+       struct breakpoint *breakpoint, uint8_t matchmode)
 {
        int retval = ERROR_FAIL;
-       int brp_i=0;
+       int brp_i = 0;
        uint32_t control;
        uint8_t byte_addr_select = 0x0F;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct cortex_a8_brp * brp_list = cortex_a8->brp_list;
-       
-       if (breakpoint->set)
-       {
+       struct cortex_a8_brp *brp_list = cortex_a8->brp_list;
+
+       if (breakpoint->set) {
                LOG_WARNING("breakpoint already set");
-               return retval ;
+               return retval;
        }
        /*check available context BRPs*/
-       while ((brp_list[brp_i].used || (brp_list[brp_i].type!=BRP_CONTEXT)) && (brp_i < cortex_a8->brp_num))
-                       brp_i++ ;
-       
-       if (brp_i >= cortex_a8->brp_num)
-       {
+       while ((brp_list[brp_i].used ||
+               (brp_list[brp_i].type != BRP_CONTEXT)) && (brp_i < cortex_a8->brp_num))
+               brp_i++;
+
+       if (brp_i >= cortex_a8->brp_num) {
                LOG_ERROR("ERROR Can not find free Breakpoint Register Pair");
                return ERROR_FAIL;
        }
 
        breakpoint->set = brp_i + 1;
        control = ((matchmode & 0x7) << 20)
-                               | (byte_addr_select << 5)
-                               | (3 << 1) | 1;
+               | (byte_addr_select << 5)
+               | (3 << 1) | 1;
        brp_list[brp_i].used = 1;
        brp_list[brp_i].value = (breakpoint->asid);
        brp_list[brp_i].control = control;
        retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
-                       + CPUDBG_BVR_BASE + 4 * brp_list[brp_i].BRPn, 
+                       + CPUDBG_BVR_BASE + 4 * brp_list[brp_i].BRPn,
                        brp_list[brp_i].value);
-       if(retval != ERROR_OK)
+       if (retval != ERROR_OK)
                return retval;
        retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
                        + CPUDBG_BCR_BASE + 4 * brp_list[brp_i].BRPn,
                        brp_list[brp_i].control);
-       if(retval != ERROR_OK)
+       if (retval != ERROR_OK)
                return retval;
        LOG_DEBUG("brp %i control 0x%0" PRIx32 " value 0x%0" PRIx32, brp_i,
-       brp_list[brp_i].control,
-       brp_list[brp_i].value);
+               brp_list[brp_i].control,
+               brp_list[brp_i].value);
        return ERROR_OK;
-               
+
 }
 
 static int cortex_a8_set_hybrid_breakpoint(struct target *target, struct breakpoint *breakpoint)
 {
        int retval = ERROR_FAIL;
-       int brp_1=0; //holds the contextID pair
-       int brp_2=0; // holds the IVA pair
+       int brp_1 = 0;  /* holds the contextID pair */
+       int brp_2 = 0;  /* holds the IVA pair */
        uint32_t control_CTX, control_IVA;
        uint8_t CTX_byte_addr_select = 0x0F;
        uint8_t IVA_byte_addr_select = 0x0F;
@@ -1573,105 +1478,97 @@ static int cortex_a8_set_hybrid_breakpoint(struct target *target, struct breakpo
        uint8_t IVA_machmode = 0x01;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct cortex_a8_brp * brp_list = cortex_a8->brp_list;
-       
-       
-       
-       if (breakpoint->set)
-       {
+       struct cortex_a8_brp *brp_list = cortex_a8->brp_list;
+
+       if (breakpoint->set) {
                LOG_WARNING("breakpoint already set");
-               return retval ;
+               return retval;
        }
        /*check available context BRPs*/
-       while ((brp_list[brp_1].used || (brp_list[brp_1].type!=BRP_CONTEXT)) && (brp_1 < cortex_a8->brp_num))
-                       brp_1++ ;
-       
-       printf("brp(CTX) found num: %d \n",brp_1);
-       if (brp_1 >= cortex_a8->brp_num)
-       {
+       while ((brp_list[brp_1].used ||
+               (brp_list[brp_1].type != BRP_CONTEXT)) && (brp_1 < cortex_a8->brp_num))
+               brp_1++;
+
+       printf("brp(CTX) found num: %d\n", brp_1);
+       if (brp_1 >= cortex_a8->brp_num) {
                LOG_ERROR("ERROR Can not find free Breakpoint Register Pair");
                return ERROR_FAIL;
        }
 
-       while ((brp_list[brp_2].used || (brp_list[brp_2].type!=BRP_NORMAL)) && (brp_2 < cortex_a8->brp_num))
-                       brp_2++ ;
-       
-       printf("brp(IVA) found num: %d \n",brp_2);
-       if (brp_2 >= cortex_a8->brp_num)
-       {
+       while ((brp_list[brp_2].used ||
+               (brp_list[brp_2].type != BRP_NORMAL)) && (brp_2 < cortex_a8->brp_num))
+               brp_2++;
+
+       printf("brp(IVA) found num: %d\n", brp_2);
+       if (brp_2 >= cortex_a8->brp_num) {
                LOG_ERROR("ERROR Can not find free Breakpoint Register Pair");
                return ERROR_FAIL;
        }
 
        breakpoint->set = brp_1 + 1;
-       breakpoint->linked_BRP= brp_2;
+       breakpoint->linked_BRP = brp_2;
        control_CTX = ((CTX_machmode & 0x7) << 20)
-                               | (brp_2 << 16)
-                               | (0 << 14)
-                               | (CTX_byte_addr_select << 5)
-                               | (3 << 1) | 1;
-               brp_list[brp_1].used = 1;
-               brp_list[brp_1].value = (breakpoint->asid);
-               brp_list[brp_1].control = control_CTX;
-               retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
-                               + CPUDBG_BVR_BASE + 4 * brp_list[brp_1].BRPn, 
-                               brp_list[brp_1].value);
-               if (retval != ERROR_OK)
-                       return retval;
-               retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
-                               + CPUDBG_BCR_BASE + 4 * brp_list[brp_1].BRPn,
-                               brp_list[brp_1].control);
-               if( retval != ERROR_OK )
-                       return retval;
+               | (brp_2 << 16)
+               | (0 << 14)
+               | (CTX_byte_addr_select << 5)
+               | (3 << 1) | 1;
+       brp_list[brp_1].used = 1;
+       brp_list[brp_1].value = (breakpoint->asid);
+       brp_list[brp_1].control = control_CTX;
+       retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
+                       + CPUDBG_BVR_BASE + 4 * brp_list[brp_1].BRPn,
+                       brp_list[brp_1].value);
+       if (retval != ERROR_OK)
+               return retval;
+       retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
+                       + CPUDBG_BCR_BASE + 4 * brp_list[brp_1].BRPn,
+                       brp_list[brp_1].control);
+       if (retval != ERROR_OK)
+               return retval;
 
-               control_IVA = ((IVA_machmode & 0x7) << 20)
-                               | (brp_1 << 16)
-                               | (IVA_byte_addr_select << 5)
-                               | (3 << 1) | 1;
-               brp_list[brp_2].used = 1;
-               brp_list[brp_2].value = (breakpoint->address & 0xFFFFFFFC);
-               brp_list[brp_2].control = control_IVA;
-               retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
-                               + CPUDBG_BVR_BASE + 4 * brp_list[brp_2].BRPn, 
-                               brp_list[brp_2].value);
-               if (retval != ERROR_OK)
-                       return retval;
-               retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
-                               + CPUDBG_BCR_BASE + 4 * brp_list[brp_2].BRPn,
-                               brp_list[brp_2].control);
-               if (retval != ERROR_OK )
-                       return retval;
+       control_IVA = ((IVA_machmode & 0x7) << 20)
+               | (brp_1 << 16)
+               | (IVA_byte_addr_select << 5)
+               | (3 << 1) | 1;
+       brp_list[brp_2].used = 1;
+       brp_list[brp_2].value = (breakpoint->address & 0xFFFFFFFC);
+       brp_list[brp_2].control = control_IVA;
+       retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
+                       + CPUDBG_BVR_BASE + 4 * brp_list[brp_2].BRPn,
+                       brp_list[brp_2].value);
+       if (retval != ERROR_OK)
+               return retval;
+       retval = cortex_a8_dap_write_memap_register_u32(target, armv7a->debug_base
+                       + CPUDBG_BCR_BASE + 4 * brp_list[brp_2].BRPn,
+                       brp_list[brp_2].control);
+       if (retval != ERROR_OK)
+               return retval;
 
        return ERROR_OK;
 }
 
-
 static int cortex_a8_unset_breakpoint(struct target *target, struct breakpoint *breakpoint)
 {
        int retval;
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct cortex_a8_brp * brp_list = cortex_a8->brp_list;
+       struct cortex_a8_brp *brp_list = cortex_a8->brp_list;
 
-       if (!breakpoint->set)
-       {
+       if (!breakpoint->set) {
                LOG_WARNING("breakpoint not set");
                return ERROR_OK;
        }
 
-       if (breakpoint->type == BKPT_HARD)
-       {
-               if ((breakpoint->address != 0) && (breakpoint->asid != 0))
-               {
+       if (breakpoint->type == BKPT_HARD) {
+               if ((breakpoint->address != 0) && (breakpoint->asid != 0)) {
                        int brp_i = breakpoint->set - 1;
                        int brp_j = breakpoint->linked_BRP;
-                       if ((brp_i < 0) || (brp_i >= cortex_a8->brp_num))
-                       {
+                       if ((brp_i < 0) || (brp_i >= cortex_a8->brp_num)) {
                                LOG_DEBUG("Invalid BRP number in breakpoint");
                                return ERROR_OK;
                        }
                        LOG_DEBUG("rbp %i control 0x%0" PRIx32 " value 0x%0" PRIx32, brp_i,
-                                       brp_list[brp_i].control, brp_list[brp_i].value);
+                               brp_list[brp_i].control, brp_list[brp_i].value);
                        brp_list[brp_i].used = 0;
                        brp_list[brp_i].value = 0;
                        brp_list[brp_i].control = 0;
@@ -1684,14 +1581,13 @@ static int cortex_a8_unset_breakpoint(struct target *target, struct breakpoint *
                                        + CPUDBG_BVR_BASE + 4 * brp_list[brp_i].BRPn,
                                        brp_list[brp_i].value);
                        if (retval != ERROR_OK)
-                               return retval;                                          
-                       if ((brp_j < 0) || (brp_j >= cortex_a8->brp_num))
-                       {
+                               return retval;
+                       if ((brp_j < 0) || (brp_j >= cortex_a8->brp_num)) {
                                LOG_DEBUG("Invalid BRP number in breakpoint");
                                return ERROR_OK;
                        }
                        LOG_DEBUG("rbp %i control 0x%0" PRIx32 " value 0x%0" PRIx32, brp_j,
-                                       brp_list[brp_j].control, brp_list[brp_j].value);
+                               brp_list[brp_j].control, brp_list[brp_j].value);
                        brp_list[brp_j].used = 0;
                        brp_list[brp_j].value = 0;
                        brp_list[brp_j].control = 0;
@@ -1708,18 +1604,15 @@ static int cortex_a8_unset_breakpoint(struct target *target, struct breakpoint *
                        breakpoint->linked_BRP = 0;
                        breakpoint->set = 0;
                        return ERROR_OK;
-                       
-               }
-               else
-               {
+
+               } else {
                        int brp_i = breakpoint->set - 1;
-                       if ((brp_i < 0) || (brp_i >= cortex_a8->brp_num))
-                       {
+                       if ((brp_i < 0) || (brp_i >= cortex_a8->brp_num)) {
                                LOG_DEBUG("Invalid BRP number in breakpoint");
                                return ERROR_OK;
                        }
                        LOG_DEBUG("rbp %i control 0x%0" PRIx32 " value 0x%0" PRIx32, brp_i,
-                                       brp_list[brp_i].control, brp_list[brp_i].value);
+                               brp_list[brp_i].control, brp_list[brp_i].value);
                        brp_list[brp_i].used = 0;
                        brp_list[brp_i].value = 0;
                        brp_list[brp_i].control = 0;
@@ -1735,22 +1628,17 @@ static int cortex_a8_unset_breakpoint(struct target *target, struct breakpoint *
                                return retval;
                        breakpoint->set = 0;
                        return ERROR_OK;
-               }                                       
-       }
-       else
-       {
+               }
+       } else {
                /* restore original instruction (kept in target endianness) */
-               if (breakpoint->length == 4)
-               {
-                       retval = target->type->write_memory(target,
+               if (breakpoint->length == 4) {
+                       retval = target_write_memory(target,
                                        breakpoint->address & 0xFFFFFFFE,
                                        4, 1, breakpoint->orig_instr);
                        if (retval != ERROR_OK)
                                return retval;
-               }
-               else
-               {
-                       retval = target->type->write_memory(target,
+               } else {
+                       retval = target_write_memory(target,
                                        breakpoint->address & 0xFFFFFFFE,
                                        2, 1, breakpoint->orig_instr);
                        if (retval != ERROR_OK)
@@ -1763,12 +1651,11 @@ static int cortex_a8_unset_breakpoint(struct target *target, struct breakpoint *
 }
 
 static int cortex_a8_add_breakpoint(struct target *target,
-               struct breakpoint *breakpoint)
+       struct breakpoint *breakpoint)
 {
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
 
-       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1))
-       {
+       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1)) {
                LOG_INFO("no hardware breakpoint available");
                return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
        }
@@ -1776,16 +1663,15 @@ static int cortex_a8_add_breakpoint(struct target *target,
        if (breakpoint->type == BKPT_HARD)
                cortex_a8->brp_num_available--;
 
-       return cortex_a8_set_breakpoint(target, breakpoint, 0x00); /* Exact match */
+       return cortex_a8_set_breakpoint(target, breakpoint, 0x00);      /* Exact match */
 }
 
 static int cortex_a8_add_context_breakpoint(struct target *target,
-               struct breakpoint *breakpoint)
+       struct breakpoint *breakpoint)
 {
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
 
-       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1))
-       {
+       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1)) {
                LOG_INFO("no hardware breakpoint available");
                return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
        }
@@ -1793,16 +1679,15 @@ static int cortex_a8_add_context_breakpoint(struct target *target,
        if (breakpoint->type == BKPT_HARD)
                cortex_a8->brp_num_available--;
 
-       return cortex_a8_set_context_breakpoint(target, breakpoint, 0x02); /* asid match */
+       return cortex_a8_set_context_breakpoint(target, breakpoint, 0x02);      /* asid match */
 }
 
 static int cortex_a8_add_hybrid_breakpoint(struct target *target,
-               struct breakpoint *breakpoint)
+       struct breakpoint *breakpoint)
 {
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
 
-       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1))
-       {
+       if ((breakpoint->type == BKPT_HARD) && (cortex_a8->brp_num_available < 1)) {
                LOG_INFO("no hardware breakpoint available");
                return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
        }
@@ -1810,7 +1695,7 @@ static int cortex_a8_add_hybrid_breakpoint(struct target *target,
        if (breakpoint->type == BKPT_HARD)
                cortex_a8->brp_num_available--;
 
-       return cortex_a8_set_hybrid_breakpoint(target, breakpoint); /* ??? */
+       return cortex_a8_set_hybrid_breakpoint(target, breakpoint);     /* ??? */
 }
 
 
@@ -1820,26 +1705,22 @@ static int cortex_a8_remove_breakpoint(struct target *target, struct breakpoint
 
 #if 0
 /* It is perfectly possible to remove breakpoints while the target is running */
-       if (target->state != TARGET_HALTED)
-       {
+       if (target->state != TARGET_HALTED) {
                LOG_WARNING("target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
 #endif
 
-       if (breakpoint->set)
-       {
+       if (breakpoint->set) {
                cortex_a8_unset_breakpoint(target, breakpoint);
                if (breakpoint->type == BKPT_HARD)
-                       cortex_a8->brp_num_available++ ;
+                       cortex_a8->brp_num_available++;
        }
 
 
        return ERROR_OK;
 }
 
-
-
 /*
  * Cortex-A8 Reset functions
  */
@@ -1853,9 +1734,9 @@ static int cortex_a8_assert_reset(struct target *target)
        /* FIXME when halt is requested, make it work somehow... */
 
        /* Issue some kind of warm reset. */
-       if (target_has_event_action(target, TARGET_EVENT_RESET_ASSERT)) {
+       if (target_has_event_action(target, TARGET_EVENT_RESET_ASSERT))
                target_handle_event(target, TARGET_EVENT_RESET_ASSERT);
-       else if (jtag_get_reset_config() & RESET_HAS_SRST) {
+       else if (jtag_get_reset_config() & RESET_HAS_SRST) {
                /* REVISIT handle "pulls" cases, if there's
                 * hardware that needs them to work.
                 */
@@ -1866,7 +1747,7 @@ static int cortex_a8_assert_reset(struct target *target)
        }
 
        /* registers are now invalid */
-       register_cache_invalidate(armv7a->armv4_5_common.core_cache);
+       register_cache_invalidate(armv7a->arm.core_cache);
 
        target->state = TARGET_RESET;
 
@@ -1889,8 +1770,9 @@ static int cortex_a8_deassert_reset(struct target *target)
        if (target->reset_halt) {
                if (target->state != TARGET_HALTED) {
                        LOG_WARNING("%s: ran after reset and before halt ...",
-                                       target_name(target));
-                       if ((retval = target_halt(target)) != ERROR_OK)
+                               target_name(target));
+                       retval = target_halt(target);
+                       if (retval != ERROR_OK)
                                return retval;
                }
        }
@@ -1898,140 +1780,313 @@ static int cortex_a8_deassert_reset(struct target *target)
        return ERROR_OK;
 }
 
-
 static int cortex_a8_write_apb_ab_memory(struct target *target,
-                uint32_t address, uint32_t size,
-                uint32_t count, const uint8_t *buffer)
+       uint32_t address, uint32_t size,
+       uint32_t count, const uint8_t *buffer)
 {
-
        /* write memory through APB-AP */
 
        int retval = ERROR_COMMAND_SYNTAX_ERROR;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
+       struct arm *arm = &armv7a->arm;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int total_bytes = count * size;
-       int start_byte, nbytes_to_write, i;
+       int total_u32;
+       int start_byte = address & 0x3;
+       int end_byte   = (address + total_bytes) & 0x3;
        struct reg *reg;
-       union _data {
-               uint8_t uc_a[4];
-               uint32_t ui;
-       } data;
+       uint32_t dscr;
+       uint8_t *tmp_buff = NULL;
 
-       if (target->state != TARGET_HALTED)
-       {
+       if (target->state != TARGET_HALTED) {
                LOG_WARNING("target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       reg = arm_reg_current(armv4_5, 0);
-       reg->dirty = 1;
-       reg = arm_reg_current(armv4_5, 1);
-       reg->dirty = 1;
+       total_u32 = DIV_ROUND_UP((address & 3) + total_bytes, 4);
+
+       /* Mark register R0 as dirty, as it will be used
+        * for transferring the data.
+        * It will be restored automatically when exiting
+        * debug mode
+        */
+       reg = arm_reg_current(arm, 0);
+       reg->dirty = true;
 
-       retval = cortex_a8_dap_write_coreregister_u32(target, address & 0xFFFFFFFC, 0);
+       /*  clear any abort  */
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap, armv7a->debug_base + CPUDBG_DRCR, 1<<2);
        if (retval != ERROR_OK)
                return retval;
 
-       start_byte = address & 0x3;
-
-       while (total_bytes > 0) {
+       /* This algorithm comes from either :
+        * Cortex-A8 TRM Example 12-25
+        * Cortex-R4 TRM Example 11-26
+        * (slight differences)
+        */
 
-               nbytes_to_write = 4 - start_byte;
-               if (total_bytes < nbytes_to_write)
-                       nbytes_to_write = total_bytes; 
-                       
-               if ( nbytes_to_write != 4 ) {
-               
-                       /* execute instruction LDR r1, [r0] */
-                       retval = cortex_a8_exec_opcode(target,  ARMV4_5_LDR(1, 0), NULL);
-                       if (retval != ERROR_OK)
-                               return retval;
+       /* The algorithm only copies 32 bit words, so the buffer
+        * should be expanded to include the words at either end.
+        * The first and last words will be read first to avoid
+        * corruption if needed.
+        */
+       tmp_buff = (uint8_t *) malloc(total_u32 << 2);
 
-                       retval = cortex_a8_dap_read_coreregister_u32(target, &data.ui, 1);
-                       if (retval != ERROR_OK)
-                               return retval;
-               }
-       
-               for (i = 0; i < nbytes_to_write; ++i)
-                       data.uc_a[i + start_byte] = *buffer++;
 
-               retval = cortex_a8_dap_write_coreregister_u32(target, data.ui, 1);
+       if ((start_byte != 0) && (total_u32 > 1)) {
+               /* First bytes not aligned - read the 32 bit word to avoid corrupting
+                * the other bytes in the word.
+                */
+               retval = cortex_a8_read_apb_ab_memory(target, (address & ~0x3), 4, 1, tmp_buff);
                if (retval != ERROR_OK)
-                       return retval;
+                       goto error_free_buff_w;
+       }
+
+       /* If end of write is not aligned, or the write is less than 4 bytes */
+       if ((end_byte != 0) ||
+               ((total_u32 == 1) && (total_bytes != 4))) {
 
-               /* execute instruction STRW r1, [r0], 1 (0xe4801004) */
-               retval = cortex_a8_exec_opcode(target, ARMV4_5_STRW_IP(1, 0) , NULL);
+               /* Read the last word to avoid corruption during 32 bit write */
+               int mem_offset = (total_u32-1) << 4;
+               retval = cortex_a8_read_apb_ab_memory(target, (address & ~0x3) + mem_offset, 4, 1, &tmp_buff[mem_offset]);
                if (retval != ERROR_OK)
-                               return retval;
+                       goto error_free_buff_w;
+       }
+
+       /* Copy the write buffer over the top of the temporary buffer */
+       memcpy(&tmp_buff[start_byte], buffer, total_bytes);
+
+       /* We now have a 32 bit aligned buffer that can be written */
+
+       /* Read DSCR */
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       if (retval != ERROR_OK)
+               goto error_free_buff_w;
+
+       /* Set DTR mode to Fast (2) */
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_FAST_MODE;
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, dscr);
+       if (retval != ERROR_OK)
+               goto error_free_buff_w;
+
+       /* Copy the destination address into R0 */
+       /*  - pend an instruction  MRC p14, 0, R0, c5, c0 */
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_ITR, ARMV4_5_MRC(14, 0, 0, 0, 5, 0));
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_w;
+       /* Write address into DTRRX, which triggers previous instruction */
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DTRRX, address & (~0x3));
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_w;
+
+       /* Write the data transfer instruction into the ITR
+        * (STC p14, c5, [R0], 4)
+        */
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_ITR, ARMV4_5_STC(0, 1, 0, 1, 14, 5, 0, 4));
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_w;
 
-               total_bytes -= nbytes_to_write;
-               start_byte = 0;
+       /* Do the write */
+       retval = mem_ap_sel_write_buf_u32_noincr(swjdp, armv7a->debug_ap,
+                                       tmp_buff, (total_u32)<<2, armv7a->debug_base + CPUDBG_DTRRX);
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_w;
+
+
+       /* Switch DTR mode back to non-blocking (0) */
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_NON_BLOCKING;
+       retval = mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, dscr);
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_w;
+
+    /* Check for sticky abort flags in the DSCR */
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       if (retval != ERROR_OK)
+               goto error_free_buff_w;
+       if (dscr & (DSCR_STICKY_ABORT_PRECISE | DSCR_STICKY_ABORT_IMPRECISE)) {
+               /* Abort occurred - clear it and exit */
+               LOG_ERROR("abort occurred - dscr = 0x%08x", dscr);
+               mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                                       armv7a->debug_base + CPUDBG_DRCR, 1<<2);
+               goto error_free_buff_w;
        }
 
-       return retval;
-}
+       /* Done */
+       free(tmp_buff);
+       return ERROR_OK;
 
+error_unset_dtr_w:
+       /* Unset DTR mode */
+       mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_NON_BLOCKING;
+       mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, dscr);
+error_free_buff_w:
+       LOG_ERROR("error");
+       free(tmp_buff);
+       return ERROR_FAIL;
+}
 
 static int cortex_a8_read_apb_ab_memory(struct target *target,
-                uint32_t address, uint32_t size,
-                uint32_t count, uint8_t *buffer)
+       uint32_t address, uint32_t size,
+       uint32_t count, uint8_t *buffer)
 {
-
        /* read memory through APB-AP */
 
        int retval = ERROR_COMMAND_SYNTAX_ERROR;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct arm *armv4_5 = &armv7a->armv4_5_common;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
+       struct arm *arm = &armv7a->arm;
        int total_bytes = count * size;
-       int start_byte, nbytes_to_read, i;
+       int total_u32;
+       int start_byte = address & 0x3;
        struct reg *reg;
-       union _data {
-               uint8_t uc_a[4];
-               uint32_t ui;
-       } data;
-
-       if (target->state != TARGET_HALTED)
-       {
+       uint32_t dscr;
+       uint32_t *tmp_buff;
+       uint32_t buff32[2];
+       if (target->state != TARGET_HALTED) {
                LOG_WARNING("target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       reg = arm_reg_current(armv4_5, 0);
-       reg->dirty = 1;
-       reg = arm_reg_current(armv4_5, 1);
-       reg->dirty = 1;
+       total_u32 = DIV_ROUND_UP((address & 3) + total_bytes, 4);
+
+       /* Due to offset word alignment, the  buffer may not have space
+        * to read the full first and last int32 words,
+        * hence, malloc space to read into, then copy and align into the buffer.
+        */
+       tmp_buff = malloc(total_u32 * 4);
+       if (tmp_buff == NULL)
+               return ERROR_FAIL;
 
-       retval = cortex_a8_dap_write_coreregister_u32(target, address & 0xFFFFFFFC, 0);
+       /* Mark register R0 as dirty, as it will be used
+        * for transferring the data.
+        * It will be restored automatically when exiting
+        * debug mode
+        */
+       reg = arm_reg_current(arm, 0);
+       reg->dirty = true;
+
+       /*  clear any abort  */
+       retval =
+               mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap, armv7a->debug_base + CPUDBG_DRCR, 1<<2);
        if (retval != ERROR_OK)
                return retval;
 
-       start_byte = address & 0x3;
+       /* Read DSCR */
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
+
+       /* This algorithm comes from either :
+        * Cortex-A8 TRM Example 12-24
+        * Cortex-R4 TRM Example 11-25
+        * (slight differences)
+        */
+
+       /* Set DTR access mode to stall mode b01  */
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_STALL_MODE;
+       retval +=  mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DSCR, dscr);
+
+    /* Write R0 with value 'address' using write procedure for stall mode */
+       /*   - Write the address for read access into DTRRX */
+       retval += mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DTRRX, address & ~0x3);
+       /*  - Copy value from DTRRX to R0 using instruction mrc p14, 0, r0, c5, c0 */
+       cortex_a8_exec_opcode(target, ARMV4_5_MRC(14, 0, 0, 0, 5, 0), &dscr);
+
 
-       while (total_bytes > 0) {
+       /* Write the data transfer instruction (ldc p14, c5, [r0],4)
+        * and the DTR mode setting to fast mode
+        * in one combined write (since they are adjacent registers)
+        */
+       buff32[0] = ARMV4_5_LDC(0, 1, 0, 1, 14, 5, 0, 4);
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_FAST_MODE;
+       buff32[1] = dscr;
+       /*  group the 2 access CPUDBG_ITR 0x84 and CPUDBG_DSCR 0x88 */
+       retval += mem_ap_sel_write_buf_u32(swjdp, armv7a->debug_ap, (uint8_t *)buff32, 8,
+                       armv7a->debug_base + CPUDBG_ITR);
+       if (retval != ERROR_OK)
+               goto error_unset_dtr_r;
 
-               /* execute instruction LDRW r1, [r0], 4 (0xe4901004)  */
-               retval = cortex_a8_exec_opcode(target,  ARMV4_5_LDRW_IP(1, 0), NULL);
+
+       /* The last word needs to be handled separately - read all other words in one go.
+        */
+       if (total_u32 > 1) {
+               /* Read the data - Each read of the DTRTX register causes the instruction to be reissued
+                * Abort flags are sticky, so can be read at end of transactions
+                *
+                * This data is read in aligned to 32 bit boundary, hence may need shifting later.
+                */
+               retval = mem_ap_sel_read_buf_u32_noincr(swjdp, armv7a->debug_ap, (uint8_t *)tmp_buff, (total_u32-1) * 4,
+                                                                       armv7a->debug_base + CPUDBG_DTRTX);
                if (retval != ERROR_OK)
-                       return retval;
+                       goto error_unset_dtr_r;
+       }
 
-               retval = cortex_a8_dap_read_coreregister_u32(target, &data.ui, 1);
+       /* set DTR access mode back to non blocking b00  */
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_NON_BLOCKING;
+       retval =  mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                                       armv7a->debug_base + CPUDBG_DSCR, dscr);
+       if (retval != ERROR_OK)
+               goto error_free_buff_r;
+
+       /* Wait for the final read instruction to finish */
+       do {
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                                       armv7a->debug_base + CPUDBG_DSCR, &dscr);
                if (retval != ERROR_OK)
-                       return retval;
+                       goto error_free_buff_r;
+       } while ((dscr & DSCR_INSTR_COMP) == 0);
 
-               nbytes_to_read = 4 - start_byte;
-               if (total_bytes < nbytes_to_read)
-                       nbytes_to_read = total_bytes; 
-       
-               for (i = 0; i < nbytes_to_read; ++i)
-                       *buffer++ = data.uc_a[i + start_byte];
-                       
-               total_bytes -= nbytes_to_read;
-               start_byte = 0;
-       }
 
-       return retval;
-}
+       /* Check for sticky abort flags in the DSCR */
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       if (retval != ERROR_OK)
+               goto error_free_buff_r;
+       if (dscr & (DSCR_STICKY_ABORT_PRECISE | DSCR_STICKY_ABORT_IMPRECISE)) {
+               /* Abort occurred - clear it and exit */
+               LOG_ERROR("abort occurred - dscr = 0x%08x", dscr);
+               mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                                       armv7a->debug_base + CPUDBG_DRCR, 1<<2);
+               goto error_free_buff_r;
+       }
+
+       /* Read the last word */
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DTRTX, &tmp_buff[total_u32 - 1]);
+       if (retval != ERROR_OK)
+               goto error_free_buff_r;
+
+       /* Copy and align the data into the output buffer */
+       memcpy(buffer, (uint8_t *)tmp_buff + start_byte, total_bytes);
 
+       free(tmp_buff);
+
+       /* Done */
+       return ERROR_OK;
+
+
+error_unset_dtr_r:
+       /* Unset DTR mode */
+       mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
+       dscr = (dscr & ~DSCR_EXT_DCC_MASK) | DSCR_EXT_DCC_NON_BLOCKING;
+       mem_ap_sel_write_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, dscr);
+error_free_buff_r:
+       LOG_ERROR("error");
+       free(tmp_buff);
+       return ERROR_FAIL;
+}
 
 
 /*
@@ -2042,138 +2097,147 @@ static int cortex_a8_read_apb_ab_memory(struct target *target,
  */
 
 static int cortex_a8_read_phys_memory(struct target *target,
-                uint32_t address, uint32_t size,
-                uint32_t count, uint8_t *buffer)
+       uint32_t address, uint32_t size,
+       uint32_t count, uint8_t *buffer)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int retval = ERROR_COMMAND_SYNTAX_ERROR;
        uint8_t apsel = swjdp->apsel;
        LOG_DEBUG("Reading memory at real address 0x%x; size %d; count %d",
-                       address, size, count);
+               address, size, count);
 
        if (count && buffer) {
 
-               if ( apsel == swjdp_memoryap ) {
+               if (armv7a->memory_ap_available && (apsel == armv7a->memory_ap)) {
 
                        /* read memory through AHB-AP */
 
                        switch (size) {
-                       case 4:
-                               retval = mem_ap_sel_read_buf_u32(swjdp, swjdp_memoryap,
+                               case 4:
+                                       retval = mem_ap_sel_read_buf_u32(swjdp, armv7a->memory_ap,
                                                buffer, 4 * count, address);
-                               break;
-                       case 2:
-                               retval = mem_ap_sel_read_buf_u16(swjdp, swjdp_memoryap,
+                                       break;
+                               case 2:
+                                       retval = mem_ap_sel_read_buf_u16(swjdp, armv7a->memory_ap,
                                                buffer, 2 * count, address);
-                               break;
-                       case 1:
-                               retval = mem_ap_sel_read_buf_u8(swjdp, swjdp_memoryap,
+                                       break;
+                               case 1:
+                                       retval = mem_ap_sel_read_buf_u8(swjdp, armv7a->memory_ap,
                                                buffer, count, address);
-                               break;
+                                       break;
                        }
                } else {
 
                        /* read memory through APB-AP */
-                       /*  disable mmu */
-                       retval = cortex_a8_mmu_modify(target, 0);
-            if (retval != ERROR_OK) return retval;
-                       retval =  cortex_a8_read_apb_ab_memory(target, address, size, count, buffer);
+                       if (!armv7a->is_armv7r) {
+                               /*  disable mmu */
+                               retval = cortex_a8_mmu_modify(target, 0);
+                               if (retval != ERROR_OK)
+                                       return retval;
+                       }
+                       retval = cortex_a8_read_apb_ab_memory(target, address, size, count, buffer);
                }
        }
        return retval;
 }
 
 static int cortex_a8_read_memory(struct target *target, uint32_t address,
-               uint32_t size, uint32_t count, uint8_t *buffer)
+       uint32_t size, uint32_t count, uint8_t *buffer)
 {
        int enabled = 0;
        uint32_t virt, phys;
        int retval;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        uint8_t apsel = swjdp->apsel;
 
        /* cortex_a8 handles unaligned memory access */
        LOG_DEBUG("Reading memory at address 0x%x; size %d; count %d", address,
-                       size, count);
-       if (apsel == swjdp_memoryap) {
-               retval = cortex_a8_mmu(target, &enabled);
-               if (retval != ERROR_OK)
-                       return retval;
-
-
-               if(enabled)
-               {
-                       virt = address;
-                       retval = cortex_a8_virt2phys(target, virt, &phys);
+               size, count);
+       if (armv7a->memory_ap_available && (apsel == armv7a->memory_ap)) {
+               if (!armv7a->is_armv7r) {
+                       retval = cortex_a8_mmu(target, &enabled);
                        if (retval != ERROR_OK)
                                return retval;
 
-                       LOG_DEBUG("Reading at virtual address. Translating v:0x%x to r:0x%x",
+
+                       if (enabled) {
+                               virt = address;
+                               retval = cortex_a8_virt2phys(target, virt, &phys);
+                               if (retval != ERROR_OK)
+                                       return retval;
+
+                               LOG_DEBUG("Reading at virtual address. Translating v:0x%x to r:0x%x",
                                        virt, phys);
-                       address = phys;
+                               address = phys;
+                       }
                }
                retval = cortex_a8_read_phys_memory(target, address, size, count, buffer);
        } else {
-               retval = cortex_a8_check_address(target, address);
-        if (retval != ERROR_OK) return retval;
-               /*  enable mmu */
-               retval = cortex_a8_mmu_modify(target, 1);
-           if (retval != ERROR_OK) return retval;
+               if (!armv7a->is_armv7r) {
+                       retval = cortex_a8_check_address(target, address);
+                       if (retval != ERROR_OK)
+                               return retval;
+                       /*  enable mmu */
+                       retval = cortex_a8_mmu_modify(target, 1);
+                       if (retval != ERROR_OK)
+                               return retval;
+               }
                retval = cortex_a8_read_apb_ab_memory(target, address, size, count, buffer);
        }
        return retval;
 }
 
 static int cortex_a8_write_phys_memory(struct target *target,
-                uint32_t address, uint32_t size,
-                uint32_t count, const uint8_t *buffer)
+       uint32_t address, uint32_t size,
+       uint32_t count, const uint8_t *buffer)
 {
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int retval = ERROR_COMMAND_SYNTAX_ERROR;
        uint8_t apsel = swjdp->apsel;
 
        LOG_DEBUG("Writing memory to real address 0x%x; size %d; count %d", address,
-                       size, count);
+               size, count);
 
        if (count && buffer) {
 
-               if ( apsel == swjdp_memoryap ) {
+               if (armv7a->memory_ap_available && (apsel == armv7a->memory_ap)) {
 
                        /* write memory through AHB-AP */
 
                        switch (size) {
                                case 4:
-                                       retval = mem_ap_sel_write_buf_u32(swjdp, swjdp_memoryap,
-                                                       buffer, 4 * count, address);
+                                       retval = mem_ap_sel_write_buf_u32(swjdp, armv7a->memory_ap,
+                                               buffer, 4 * count, address);
                                        break;
                                case 2:
-                                       retval = mem_ap_sel_write_buf_u16(swjdp, swjdp_memoryap,
-                                                       buffer, 2 * count, address);
+                                       retval = mem_ap_sel_write_buf_u16(swjdp, armv7a->memory_ap,
+                                               buffer, 2 * count, address);
                                        break;
                                case 1:
-                                       retval = mem_ap_sel_write_buf_u8(swjdp, swjdp_memoryap,
-                                                       buffer, count, address);
+                                       retval = mem_ap_sel_write_buf_u8(swjdp, armv7a->memory_ap,
+                                               buffer, count, address);
                                        break;
                        }
 
                } else {
 
                        /* write memory through APB-AP */
-                       retval = cortex_a8_mmu_modify(target, 0);
-                       if (retval != ERROR_OK)
-                               return retval;
-                       return  cortex_a8_write_apb_ab_memory(target, address, size, count, buffer);
+                       if (!armv7a->is_armv7r) {
+                               retval = cortex_a8_mmu_modify(target, 0);
+                               if (retval != ERROR_OK)
+                                       return retval;
+                       }
+                       return cortex_a8_write_apb_ab_memory(target, address, size, count, buffer);
                }
        }
 
 
        /* REVISIT this op is generic ARMv7-A/R stuff */
-       if (retval == ERROR_OK && target->state == TARGET_HALTED)
-       {
-               struct arm_dpm *dpm = armv7a->armv4_5_common.dpm;
+       if (retval == ERROR_OK && target->state == TARGET_HALTED) {
+               struct arm_dpm *dpm = armv7a->arm.dpm;
 
                retval = dpm->prepare(dpm);
                if (retval != ERROR_OK)
@@ -2189,15 +2253,14 @@ static int cortex_a8_write_phys_memory(struct target *target,
                 */
 
                /* invalidate I-Cache */
-               if (armv7a->armv7a_mmu.armv7a_cache.i_cache_enabled)
-               {
+               if (armv7a->armv7a_mmu.armv7a_cache.i_cache_enabled) {
                        /* ICIMVAU - Invalidate Cache single entry
                         * with MVA to PoU
                         *      MCR p15, 0, r0, c7, c5, 1
                         */
                        for (uint32_t cacheline = address;
-                                       cacheline < address + size * count;
-                                       cacheline += 64) {
+                               cacheline < address + size * count;
+                               cacheline += 64) {
                                retval = dpm->instr_write_data_r0(dpm,
                                                ARMV4_5_MCR(15, 0, 0, 7, 5, 1),
                                                cacheline);
@@ -2207,15 +2270,14 @@ static int cortex_a8_write_phys_memory(struct target *target,
                }
 
                /* invalidate D-Cache */
-               if (armv7a->armv7a_mmu.armv7a_cache.d_u_cache_enabled)
-               {
+               if (armv7a->armv7a_mmu.armv7a_cache.d_u_cache_enabled) {
                        /* DCIMVAC - Invalidate data Cache line
                         * with MVA to PoC
                         *      MCR p15, 0, r0, c7, c6, 1
                         */
                        for (uint32_t cacheline = address;
-                                       cacheline < address + size * count;
-                                       cacheline += 64) {
+                               cacheline < address + size * count;
+                               cacheline += 64) {
                                retval = dpm->instr_write_data_r0(dpm,
                                                ARMV4_5_MCR(15, 0, 0, 7, 6, 1),
                                                cacheline);
@@ -2231,60 +2293,60 @@ static int cortex_a8_write_phys_memory(struct target *target,
 }
 
 static int cortex_a8_write_memory(struct target *target, uint32_t address,
-                uint32_t size, uint32_t count, const uint8_t *buffer)
+       uint32_t size, uint32_t count, const uint8_t *buffer)
 {
        int enabled = 0;
        uint32_t virt, phys;
        int retval;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        uint8_t apsel = swjdp->apsel;
        /* cortex_a8 handles unaligned memory access */
-       LOG_DEBUG("Reading memory at address 0x%x; size %d; count %d", address,
-                       size, count);
-       if (apsel == swjdp_memoryap) {
-
-               LOG_DEBUG("Writing memory to address 0x%x; size %d; count %d", address, size, count);
-               retval = cortex_a8_mmu(target, &enabled);
-               if (retval != ERROR_OK)
-                       return retval;
-
-               if(enabled)
-               {
-                       virt = address;
-                       retval = cortex_a8_virt2phys(target, virt, &phys);
+       LOG_DEBUG("Writing memory at address 0x%x; size %d; count %d", address,
+               size, count);
+       if (armv7a->memory_ap_available && (apsel == armv7a->memory_ap)) {
+
+               LOG_DEBUG("Writing memory to address 0x%x; size %d; count %d", address, size,
+                       count);
+               if (!armv7a->is_armv7r) {
+                       retval = cortex_a8_mmu(target, &enabled);
                        if (retval != ERROR_OK)
                                return retval;
-                       LOG_DEBUG("Writing to virtual address. Translating v:0x%x to r:0x%x", virt, phys);
-                       address = phys;
+
+                       if (enabled) {
+                               virt = address;
+                               retval = cortex_a8_virt2phys(target, virt, &phys);
+                               if (retval != ERROR_OK)
+                                       return retval;
+                               LOG_DEBUG("Writing to virtual address. Translating v:0x%x to r:0x%x",
+                                       virt,
+                                       phys);
+                               address = phys;
+                       }
                }
 
-               retval = cortex_a8_write_phys_memory(target, address, size, 
+               retval = cortex_a8_write_phys_memory(target, address, size,
                                count, buffer);
-       }
-       else {
-               retval = cortex_a8_check_address(target, address);
-               if (retval != ERROR_OK) return retval;
-               /*  enable mmu  */
-               retval = cortex_a8_mmu_modify(target, 1);
-               if (retval != ERROR_OK) return retval;
+       } else {
+               if (!armv7a->is_armv7r) {
+                       retval = cortex_a8_check_address(target, address);
+                       if (retval != ERROR_OK)
+                               return retval;
+                       /*  enable mmu  */
+                       retval = cortex_a8_mmu_modify(target, 1);
+                       if (retval != ERROR_OK)
+                               return retval;
+               }
                retval = cortex_a8_write_apb_ab_memory(target, address, size, count, buffer);
        }
-    return retval;
-}
-
-static int cortex_a8_bulk_write_memory(struct target *target, uint32_t address,
-               uint32_t count, const uint8_t *buffer)
-{
-       return cortex_a8_write_memory(target, address, 4, count, buffer);
+       return retval;
 }
 
-
 static int cortex_a8_handle_target_request(void *priv)
 {
        struct target *target = priv;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int retval;
 
        if (!target_was_examined(target))
@@ -2292,23 +2354,20 @@ static int cortex_a8_handle_target_request(void *priv)
        if (!target->dbg_msg_enabled)
                return ERROR_OK;
 
-       if (target->state == TARGET_RUNNING)
-       {
+       if (target->state == TARGET_RUNNING) {
                uint32_t request;
                uint32_t dscr;
-               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                                       armv7a->debug_base      + CPUDBG_DSCR, &dscr);
+               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
 
                /* check if we have data */
-               while ((dscr & DSCR_DTR_TX_FULL) && (retval==ERROR_OK))
-               {
-                       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                                       armv7a->debug_base+ CPUDBG_DTRTX, &request);
-                       if (retval == ERROR_OK)
-                       {
+               while ((dscr & DSCR_DTR_TX_FULL) && (retval == ERROR_OK)) {
+                       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                                       armv7a->debug_base + CPUDBG_DTRTX, &request);
+                       if (retval == ERROR_OK) {
                                target_request(target, request);
-                               retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                                               armv7a->debug_base+ CPUDBG_DSCR, &dscr);
+                               retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                                               armv7a->debug_base + CPUDBG_DSCR, &dscr);
                        }
                }
        }
@@ -2324,7 +2383,7 @@ static int cortex_a8_examine_first(struct target *target)
 {
        struct cortex_a8_common *cortex_a8 = target_to_cortex_a8(target);
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        int i;
        int retval = ERROR_OK;
        uint32_t didr, ctypr, ttypr, cpuid;
@@ -2336,8 +2395,24 @@ static int cortex_a8_examine_first(struct target *target)
        if (retval != ERROR_OK)
                return retval;
 
-       if (!target->dbgbase_set)
-       {
+       /* Search for the APB-AB - it is needed for access to debug registers */
+       retval = dap_find_ap(swjdp, AP_TYPE_APB_AP, &armv7a->debug_ap);
+       if (retval != ERROR_OK) {
+               LOG_ERROR("Could not find APB-AP for debug access");
+               return retval;
+       }
+       /* Search for the AHB-AB */
+       retval = dap_find_ap(swjdp, AP_TYPE_AHB_AP, &armv7a->memory_ap);
+       if (retval != ERROR_OK) {
+               /* AHB-AP not found - use APB-AP */
+               LOG_DEBUG("Could not find AHB-AP - using APB-AP for memory access");
+               armv7a->memory_ap_available = false;
+       } else {
+               armv7a->memory_ap_available = true;
+       }
+
+
+       if (!target->dbgbase_set) {
                uint32_t dbgbase;
                /* Get ROM Table base */
                uint32_t apid;
@@ -2349,41 +2424,38 @@ static int cortex_a8_examine_first(struct target *target)
                                &armv7a->debug_base);
                if (retval != ERROR_OK)
                        return retval;
-       }
-    else
-       {
+       } else
                armv7a->debug_base = target->dbgbase;
-       }
 
-       retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
                        armv7a->debug_base + CPUDBG_CPUID, &cpuid);
        if (retval != ERROR_OK)
                return retval;
 
-       if ((retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_CPUID, &cpuid)) != ERROR_OK)
-       {
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_CPUID, &cpuid);
+       if (retval != ERROR_OK) {
                LOG_DEBUG("Examine %s failed", "CPUID");
                return retval;
        }
 
-       if ((retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_CTYPR, &ctypr)) != ERROR_OK)
-       {
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_CTYPR, &ctypr);
+       if (retval != ERROR_OK) {
                LOG_DEBUG("Examine %s failed", "CTYPR");
                return retval;
        }
 
-       if ((retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_TTYPR, &ttypr)) != ERROR_OK)
-       {
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_TTYPR, &ttypr);
+       if (retval != ERROR_OK) {
                LOG_DEBUG("Examine %s failed", "TTYPR");
                return retval;
        }
 
-       if ((retval = mem_ap_sel_read_atomic_u32(swjdp, swjdp_debugap,
-                       armv7a->debug_base + CPUDBG_DIDR, &didr)) != ERROR_OK)
-       {
+       retval = mem_ap_sel_read_atomic_u32(swjdp, armv7a->debug_ap,
+                       armv7a->debug_base + CPUDBG_DIDR, &didr);
+       if (retval != ERROR_OK) {
                LOG_DEBUG("Examine %s failed", "DIDR");
                return retval;
        }
@@ -2393,7 +2465,7 @@ static int cortex_a8_examine_first(struct target *target)
        LOG_DEBUG("ttypr = 0x%08" PRIx32, ttypr);
        LOG_DEBUG("didr = 0x%08" PRIx32, didr);
 
-       armv7a->armv4_5_common.core_type = ARM_MODE_MON;
+       armv7a->arm.core_type = ARM_MODE_MON;
        retval = cortex_a8_dpm_setup(cortex_a8, didr);
        if (retval != ERROR_OK)
                return retval;
@@ -2403,9 +2475,8 @@ static int cortex_a8_examine_first(struct target *target)
        cortex_a8->brp_num_context = ((didr >> 20) & 0x0F) + 1;
        cortex_a8->brp_num_available = cortex_a8->brp_num;
        cortex_a8->brp_list = calloc(cortex_a8->brp_num, sizeof(struct cortex_a8_brp));
-//     cortex_a8->brb_enabled = ????;
-       for (i = 0; i < cortex_a8->brp_num; i++)
-       {
+/*     cortex_a8->brb_enabled = ????; */
+       for (i = 0; i < cortex_a8->brp_num; i++) {
                cortex_a8->brp_list[i].used = 0;
                if (i < (cortex_a8->brp_num-cortex_a8->brp_num_context))
                        cortex_a8->brp_list[i].type = BRP_NORMAL;
@@ -2442,42 +2513,40 @@ static int cortex_a8_examine(struct target *target)
  */
 
 static int cortex_a8_init_target(struct command_context *cmd_ctx,
-               struct target *target)
+       struct target *target)
 {
        /* examine_first() does a bunch of this */
        return ERROR_OK;
 }
 
 static int cortex_a8_init_arch_info(struct target *target,
-               struct cortex_a8_common *cortex_a8, struct jtag_tap *tap)
+       struct cortex_a8_common *cortex_a8, struct jtag_tap *tap)
 {
        struct armv7a_common *armv7a = &cortex_a8->armv7a_common;
        struct adiv5_dap *dap = &armv7a->dap;
 
-       armv7a->armv4_5_common.dap = dap;
+       armv7a->arm.dap = dap;
 
        /* Setup struct cortex_a8_common */
        cortex_a8->common_magic = CORTEX_A8_COMMON_MAGIC;
        /*  tap has no dap initialized */
-       if (!tap->dap)
-       {
-       armv7a->armv4_5_common.dap = dap;
-       /* Setup struct cortex_a8_common */
+       if (!tap->dap) {
+               armv7a->arm.dap = dap;
+               /* Setup struct cortex_a8_common */
 
-       /* prepare JTAG information for the new target */
-       cortex_a8->jtag_info.tap = tap;
-       cortex_a8->jtag_info.scann_size = 4;
+               /* prepare JTAG information for the new target */
+               cortex_a8->jtag_info.tap = tap;
+               cortex_a8->jtag_info.scann_size = 4;
 
-       /* Leave (only) generic DAP stuff for debugport_init() */
-       dap->jtag_info = &cortex_a8->jtag_info;
+               /* Leave (only) generic DAP stuff for debugport_init() */
+               dap->jtag_info = &cortex_a8->jtag_info;
 
-       /* Number of bits for tar autoincrement, impl. dep. at least 10 */
-       dap->tar_autoincr_block = (1 << 10);
-       dap->memaccess_tck = 80;
-       tap->dap = dap;
-    }
-       else
-       armv7a->armv4_5_common.dap = tap->dap;
+               /* Number of bits for tar autoincrement, impl. dep. at least 10 */
+               dap->tar_autoincr_block = (1 << 10);
+               dap->memaccess_tck = 80;
+               tap->dap = dap;
+       } else
+               armv7a->arm.dap = tap->dap;
 
        cortex_a8->fast_reg_read = 0;
 
@@ -2490,8 +2559,8 @@ static int cortex_a8_init_arch_info(struct target *target,
 
        armv7a->armv7a_mmu.read_physical_memory = cortex_a8_read_phys_memory;
 
-       
-//     arm7_9->handle_target_request = cortex_a8_handle_target_request;
+
+/*     arm7_9->handle_target_request = cortex_a8_handle_target_request; */
 
        /* REVISIT v7a setup should be in a v7a-specific routine */
        armv7a_init_arch_info(target, armv7a);
@@ -2504,9 +2573,19 @@ static int cortex_a8_target_create(struct target *target, Jim_Interp *interp)
 {
        struct cortex_a8_common *cortex_a8 = calloc(1, sizeof(struct cortex_a8_common));
 
+       cortex_a8->armv7a_common.is_armv7r = false;
+
        return cortex_a8_init_arch_info(target, cortex_a8, target->tap);
 }
 
+static int cortex_r4_target_create(struct target *target, Jim_Interp *interp)
+{
+       struct cortex_a8_common *cortex_a8 = calloc(1, sizeof(struct cortex_a8_common));
+
+       cortex_a8->armv7a_common.is_armv7r = true;
+
+       return cortex_a8_init_arch_info(target, cortex_a8, target->tap);
+}
 
 
 static int cortex_a8_mmu(struct target *target, int *enabled)
@@ -2521,26 +2600,24 @@ static int cortex_a8_mmu(struct target *target, int *enabled)
 }
 
 static int cortex_a8_virt2phys(struct target *target,
-               uint32_t virt, uint32_t *phys)
+       uint32_t virt, uint32_t *phys)
 {
        int retval = ERROR_FAIL;
        struct armv7a_common *armv7a = target_to_armv7a(target);
-       struct adiv5_dap *swjdp = armv7a->armv4_5_common.dap;
+       struct adiv5_dap *swjdp = armv7a->arm.dap;
        uint8_t apsel = swjdp->apsel;
-       if (apsel == swjdp_memoryap)
-       {
+       if (armv7a->memory_ap_available && (apsel == armv7a->memory_ap)) {
                uint32_t ret;
                retval = armv7a_mmu_translate_va(target,
-                                virt, &ret);
+                               virt, &ret);
                if (retval != ERROR_OK)
                        goto done;
                *phys = ret;
-       } 
-       else
-       { /*  use this method if swjdp_memoryap not selected */
-               /*  mmu must be enable in order to get a correct translation */
+       } else {/*  use this method if armv7a->memory_ap not selected
+                *  mmu must be enable in order to get a correct translation */
                retval = cortex_a8_mmu_modify(target, 1);
-               if (retval != ERROR_OK) goto done;
+               if (retval != ERROR_OK)
+                       goto done;
                retval = armv7a_mmu_translate_va_pa(target, virt,  phys, 1);
        }
 done:
@@ -2560,8 +2637,7 @@ COMMAND_HANDLER(cortex_a8_handle_cache_info_command)
 COMMAND_HANDLER(cortex_a8_handle_dbginit_command)
 {
        struct target *target = get_current_target(CMD_CTX);
-       if (!target_was_examined(target))
-       {
+       if (!target_was_examined(target)) {
                LOG_ERROR("target not examined yet");
                return ERROR_FAIL;
        }
@@ -2571,15 +2647,13 @@ COMMAND_HANDLER(cortex_a8_handle_dbginit_command)
 COMMAND_HANDLER(cortex_a8_handle_smp_off_command)
 {
        struct target *target = get_current_target(CMD_CTX);
-    /* check target is an smp target */
-    struct target_list *head;
+       /* check target is an smp target */
+       struct target_list *head;
        struct target *curr;
        head = target->head;
        target->smp = 0;
-       if (head != (struct target_list*)NULL)
-       {
-               while (head != (struct target_list*)NULL)
-               {
+       if (head != (struct target_list *)NULL) {
+               while (head != (struct target_list *)NULL) {
                        curr = head->target;
                        curr->smp = 0;
                        head = head->next;
@@ -2596,10 +2670,9 @@ COMMAND_HANDLER(cortex_a8_handle_smp_on_command)
        struct target_list *head;
        struct target *curr;
        head = target->head;
-       if (head != (struct target_list*)NULL)
-       {   target->smp=1;
-               while (head != (struct target_list*)NULL)
-               {
+       if (head != (struct target_list *)NULL) {
+               target->smp = 1;
+               while (head != (struct target_list *)NULL) {
                        curr = head->target;
                        curr->smp = 1;
                        head = head->next;
@@ -2614,19 +2687,17 @@ COMMAND_HANDLER(cortex_a8_handle_smp_gdb_command)
        int retval = ERROR_OK;
        struct target_list *head;
        head = target->head;
-       if (head != (struct target_list*)NULL)
-       {
-               if (CMD_ARGC == 1)
-               {
+       if (head != (struct target_list *)NULL) {
+               if (CMD_ARGC == 1) {
                        int coreid = 0;
                        COMMAND_PARSE_NUMBER(int, CMD_ARGV[0], coreid);
                        if (ERROR_OK != retval)
                                return retval;
-                       target->gdb_service->core[1]=coreid;
+                       target->gdb_service->core[1] = coreid;
 
                }
                command_print(CMD_CTX, "gdb coreid  %d -> %d", target->gdb_service->core[0]
-                               , target->gdb_service->core[1]);
+                       , target->gdb_service->core[1]);
        }
        return ERROR_OK;
 }
@@ -2646,22 +2717,21 @@ static const struct command_registration cortex_a8_exec_command_handlers[] = {
                .help = "Initialize core debug",
                .usage = "",
        },
-       {   .name ="smp_off",
+       {   .name = "smp_off",
            .handler = cortex_a8_handle_smp_off_command,
-               .mode = COMMAND_EXEC,
-               .help = "Stop smp handling",
-               .usage = "",
-       },
+           .mode = COMMAND_EXEC,
+           .help = "Stop smp handling",
+           .usage = "",},
        {
-        .name ="smp_on",
-        .handler = cortex_a8_handle_smp_on_command,
+               .name = "smp_on",
+               .handler = cortex_a8_handle_smp_on_command,
                .mode = COMMAND_EXEC,
                .help = "Restart smp handling",
                .usage = "",
        },
        {
-        .name ="smp_gdb",
-        .handler = cortex_a8_handle_smp_gdb_command,
+               .name = "smp_gdb",
+               .handler = cortex_a8_handle_smp_gdb_command,
                .mode = COMMAND_EXEC,
                .help = "display/fix current core played to gdb",
                .usage = "",
@@ -2678,9 +2748,9 @@ static const struct command_registration cortex_a8_command_handlers[] = {
                .chain = armv7a_command_handlers,
        },
        {
-               .name = "cortex_a8",
+               .name = "cortex_a",
                .mode = COMMAND_ANY,
-               .help = "Cortex-A8 command group",
+               .help = "Cortex-A command group",
                .usage = "",
                .chain = cortex_a8_exec_command_handlers,
        },
@@ -2688,7 +2758,8 @@ static const struct command_registration cortex_a8_command_handlers[] = {
 };
 
 struct target_type cortexa8_target = {
-       .name = "cortex_a8",
+       .name = "cortex_a",
+       .deprecated_name = "cortex_a8",
 
        .poll = cortex_a8_poll,
        .arch_state = armv7a_arch_state,
@@ -2708,7 +2779,6 @@ struct target_type cortexa8_target = {
 
        .read_memory = cortex_a8_read_memory,
        .write_memory = cortex_a8_write_memory,
-       .bulk_write_memory = cortex_a8_bulk_write_memory,
 
        .checksum_memory = arm_checksum_memory,
        .blank_check_memory = arm_blank_check_memory,
@@ -2732,3 +2802,78 @@ struct target_type cortexa8_target = {
        .mmu = cortex_a8_mmu,
        .virt2phys = cortex_a8_virt2phys,
 };
+
+static const struct command_registration cortex_r4_exec_command_handlers[] = {
+       {
+               .name = "cache_info",
+               .handler = cortex_a8_handle_cache_info_command,
+               .mode = COMMAND_EXEC,
+               .help = "display information about target caches",
+               .usage = "",
+       },
+       {
+               .name = "dbginit",
+               .handler = cortex_a8_handle_dbginit_command,
+               .mode = COMMAND_EXEC,
+               .help = "Initialize core debug",
+               .usage = "",
+       },
+
+       COMMAND_REGISTRATION_DONE
+};
+static const struct command_registration cortex_r4_command_handlers[] = {
+       {
+               .chain = arm_command_handlers,
+       },
+       {
+               .chain = armv7a_command_handlers,
+       },
+       {
+               .name = "cortex_r4",
+               .mode = COMMAND_ANY,
+               .help = "Cortex-R4 command group",
+               .usage = "",
+               .chain = cortex_r4_exec_command_handlers,
+       },
+       COMMAND_REGISTRATION_DONE
+};
+
+struct target_type cortexr4_target = {
+       .name = "cortex_r4",
+
+       .poll = cortex_a8_poll,
+       .arch_state = armv7a_arch_state,
+
+       .target_request_data = NULL,
+
+       .halt = cortex_a8_halt,
+       .resume = cortex_a8_resume,
+       .step = cortex_a8_step,
+
+       .assert_reset = cortex_a8_assert_reset,
+       .deassert_reset = cortex_a8_deassert_reset,
+       .soft_reset_halt = NULL,
+
+       /* REVISIT allow exporting VFP3 registers ... */
+       .get_gdb_reg_list = arm_get_gdb_reg_list,
+
+       .read_memory = cortex_a8_read_memory,
+       .write_memory = cortex_a8_write_memory,
+
+       .checksum_memory = arm_checksum_memory,
+       .blank_check_memory = arm_blank_check_memory,
+
+       .run_algorithm = armv4_5_run_algorithm,
+
+       .add_breakpoint = cortex_a8_add_breakpoint,
+       .add_context_breakpoint = cortex_a8_add_context_breakpoint,
+       .add_hybrid_breakpoint = cortex_a8_add_hybrid_breakpoint,
+       .remove_breakpoint = cortex_a8_remove_breakpoint,
+       .add_watchpoint = NULL,
+       .remove_watchpoint = NULL,
+
+       .commands = cortex_r4_command_handlers,
+       .target_create = cortex_r4_target_create,
+       .init_target = cortex_a8_init_target,
+       .examine = cortex_a8_examine,
+};

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)