- ST STM32x cortex support added

- ST STM32x flash support added
- cleaned up armv7m and cortex-m3 support, removed luminary specific code
- cortex-m3 16bit read/write added (required for STM32x flash programming)

git-svn-id: svn://svn.berlios.de/openocd/trunk@177 b42882b7-edfa-0310-969c-e2dbd0fdcd60

9 files changed, 322 insertions, 319 deletions

diff --git a/src/target/arm_disassembler.c b/src/target/arm_disassembler.c
index ed7fe655..7028e9f5 100644
--- a/src/target/arm_disassembler.c
+++ b/src/target/arm_disassembler.c
@@ -346,7 +346,7 @@ int evaluate_load_store(u32 opcode, u32 address, arm_instruction_t *instruction)
 		if (offset_12)
 			snprintf(offset, 32, ", #%s0x%x", (U) ? "" : "-", offset_12);
 		else
-			snprintf(offset, 32, "");
+			snprintf(offset, 32, "%s", "");
 		
 		instruction->info.load_store.offset_mode = 0;
 		instruction->info.load_store.offset.offset = offset_12;
diff --git a/src/target/armv7m.c b/src/target/armv7m.c
index d77eb0bd..335d0acc 100644
--- a/src/target/armv7m.c
+++ b/src/target/armv7m.c
@@ -129,8 +129,10 @@ enum armv7m_runcontext armv7m_get_context(target_t *target)
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	
-	if (armv7m->process_context == armv7m->core_cache) return ARMV7M_PROCESS_CONTEXT;
-	if (armv7m->debug_context == armv7m->core_cache) return ARMV7M_DEBUG_CONTEXT;
+	if (armv7m->process_context == armv7m->core_cache)
+		return ARMV7M_PROCESS_CONTEXT;
+	if (armv7m->debug_context == armv7m->core_cache)
+		return ARMV7M_DEBUG_CONTEXT;
 	
 	ERROR("Invalid runcontext");
 	exit(-1);
@@ -177,9 +179,11 @@ int armv7m_use_context(target_t *target, enum armv7m_runcontext new_ctx)
 char enamebuf[32];
 char *armv7m_exception_string(int number)
 {
-	if ((number<0)|(number>511)) return "Invalid exception";
-	if (number<16) return armv7m_exception_strings[number];
-	sprintf(enamebuf,"External Interrupt(%i)",number-16);
+	if ((number < 0) | (number > 511))
+		return "Invalid exception";
+	if (number < 16)
+		return armv7m_exception_strings[number];
+	sprintf(enamebuf, "External Interrupt(%i)", number - 16);
 	return enamebuf;
 }
 
@@ -252,26 +256,22 @@ int armv7m_write_core_reg(struct target_s *target, int num)
 	if ((num < 0) || (num >= ARMV7NUMCOREREGS))
 		return ERROR_INVALID_ARGUMENTS;
 	
-	
 	reg_value = buf_get_u32(armv7m->core_cache->reg_list[num].value, 0, 32);
 	armv7m_core_reg = armv7m->core_cache->reg_list[num].arch_info;
 	retval = armv7m->store_core_reg_u32(target, armv7m_core_reg->type, armv7m_core_reg->num, reg_value);
 	if (retval != ERROR_OK)
 	{
-			ERROR("JTAG failure");
-			armv7m->core_cache->reg_list[num].dirty=1;
-			return ERROR_JTAG_DEVICE_ERROR;
+		ERROR("JTAG failure");
+		armv7m->core_cache->reg_list[num].dirty=1;
+		return ERROR_JTAG_DEVICE_ERROR;
 	}
 	DEBUG("write core reg %i value 0x%x",num ,reg_value);
 	armv7m->core_cache->reg_list[num].valid=1;
 	armv7m->core_cache->reg_list[num].dirty=0;
 	
 	return ERROR_OK;
-	
 }
 
-
-
 int armv7m_invalidate_core_regs(target_t *target)
 {
 	/* get pointers to arch-specific information */
@@ -287,14 +287,12 @@ int armv7m_invalidate_core_regs(target_t *target)
 	return ERROR_OK;
 }
 
-
 int armv7m_get_gdb_reg_list(target_t *target, reg_t **reg_list[], int *reg_list_size)
 {
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	int i;
 	
-	
 	if (target->state != TARGET_HALTED)
 	{
 		return ERROR_TARGET_NOT_HALTED;
@@ -306,7 +304,7 @@ int armv7m_get_gdb_reg_list(target_t *target, reg_t **reg_list[], int *reg_list_
 	/* TODOLATER correct list of registers, names ? */
 	for (i = 0; i < *reg_list_size; i++)
 	{
-		if (i<ARMV7NUMCOREREGS)
+		if (i < ARMV7NUMCOREREGS)
 			(*reg_list)[i] = &armv7m->process_context->reg_list[i];
 			//(*reg_list)[i] = &armv7m->core_cache->reg_list[i];
 		else
@@ -382,7 +380,7 @@ int armv7m_run_algorithm(struct target_s *target, int num_mem_params, mem_param_
 		armv7m_set_core_reg(reg, reg_params[i].value);
 	}
 	
-	/* ARMV7M always runs in Tumb state */
+	/* ARMV7M always runs in Thumb state */
 	exit_breakpoint_size = 2;
 	if ((retval = breakpoint_add(target, exit_point, exit_breakpoint_size, BKPT_SOFT)) != ERROR_OK)
 	{
@@ -415,7 +413,7 @@ int armv7m_run_algorithm(struct target_s *target, int num_mem_params, mem_param_
 				}
 			}
 			armv7m->load_core_reg_u32(target, ARMV7M_REGISTER_CORE_GP, 15, &pc);
-			DEBUG("failed algoritm halted at 0x%x ",pc); 
+			DEBUG("failed algoritm halted at 0x%x ", pc); 
 			retval = ERROR_TARGET_TIMEOUT;
 		}
 	}
@@ -450,7 +448,7 @@ int armv7m_run_algorithm(struct target_s *target, int num_mem_params, mem_param_
 				exit(-1);
 			}
 			
-			armv7m_core_reg_t * armv7m_core_reg = reg->arch_info;
+			armv7m_core_reg_t *armv7m_core_reg = reg->arch_info;
 			//armv7m->load_core_reg_u32(target, armv7m_core_reg->type, armv7m_core_reg->num, &regvalue);
 			//buf_set_u32(reg_params[i].value, 0, 32, regvalue);
 			buf_set_u32(reg_params[i].value, 0, 32, buf_get_u32(reg->value, 0, 32));
@@ -464,11 +462,9 @@ int armv7m_run_algorithm(struct target_s *target, int num_mem_params, mem_param_
 	//	armv7m->core_cache->reg_list[i].dirty = 1;
 	//}
 
-	
 	// ????armv7m->core_state = core_state;
 	// ????armv7m->core_mode = core_mode;
 
-
 	return retval;
 }
 
@@ -478,20 +474,19 @@ int armv7m_arch_state(struct target_s *target, char *buf, int buf_size)
 	armv7m_common_t *armv7m = target->arch_info;
 	
 	snprintf(buf, buf_size,
-			 "target halted in %s state due to %s, current mode: %s %s\nxPSR: 0x%8.8x pc: 0x%8.8x",
-			 armv7m_state_strings[armv7m->core_state],
-			 target_debug_reason_strings[target->debug_reason],
-			 armv7m_mode_strings[armv7m->core_mode],
-			 armv7m_exception_string(armv7m->exception_number),
-			 buf_get_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0, 32),
-			 buf_get_u32(armv7m->core_cache->reg_list[15].value, 0, 32));
+		 "target halted in %s state due to %s, current mode: %s %s\nxPSR: 0x%8.8x pc: 0x%8.8x",
+		 armv7m_state_strings[armv7m->core_state],
+		 target_debug_reason_strings[target->debug_reason],
+		 armv7m_mode_strings[armv7m->core_mode],
+		 armv7m_exception_string(armv7m->exception_number),
+		 buf_get_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0, 32),
+		 buf_get_u32(armv7m->core_cache->reg_list[15].value, 0, 32));
 	
 	return ERROR_OK;
 }
 
 reg_cache_t *armv7m_build_reg_cache(target_t *target)
 {
-
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	arm_jtag_t *jtag_info = &armv7m->jtag_info;
@@ -560,16 +555,13 @@ reg_cache_t *armv7m_build_reg_cache(target_t *target)
 
 int armv7m_init_target(struct command_context_s *cmd_ctx, struct target_s *target)
 {
-	
 	armv7m_build_reg_cache(target);
 	
 	return ERROR_OK;
-	
 }
 
 int armv7m_init_arch_info(target_t *target, armv7m_common_t *armv7m)
 {
-
 	/* register arch-specific functions */
 	
 	target->arch_info = armv7m;
@@ -580,11 +572,9 @@ int armv7m_init_arch_info(target_t *target, armv7m_common_t *armv7m)
 	return ERROR_OK;
 }
 
-
 int armv7m_register_commands(struct command_context_s *cmd_ctx)
 {
 	int retval;
 	
 	return ERROR_OK;
-
 }
diff --git a/src/target/armv7m.h b/src/target/armv7m.h
index cf2b3c70..49043e1d 100644
--- a/src/target/armv7m.h
+++ b/src/target/armv7m.h
@@ -26,7 +26,6 @@
 #include "target.h"
 #include "arm_jtag.h"
 
-
 enum armv7m_mode
 {
 	ARMV7M_MODE_HANDLER = 0, 
@@ -56,17 +55,18 @@ enum armv7m_runcontext
 };
 
 extern char* armv7m_state_strings[];
+extern char* armv7m_exception_strings[];
 
-//#define ARMV7NUMCOREREGS 23
+extern char *armv7m_exception_string(int number);
 
 /* offsets into armv7m core register cache */
 enum 
 {
 	ARMV7M_PC = 15,
 	ARMV7M_xPSR = 16,
-	ARMV7M_MSP ,
-	ARMV7M_PSP ,
-	ARMV7M_PRIMASK ,
+	ARMV7M_MSP,
+	ARMV7M_PSP,
+	ARMV7M_PRIMASK,
 	ARMV7M_BASEPRI,
 	ARMV7M_FAULTMASK,
 	ARMV7M_CONTROL,
@@ -163,8 +163,9 @@ extern int armv7m_run_algorithm(struct target_s *target, int num_mem_params, mem
 
 extern int armv7m_invalidate_core_regs(target_t *target);
 
-
-
+extern enum armv7m_runcontext armv7m_get_context(target_t *target);
+extern int armv7m_use_context(target_t *target, enum armv7m_runcontext new_ctx);
+extern enum armv7m_runcontext armv7m_get_context(target_t *target);
 
 /* Thumb mode instructions
  */
diff --git a/src/target/breakpoints.c b/src/target/breakpoints.c
index 8b140b3e..ac8d7dfd 100644
--- a/src/target/breakpoints.c
+++ b/src/target/breakpoints.c
@@ -71,13 +71,15 @@ int breakpoint_add(target_t *target, u32 address, u32 length, enum breakpoint_ty
 		{
 			case ERROR_TARGET_RESOURCE_NOT_AVAILABLE:
 				INFO("can't add %s breakpoint, resource not available", breakpoint_type_strings[(*breakpoint_p)->type]);
-				free (*breakpoint_p);
+				free((*breakpoint_p)->orig_instr);
+				free(*breakpoint_p);
 				*breakpoint_p = NULL;
 				return retval;
 				break;
 			case ERROR_TARGET_NOT_HALTED:
 				INFO("can't add breakpoint while target is running");
-				free (*breakpoint_p);
+				free((*breakpoint_p)->orig_instr);
+				free(*breakpoint_p);
 				*breakpoint_p = NULL;
 				return retval;
 				break;
diff --git a/src/target/cortex_m3.c b/src/target/cortex_m3.c
index 93521cf6..b342fce2 100644
--- a/src/target/cortex_m3.c
+++ b/src/target/cortex_m3.c
@@ -28,6 +28,7 @@
 #include "replacements.h"
 
 #include "cortex_m3.h"
+#include "armv7m.h"
 
 #include "register.h"
 #include "target.h"
@@ -88,7 +89,6 @@ target_type_t cortexm3_target =
 
 int cortex_m3_clear_halt(target_t *target)
 {
-
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
@@ -110,11 +110,11 @@ int cortex_m3_single_step_core(target_t *target)
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
 	swjdp_common_t *swjdp = &cortex_m3->swjdp_info;
 
-	if (!(cortex_m3->dcb_dhcsr&C_MASKINTS))
+	if (!(cortex_m3->dcb_dhcsr & C_MASKINTS))
 		ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, DBGKEY | C_MASKINTS | C_HALT | C_DEBUGEN );
 	ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, DBGKEY | C_MASKINTS | C_STEP | C_DEBUGEN );
 	cortex_m3->dcb_dhcsr |= C_MASKINTS;
-	DEBUG("");
+	DEBUG(" ");
 	cortex_m3_clear_halt(target);
 	
 	return ERROR_OK;
@@ -134,7 +134,7 @@ int cortex_m3_exec_opcode(target_t *target,u32 opcode, int len /* MODE, r0_inval
 		ahbap_write_system_u32(swjdp, 0x20000000, opcode);
 		ahbap_write_coreregister_u32(swjdp, 0x20000000, 15);
 		cortex_m3_single_step_core(target);
-		armv7m->core_cache->reg_list[15].dirty=1;
+		armv7m->core_cache->reg_list[15].dirty = 1;
 		retvalue = ahbap_write_system_atomic_u32(swjdp, 0x20000000, savedram);		
 	}
 	
@@ -162,15 +162,15 @@ int cortex_m3_endreset_event(target_t *target)
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
 	swjdp_common_t *swjdp = &cortex_m3->swjdp_info;
 	cortex_m3_fp_comparator_t *fp_list = cortex_m3->fp_comparator_list; 
-	cortex_m3_dwt_comparator_t * dwt_list = cortex_m3->dwt_comparator_list;
+	cortex_m3_dwt_comparator_t *dwt_list = cortex_m3->dwt_comparator_list;
 
-	DEBUG("");
+	DEBUG(" ");
 	/* Enable debug requests */
 	ahbap_read_system_atomic_u32(swjdp, DCB_DHCSR, &cortex_m3->dcb_dhcsr);
 	if (!(cortex_m3->dcb_dhcsr&C_DEBUGEN))
 		ahbap_write_system_u32(swjdp, DCB_DHCSR, DBGKEY | C_DEBUGEN );
 	/* Enable trace and dwt */
-	ahbap_write_system_u32(swjdp, DCB_DEMCR, TRCENA|VC_HARDERR|VC_BUSERR|VC_CORERESET );
+	ahbap_write_system_u32(swjdp, DCB_DEMCR, TRCENA | VC_HARDERR | VC_BUSERR | VC_CORERESET );
 	/* Monitor bus faults */
 	ahbap_write_system_u32(swjdp, NVIC_SHCSR, SHCSR_BUSFAULTENA );
 
@@ -178,17 +178,17 @@ int cortex_m3_endreset_event(target_t *target)
 	target_write_u32(target, FP_CTRL, 3);
 
 	/* Restore FPB registers */
-	for (i=0;i<cortex_m3->fp_num_code+cortex_m3->fp_num_lit;i++)
+	for ( i = 0; i < cortex_m3->fp_num_code + cortex_m3->fp_num_lit; i++)
 	{
 		target_write_u32(target, fp_list[i].fpcr_address, fp_list[i].fpcr_value);
 	}
 	
 	/* Restore DWT registers */
-	for (i=0;i<cortex_m3->dwt_num_comp;i++)
+	for ( i = 0; i < cortex_m3->dwt_num_comp; i++)
 	{
 		target_write_u32(target, dwt_list[i].dwt_comparator_address, dwt_list[i].comp);
-		target_write_u32(target, dwt_list[i].dwt_comparator_address|0x4, dwt_list[i].mask);
-		target_write_u32(target, dwt_list[i].dwt_comparator_address|0x8, dwt_list[i].function);
+		target_write_u32(target, dwt_list[i].dwt_comparator_address | 0x4, dwt_list[i].mask);
+		target_write_u32(target, dwt_list[i].dwt_comparator_address | 0x8, dwt_list[i].function);
 	}
 	
 	/* Make sure working_areas are all free */
@@ -211,7 +211,7 @@ int cortex_m3_examine_debug_reason(target_t *target)
 	/* only check the debug reason if we don't know it already */
 	
 	if ((target->debug_reason != DBG_REASON_DBGRQ)
-			&& (target->debug_reason != DBG_REASON_SINGLESTEP))
+		&& (target->debug_reason != DBG_REASON_SINGLESTEP))
 	{
 
 		/*  INCOPMPLETE */
@@ -231,7 +231,7 @@ int cortex_m3_examine_debug_reason(target_t *target)
 
 int cortex_m3_examine_exception_reason(target_t *target)
 {
-	u32 shcsr,except_sr,cfsr=-1,except_ar=-1;
+	u32 shcsr, except_sr, cfsr = -1, except_ar = -1;
 
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
@@ -245,7 +245,7 @@ int cortex_m3_examine_exception_reason(target_t *target)
 			break;
 		case 3:	/* Hard Fault */
 			ahbap_read_system_atomic_u32(swjdp, NVIC_HFSR, &except_sr);
-			if (except_sr&0x40000000)
+			if (except_sr & 0x40000000)
 			{
 				ahbap_read_system_u32(swjdp, NVIC_CFSR, &cfsr);
 			}
@@ -262,28 +262,24 @@ int cortex_m3_examine_exception_reason(target_t *target)
 			ahbap_read_system_u32(swjdp, NVIC_CFSR, &except_sr);
 			break;
 		case 11:	/* SVCall */
-		
 			break;
 		case 12:	/* Debug Monitor */
 			ahbap_read_system_u32(swjdp, NVIC_DFSR, &except_sr);
 			break;
 		case 14:	/* PendSV */
-		
 			break;
 		case 15:	/* SysTick */
-		
 			break;
 		default:
 			except_sr = 0;
 			break;
-			
 	}
 	swjdp_transaction_endcheck(swjdp);
-    DEBUG("%s  SHCSR 0x%x,  SR 0x%x,  CFSR 0x%x, AR 0x%x",armv7m_exception_string(armv7m->exception_number),shcsr,except_sr,cfsr, except_ar);
+    DEBUG("%s SHCSR 0x%x, SR 0x%x, CFSR 0x%x, AR 0x%x", armv7m_exception_string(armv7m->exception_number), \
+    	shcsr, except_sr, cfsr, except_ar);
 	return ERROR_OK;
 }
 
-
 int cortex_m3_debug_entry(target_t *target)
 {
 	int i, irq_is_pending;
@@ -295,7 +291,7 @@ int cortex_m3_debug_entry(target_t *target)
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
 	swjdp_common_t *swjdp = &cortex_m3->swjdp_info;
 
-	DEBUG("");
+	DEBUG(" ");
 	if (armv7m->pre_debug_entry)
 		armv7m->pre_debug_entry(target);
 
@@ -316,10 +312,10 @@ int cortex_m3_debug_entry(target_t *target)
 	xPSR = buf_get_u32(armv7m->core_cache->reg_list[ARMV7M_xPSR].value, 0, 32);
 	
 	/* For IT instructions xPSR must be reloaded on resume and clear on debug exec*/
-	if (xPSR&0xf00)
+	if (xPSR & 0xf00)
 	{
 		armv7m->core_cache->reg_list[ARMV7M_xPSR].dirty = 1;
-		cortex_m3_store_core_reg_u32(target, ARMV7M_REGISTER_CORE_GP, 16, xPSR&~0xff);
+		cortex_m3_store_core_reg_u32(target, ARMV7M_REGISTER_CORE_GP, 16, xPSR &~ 0xff);
 	}
 
 
@@ -333,14 +329,14 @@ int cortex_m3_debug_entry(target_t *target)
 #endif
 
 	/* Are we in an exception handler */
-    armv7m->core_mode = (xPSR&0x1FF)?ARMV7M_MODE_HANDLER:ARMV7M_MODE_THREAD;
-    armv7m->exception_number = xPSR&0x1FF;;
+    armv7m->core_mode = (xPSR & 0x1FF) ? ARMV7M_MODE_HANDLER : ARMV7M_MODE_THREAD;
+    armv7m->exception_number = xPSR & 0x1FF;
 	if (armv7m->exception_number)
 	{
 		cortex_m3_examine_exception_reason(target);
 	}
 
-	DEBUG("entered debug state at PC 0x%x ", *(u32*)(armv7m->core_cache->reg_list[15].value), target_state_strings[target->state]);
+	DEBUG("entered debug state at PC 0x%x, target->state: %s ", *(u32*)(armv7m->core_cache->reg_list[15].value), target_state_strings[target->state]);
 
 	if (armv7m->post_debug_entry)
 		armv7m->post_debug_entry(target);
@@ -356,7 +352,7 @@ int cortex_m3_restore_context(target_t *target)
 	armv7m_common_t *armv7m = target->arch_info;
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
 
-	DEBUG("");
+	DEBUG(" ");
 
 	if (armv7m->pre_restore_context)
 		armv7m->pre_restore_context(target);
@@ -379,7 +375,6 @@ int cortex_m3_restore_context(target_t *target)
 	return ERROR_OK;		
 }
 
-
 enum target_state cortex_m3_poll(target_t *target)
 {
 	int retval;
@@ -425,13 +420,12 @@ enum target_state cortex_m3_poll(target_t *target)
 		}
 		if (prev_target_state == TARGET_DEBUG_RUNNING)
 		{
-			DEBUG("");
+			DEBUG(" ");
 			if ((retval = cortex_m3_debug_entry(target)) != ERROR_OK)
 				return retval;
 
 			target_call_event_callbacks(target, TARGET_EVENT_DEBUG_HALTED);
 		}
-
 	}
 		
 	/*
@@ -439,10 +433,9 @@ enum target_state cortex_m3_poll(target_t *target)
 		target->state = TARGET_SLEEP;
 	*/
 
-	
     /* Read Debug Fault Status Register, added to figure out the lockup when running flashtest.script  */
     ahbap_read_system_atomic_u32(swjdp, NVIC_DFSR, &cortex_m3->nvic_dfsr);
-	DEBUG("dcb_dhcsr %x, nvic_dfsr %x,  target->state: %s", cortex_m3->dcb_dhcsr, cortex_m3->nvic_dfsr, target_state_strings[target->state]);	
+	DEBUG("dcb_dhcsr %x, nvic_dfsr %x, target->state: %s", cortex_m3->dcb_dhcsr, cortex_m3->nvic_dfsr, target_state_strings[target->state]);	
 	return target->state;
 }
 
@@ -469,8 +462,8 @@ int cortex_m3_soft_reset_halt(struct target_s *target)
 	armv7m_common_t *armv7m = target->arch_info;
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
 	swjdp_common_t *swjdp = &cortex_m3->swjdp_info;
-	u32 dcb_dhcsr=0;
-	int retval, timeout=0;
+	u32 dcb_dhcsr = 0;
+	int retval, timeout = 0;
 	
 	/* Check that we are using process_context, or change and print warning */
 	if (armv7m_get_context(target) != ARMV7M_PROCESS_CONTEXT)
@@ -480,7 +473,7 @@ int cortex_m3_soft_reset_halt(struct target_s *target)
 	}
 
 	/* Enter debug state on reset, cf. end_reset_event() */
-	ahbap_write_system_u32(swjdp, DCB_DEMCR, TRCENA|VC_HARDERR|VC_BUSERR|VC_CORERESET );
+	ahbap_write_system_u32(swjdp, DCB_DEMCR, TRCENA | VC_HARDERR | VC_BUSERR | VC_CORERESET );
 	
 	/* Request a reset */ 
 	ahbap_write_system_atomic_u32(swjdp, NVIC_AIRCR, AIRCR_VECTKEY | AIRCR_VECTRESET );
@@ -495,14 +488,14 @@ int cortex_m3_soft_reset_halt(struct target_s *target)
 		if (retval == ERROR_OK)
 		{
 		    ahbap_read_system_atomic_u32(swjdp, NVIC_DFSR, &cortex_m3->nvic_dfsr);
-			if ( (dcb_dhcsr&S_HALT)&&(cortex_m3->nvic_dfsr&DFSR_VCATCH) )
+			if ((dcb_dhcsr&S_HALT) && (cortex_m3->nvic_dfsr & DFSR_VCATCH))
 			{
-				DEBUG("system reset-halted, dcb_dhcsr 0x%x, nvic_dfsr 0x%x",dcb_dhcsr,cortex_m3->nvic_dfsr);
+				DEBUG("system reset-halted, dcb_dhcsr 0x%x, nvic_dfsr 0x%x", dcb_dhcsr, cortex_m3->nvic_dfsr);
 				cortex_m3_poll(target);
 				return ERROR_OK;
 			}
 			else
-				DEBUG("waiting for system reset-halt, dcb_dhcsr 0x%x, %i ms",dcb_dhcsr,timeout);
+				DEBUG("waiting for system reset-halt, dcb_dhcsr 0x%x, %i ms", dcb_dhcsr, timeout);
 		}
 		timeout++;
 		usleep(1000);
@@ -534,14 +527,12 @@ int cortex_m3_resume(struct target_s *target, int current, u32 address, int hand
 			WARNING("Incorrect context in resume");
 			armv7m_use_context(target, ARMV7M_PROCESS_CONTEXT);
 		}
-
-
+		
 		target_free_all_working_areas(target);
 		cortex_m3_enable_breakpoints(target);
 		cortex_m3_enable_watchpoints(target);
 
 		/* TODOLATER Interrupt handling/disable for debug execution, cache ... ... */ 
-
 	}
 	
 	dcb_dhcsr = DBGKEY | C_DEBUGEN;
@@ -562,7 +553,7 @@ int cortex_m3_resume(struct target_s *target, int current, u32 address, int hand
 		buf_set_u32(armv7m->core_cache->reg_list[ARMV7M_PRIMASK].value, 0, 32, 1);
 		/* 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));
 	}
 
 	/* current = 1: continue on current pc, otherwise continue at <address> */
@@ -591,7 +582,7 @@ int cortex_m3_resume(struct target_s *target, int current, u32 address, int hand
 	}
 
 	/* Set/Clear C_MASKINTS in a separate operation */
-	if ((cortex_m3->dcb_dhcsr&C_MASKINTS) != (dcb_dhcsr&C_MASKINTS))
+	if ((cortex_m3->dcb_dhcsr & C_MASKINTS) != (dcb_dhcsr & C_MASKINTS))
 		ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, dcb_dhcsr | C_HALT );
 	/* Restart core */
 	ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, dcb_dhcsr );
@@ -612,11 +603,10 @@ int cortex_m3_resume(struct target_s *target, int current, u32 address, int hand
 		DEBUG("target debug resumed at 0x%x",resume_pc);
 	}
 	
-	
 	return ERROR_OK;
 }
 
-int irqstepcount=0;
+//int irqstepcount=0;
 int cortex_m3_step(struct target_s *target, int current, u32 address, int handle_breakpoints)
 {
 	/* get pointers to arch-specific information */
@@ -653,7 +643,7 @@ int cortex_m3_step(struct target_s *target, int current, u32 address, int handle
 	
 	target_call_event_callbacks(target, TARGET_EVENT_RESUMED);
     
-	if (cortex_m3->dcb_dhcsr&C_MASKINTS)
+	if (cortex_m3->dcb_dhcsr & C_MASKINTS)
 		ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, DBGKEY | C_HALT | C_DEBUGEN );
 	ahbap_write_system_atomic_u32(swjdp, DCB_DHCSR, DBGKEY| C_STEP | C_DEBUGEN);
 	ahbap_read_system_atomic_u32(swjdp, DCB_DHCSR, &cortex_m3->dcb_dhcsr);
@@ -665,14 +655,13 @@ int cortex_m3_step(struct target_s *target, int current, u32 address, int handle
 	if (breakpoint)
 		cortex_m3_set_breakpoint(target, breakpoint);
 
-	DEBUG("target stepped dcb_dhcsr=0x%x nvic_icsr=0x%x",cortex_m3->dcb_dhcsr,cortex_m3->nvic_icsr);
+	DEBUG("target stepped dcb_dhcsr = 0x%x nvic_icsr = 0x%x", cortex_m3->dcb_dhcsr, cortex_m3->nvic_icsr);
 
 	cortex_m3_debug_entry(target);
 	target_call_event_callbacks(target, TARGET_EVENT_HALTED);
 
-	DEBUG("target stepped dcb_dhcsr=0x%x nvic_icsr=0x%x",cortex_m3->dcb_dhcsr,cortex_m3->nvic_icsr);
+	DEBUG("target stepped dcb_dhcsr = 0x%x nvic_icsr = 0x%x", cortex_m3->dcb_dhcsr, cortex_m3->nvic_icsr);
 	return ERROR_OK;
-
 }
 
 int cortex_m3_assert_reset(target_t *target)
@@ -738,7 +727,6 @@ int cortex_m3_assert_reset(target_t *target)
 	armv7m_invalidate_core_regs(target);
 
 	return ERROR_OK;
-
 }
 
 int cortex_m3_deassert_reset(target_t *target)
@@ -749,13 +737,13 @@ int cortex_m3_deassert_reset(target_t *target)
 	jtag_add_reset(0, 0);
 		
 	return ERROR_OK;
-
 }
 
 void cortex_m3_unset_all_breakpoints_and_watchpoints(struct target_s *target)
 {
 
 }
+
 void cortex_m3_enable_breakpoints(struct target_s *target)
 {
 	breakpoint_t *breakpoint = target->breakpoints;
@@ -788,37 +776,36 @@ int cortex_m3_set_breakpoint(struct target_s *target, breakpoint_t *breakpoint)
 
 	if (cortex_m3->auto_bp_type)
 	{
-		breakpoint->type = (breakpoint->address<0x20000000)?BKPT_HARD:BKPT_SOFT;
+		breakpoint->type = (breakpoint->address < 0x20000000) ? BKPT_HARD : BKPT_SOFT;
 	}
 
 	if (breakpoint->type == BKPT_HARD)
 	{
-		while(comparator_list[fp_num].used && (fp_num<cortex_m3->fp_num_code))
+		while(comparator_list[fp_num].used && (fp_num < cortex_m3->fp_num_code))
 			fp_num++;
-		if (fp_num>=cortex_m3->fp_num_code)
+		if (fp_num >= cortex_m3->fp_num_code)
 		{
 			DEBUG("ERROR Can not find free FP Comparator");
 			WARNING("ERROR Can not find free FP Comparator");
 			exit(-1);
 		}
-		breakpoint->set = fp_num+1;
-		hilo = (breakpoint->address & 0x2)? FPCR_REPLACE_BKPT_HIGH:FPCR_REPLACE_BKPT_LOW;
+		breakpoint->set = fp_num + 1;
+		hilo = (breakpoint->address & 0x2) ? FPCR_REPLACE_BKPT_HIGH : FPCR_REPLACE_BKPT_LOW;
 		comparator_list[fp_num].used = 1;
-		comparator_list[fp_num].fpcr_value = breakpoint->address&0x1FFFFFFC | hilo | 1;
+		comparator_list[fp_num].fpcr_value = (breakpoint->address & 0x1FFFFFFC) | hilo | 1;
 		target_write_u32(target, comparator_list[fp_num].fpcr_address, comparator_list[fp_num].fpcr_value);
-		DEBUG("fpc_num %i    fpcr_value 0x%x", fp_num, comparator_list[fp_num].fpcr_value);
+		DEBUG("fpc_num %i fpcr_value 0x%x", fp_num, comparator_list[fp_num].fpcr_value);
 	}
 	else if (breakpoint->type == BKPT_SOFT)
 	{
 		u8 code[4];
 		buf_set_u32(code, 0, 32, ARMV7M_T_BKPT(0x11));
-		target->type->read_memory(target, breakpoint->address&0xFFFFFFFE, breakpoint->length, 1, breakpoint->orig_instr);
-		target->type->write_memory(target, breakpoint->address&0xFFFFFFFE, breakpoint->length, 1, code);
+		target->type->read_memory(target, breakpoint->address & 0xFFFFFFFE, breakpoint->length, 1, breakpoint->orig_instr);
+		target->type->write_memory(target, breakpoint->address & 0xFFFFFFFE, breakpoint->length, 1, code);
 		breakpoint->set = 0x11; /* Any nice value but 0 */
 	}
 
 	return ERROR_OK;
-
 }
 
 int cortex_m3_unset_breakpoint(struct target_s *target, breakpoint_t *breakpoint)
@@ -836,8 +823,8 @@ int cortex_m3_unset_breakpoint(struct target_s *target, breakpoint_t *breakpoint
 	
 	if (breakpoint->type == BKPT_HARD)
 	{
-		int fp_num = breakpoint->set-1;
-		if ((fp_num<0)||(fp_num>=cortex_m3->fp_num_code))
+		int fp_num = breakpoint->set - 1;
+		if ((fp_num < 0) || (fp_num >= cortex_m3->fp_num_code))
 		{
 			DEBUG("Invalid FP Comparator number in breakpoint");
 			return ERROR_OK;
@@ -851,11 +838,11 @@ int cortex_m3_unset_breakpoint(struct target_s *target, breakpoint_t *breakpoint
 		/* restore original instruction (kept in target endianness) */
 		if (breakpoint->length == 4)
 		{
-			target->type->write_memory(target, breakpoint->address&0xFFFFFFFE, 4, 1, breakpoint->orig_instr);
+			target->type->write_memory(target, breakpoint->address & 0xFFFFFFFE, 4, 1, breakpoint->orig_instr);
 		}
 		else
 		{
-			target->type->write_memory(target, breakpoint->address&0xFFFFFFFE, 2, 1, breakpoint->orig_instr);
+			target->type->write_memory(target, breakpoint->address & 0xFFFFFFFE, 2, 1, breakpoint->orig_instr);
 		}
 	}
 	breakpoint->set = 0;
@@ -863,7 +850,6 @@ int cortex_m3_unset_breakpoint(struct target_s *target, breakpoint_t *breakpoint
 	return ERROR_OK;
 }
 
-
 int cortex_m3_add_breakpoint(struct target_s *target, breakpoint_t *breakpoint)
 {
 	/* get pointers to arch-specific information */
@@ -872,16 +858,16 @@ int cortex_m3_add_breakpoint(struct target_s *target, breakpoint_t *breakpoint)
 	
 	if (cortex_m3->auto_bp_type)
 	{
-		breakpoint->type = (breakpoint->address<0x20000000)?BKPT_HARD:BKPT_SOFT;
+		breakpoint->type = (breakpoint->address < 0x20000000) ? BKPT_HARD : BKPT_SOFT;
 	}
 
-	if ((breakpoint->type == BKPT_HARD) && (breakpoint->address>=0x20000000))
+	if ((breakpoint->type == BKPT_HARD) && (breakpoint->address >= 0x20000000))
 	{
 		INFO("flash patch comparator requested outside code memory region");
 		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
 	}
 
-	if ((breakpoint->type == BKPT_SOFT) && (breakpoint->address<0x20000000))
+	if ((breakpoint->type == BKPT_SOFT) && (breakpoint->address < 0x20000000))
 	{
 		INFO("soft breakpoint requested in code (flash) memory region");
 		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
@@ -920,7 +906,7 @@ int cortex_m3_remove_breakpoint(struct target_s *target, breakpoint_t *breakpoin
 	
 	if (cortex_m3->auto_bp_type)
 	{
-		breakpoint->type = (breakpoint->address<0x20000000)?BKPT_HARD:BKPT_SOFT;
+		breakpoint->type = (breakpoint->address < 0x20000000) ? BKPT_HARD : BKPT_SOFT;
 	}
 
 	if (breakpoint->set)
@@ -934,19 +920,6 @@ int cortex_m3_remove_breakpoint(struct target_s *target, breakpoint_t *breakpoin
 	return ERROR_OK;
 }
 
-void cortex_m3_enable_watchpoints(struct target_s *target)
-{
-	watchpoint_t *watchpoint = target->watchpoints;
-	
-	/* set any pending watchpoints */
-	while (watchpoint)
-	{
-		if (watchpoint->set == 0)
-			cortex_m3_set_watchpoint(target, watchpoint);
-		watchpoint = watchpoint->next;
-	}
-}
-
 int cortex_m3_set_watchpoint(struct target_s *target, watchpoint_t *watchpoint)
 {
 	int dwt_num=0;
@@ -965,30 +938,30 @@ int cortex_m3_set_watchpoint(struct target_s *target, watchpoint_t *watchpoint)
 
 	if (watchpoint->mask == 0xffffffffu)
 	{
-		while(comparator_list[dwt_num].used && (dwt_num<cortex_m3->dwt_num_comp))
+		while(comparator_list[dwt_num].used && (dwt_num < cortex_m3->dwt_num_comp))
 			dwt_num++;
-		if (dwt_num>=cortex_m3->dwt_num_comp)
+		if (dwt_num >= cortex_m3->dwt_num_comp)
 		{
 			DEBUG("ERROR Can not find free DWT Comparator");
 			WARNING("ERROR Can not find free DWT Comparator");
 			return -1;
 		}
-		watchpoint->set = dwt_num+1;
+		watchpoint->set = dwt_num + 1;
 		mask = 0;
 		temp = watchpoint->length;
-		while (temp>1)
+		while (temp > 1)
 		{
-			temp = temp/2;
+			temp = temp / 2;
 			mask++;
 		}
 		comparator_list[dwt_num].used = 1;
 		comparator_list[dwt_num].comp = watchpoint->address;
 		comparator_list[dwt_num].mask = mask;
-		comparator_list[dwt_num].function = watchpoint->rw+5;
+		comparator_list[dwt_num].function = watchpoint->rw + 5;
 		target_write_u32(target, comparator_list[dwt_num].dwt_comparator_address, comparator_list[dwt_num].comp);
 		target_write_u32(target, comparator_list[dwt_num].dwt_comparator_address|0x4, comparator_list[dwt_num].mask);
 		target_write_u32(target, comparator_list[dwt_num].dwt_comparator_address|0x8, comparator_list[dwt_num].function);
-		DEBUG("dwt_num %i    0x%x 0x%x 0x%x", dwt_num, comparator_list[dwt_num].comp, comparator_list[dwt_num].mask, comparator_list[dwt_num].function);
+		DEBUG("dwt_num %i 0x%x 0x%x 0x%x", dwt_num, comparator_list[dwt_num].comp, comparator_list[dwt_num].mask, comparator_list[dwt_num].function);
 	}
 	else
 	{
@@ -1014,9 +987,9 @@ int cortex_m3_unset_watchpoint(struct target_s *target, watchpoint_t *watchpoint
 		return ERROR_OK;
 	}
 
-	dwt_num = watchpoint->set-1;
+	dwt_num = watchpoint->set - 1;
 
-	if ((dwt_num<0)||(dwt_num>=cortex_m3->dwt_num_comp))
+	if ((dwt_num < 0) || (dwt_num >= cortex_m3->dwt_num_comp))
 	{
 		DEBUG("Invalid DWT Comparator number in watchpoint");
 		return ERROR_OK;
@@ -1030,8 +1003,6 @@ int cortex_m3_unset_watchpoint(struct target_s *target, watchpoint_t *watchpoint
 	return ERROR_OK;
 }
 
-
-
 int cortex_m3_add_watchpoint(struct target_s *target, watchpoint_t *watchpoint)
 {
 	/* get pointers to arch-specific information */
@@ -1081,6 +1052,18 @@ int cortex_m3_remove_watchpoint(struct target_s *target, watchpoint_t *watchpoin
 	return ERROR_OK;
 }
 
+void cortex_m3_enable_watchpoints(struct target_s *target)
+{
+	watchpoint_t *watchpoint = target->watchpoints;
+	
+	/* set any pending watchpoints */
+	while (watchpoint)
+	{
+		if (watchpoint->set == 0)
+			cortex_m3_set_watchpoint(target, watchpoint);
+		watchpoint = watchpoint->next;
+	}
+}
 
 int cortex_m3_load_core_reg_u32(struct target_s *target, enum armv7m_regtype type, u32 num, u32 * value)
 {
@@ -1116,16 +1099,15 @@ int cortex_m3_load_core_reg_u32(struct target_s *target, enum armv7m_regtype typ
 		ahbap_write_coreregister_u32(swjdp, 0x20000000, 15);
 		cortex_m3_single_step_core(target);
 		ahbap_read_coreregister_u32(swjdp, value, 0);
-		armv7m->core_cache->reg_list[0].dirty=1;
-		armv7m->core_cache->reg_list[15].dirty=1;
+		armv7m->core_cache->reg_list[0].dirty = 1;
+		armv7m->core_cache->reg_list[15].dirty = 1;
 		ahbap_write_system_u32(swjdp, 0x20000000, savedram);
 		swjdp_transaction_endcheck(swjdp);
-		DEBUG("load from special reg %i  value 0x%x",SYSm, *value);
+		DEBUG("load from special reg %i value 0x%x", SYSm, *value);
 	}
 	else return ERROR_INVALID_ARGUMENTS;
 	
 	return ERROR_OK;
-	
 }
 
 int cortex_m3_store_core_reg_u32(struct target_s *target, enum armv7m_regtype type, u32 num, u32 value)
@@ -1142,11 +1124,11 @@ int cortex_m3_store_core_reg_u32(struct target_s *target, enum armv7m_regtype ty
 		retval = ahbap_write_coreregister_u32(swjdp, value, num);
 		if (retval != ERROR_OK)
 		{
-			ERROR("JTAG failure %i",retval);
-			armv7m->core_cache->reg_list[num].dirty=1;
+			ERROR("JTAG failure %i", retval);
+			armv7m->core_cache->reg_list[num].dirty = 1;
 			return ERROR_JTAG_DEVICE_ERROR;
 		}
-		DEBUG("write core reg %i value 0x%x",num, value);
+		DEBUG("write core reg %i value 0x%x", num, value);
 	}
 	else if (type == ARMV7M_REGISTER_CORE_SP) /* Special purpose core register */
 	{
@@ -1163,21 +1145,18 @@ int cortex_m3_store_core_reg_u32(struct target_s *target, enum armv7m_regtype ty
 		ahbap_write_coreregister_u32(swjdp, 0x20000000, 15);
 		cortex_m3_single_step_core(target);
 		ahbap_write_coreregister_u32(swjdp, tempr0, 0);
-		armv7m->core_cache->reg_list[15].dirty=1;
+		armv7m->core_cache->reg_list[15].dirty = 1;
 		ahbap_write_system_u32(swjdp, 0x20000000, savedram);
 		swjdp_transaction_endcheck(swjdp);
-		DEBUG("write special reg %i  value 0x%x ",SYSm, value);
+		DEBUG("write special reg %i value 0x%x ", SYSm, value);
 	}
 	else return ERROR_INVALID_ARGUMENTS;
 	
-	return ERROR_OK;
-	
+	return ERROR_OK;	
 }
 
-
 int cortex_m3_read_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer)
 {
-
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
@@ -1196,12 +1175,12 @@ int cortex_m3_read_memory(struct target_s *target, u32 address, u32 size, u32 co
 		case 4:
 			/* TODOLATER Check error return value ! */
 			{
-				ahbap_read_buf(swjdp, buffer, 4*count, address);
+				ahbap_read_buf(swjdp, buffer, 4 * count, address);
 			}
 			break;
 		case 2:
 			{
-				ahbap_read_buf(swjdp, buffer, 2*count, address);
+				ahbap_read_buf_u16(swjdp, buffer, 2 * count, address);
 			}	
 			break;
 		case 1:
@@ -1213,14 +1192,12 @@ int cortex_m3_read_memory(struct target_s *target, u32 address, u32 size, u32 co
 			ERROR("BUG: we shouldn't get here");
 			exit(-1);
 	}
-		
-
+	
 	return ERROR_OK;
 }
 
 int cortex_m3_write_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer)
 {
-
 	/* get pointers to arch-specific information */
 	armv7m_common_t *armv7m = target->arch_info;
 	cortex_m3_common_t *cortex_m3 = armv7m->arch_info;
@@ -1238,12 +1215,12 @@ int cortex_m3_write_memory(struct target_s *target, u32 address, u32 size, u32 c
 		case 4:
 			/* TODOLATER Check error return value ! */
 			{
-				ahbap_write_buf(swjdp, buffer, 4*count, address);
+				ahbap_write_buf(swjdp, buffer, 4 * count, address);
 			}
 			break;
 		case 2:
 			{
-				ahbap_write_buf(swjdp, buffer, 2*count, address);
+				ahbap_write_buf_u16(swjdp, buffer, 2 * count, address);
 			}	
 			break;
 		case 1:
@@ -1261,14 +1238,11 @@ int cortex_m3_write_memory(struct target_s *target, u32 address, u32 size, u32 c
 
 int cortex_m3_bulk_write_memory(target_t *target, u32 address, u32 count, u8 *buffer)
 {
-
-	cortex_m3_write_memory(target, address, 4,count,buffer);
+	cortex_m3_write_memory(target, address, 4, count, buffer);
 	
 	return ERROR_OK;
 }
 
-
-
 void cortex_m3_build_reg_cache(target_t *target)
 {
 	armv7m_build_reg_cache(target);
@@ -1287,48 +1261,43 @@ int cortex_m3_init_target(struct command_context_s *cmd_ctx, struct target_s *ta
 	cortex_m3_build_reg_cache(target);
 	ahbap_debugport_init(swjdp);
 
-	/* Read from Device Identification Registers, IS THIS CORTEX OR Luminary Micro SPECIFIC ?? */
-	target_read_u32(target, CPUID, &cpuid );
-	if (cpuid == 0x410fc231)
-		DEBUG("CORTEX-M3 processor");
-	DEBUG("cpuid %x",cpuid);
-	/* Probably only valid for LMI parts, move to flash/stellaris ? */
-	target_read_u32(target, SYSTEM_CONTROL_BASE|0x04, &did1);
-	target_read_u32(target,SYSTEM_CONTROL_BASE|0x08,&dc0);
-	DEBUG("did1 %x",did1);
-	DEBUG("dc0 %x",dc0);
+	/* Read from Device Identification Registers */
+	target_read_u32(target, CPUID, &cpuid);
+	if (((cpuid >> 4) & 0xc3f) == 0xc23)
+		DEBUG("CORTEX-M3 processor detected");
+	DEBUG("cpuid %x", cpuid);
 	
-	target_read_u32(target,NVIC_ICTR,&ictr);
-	cortex_m3->intlinesnum = (ictr&0x1F) + 1;
-	cortex_m3->intsetenable = calloc(cortex_m3->intlinesnum,4);
-	for (i=0;i<cortex_m3->intlinesnum;i++)
+	target_read_u32(target, NVIC_ICTR, &ictr);
+	cortex_m3->intlinesnum = (ictr & 0x1F) + 1;
+	cortex_m3->intsetenable = calloc(cortex_m3->intlinesnum, 4);
+	for (i = 0; i < cortex_m3->intlinesnum; i++)
 	{
-		target_read_u32(target,NVIC_ISE0+4*i,cortex_m3->intsetenable+i);
-		DEBUG(" interrupt enable[%i]=0x%x",i,cortex_m3->intsetenable[i]);
+		target_read_u32(target, NVIC_ISE0 + 4 * i, cortex_m3->intsetenable + i);
+		DEBUG("interrupt enable[%i] = 0x%x", i, cortex_m3->intsetenable[i]);
 	}
 	
 	/* Setup FPB */
 	target_read_u32(target, FP_CTRL, &fpcr);
 	cortex_m3->auto_bp_type = 1;
-	cortex_m3->fp_num_code = (fpcr>>4)&0xF;
-	cortex_m3->fp_num_lit = (fpcr>>8)&0xF;
+	cortex_m3->fp_num_code = (fpcr >> 4) & 0xF;
+	cortex_m3->fp_num_lit = (fpcr >> 8) & 0xF;
 	cortex_m3->fp_code_available = cortex_m3->fp_num_code;
 	cortex_m3->fp_comparator_list=calloc(cortex_m3->fp_num_code+cortex_m3->fp_num_lit, sizeof(cortex_m3_fp_comparator_t));
-	for (i=0;i<cortex_m3->fp_num_code+cortex_m3->fp_num_lit;i++)
+	for (i = 0; i < cortex_m3->fp_num_code + cortex_m3->fp_num_lit; i++)
 	{
-		cortex_m3->fp_comparator_list[i].type = (i<cortex_m3->fp_num_code)?FPCR_CODE:FPCR_LITERAL;
-		cortex_m3->fp_comparator_list[i].fpcr_address = FP_COMP0+4*i;
+		cortex_m3->fp_comparator_list[i].type = (i < cortex_m3->fp_num_code) ? FPCR_CODE : FPCR_LITERAL;
+		cortex_m3->fp_comparator_list[i].fpcr_address = FP_COMP0 + 4 * i;
 	}
-	DEBUG("FPB fpcr 0x%x, numcode %i, numlit %i",fpcr,cortex_m3->fp_num_code,cortex_m3->fp_num_lit);
+	DEBUG("FPB fpcr 0x%x, numcode %i, numlit %i", fpcr, cortex_m3->fp_num_code, cortex_m3->fp_num_lit);
 		
 	/* Setup DWT */
 	target_read_u32(target, DWT_CTRL, &dwtcr);
-	cortex_m3->dwt_num_comp = (dwtcr>>28)&0xF;
+	cortex_m3->dwt_num_comp = (dwtcr >> 28) & 0xF;
 	cortex_m3->dwt_comp_available = cortex_m3->dwt_num_comp;
 	cortex_m3->dwt_comparator_list=calloc(cortex_m3->dwt_num_comp, sizeof(cortex_m3_dwt_comparator_t));
-	for (i=0; i<cortex_m3->dwt_num_comp; i++)
+	for (i = 0; i < cortex_m3->dwt_num_comp; i++)
 	{
-		cortex_m3->dwt_comparator_list[i].dwt_comparator_address = DWT_COMP0+0x10*i;
+		cortex_m3->dwt_comparator_list[i].dwt_comparator_address = DWT_COMP0 + 0x10 * i;
 	}
 	
 	return ERROR_OK;
@@ -1345,7 +1314,7 @@ int cortex_m3_init_arch_info(target_t *target, cortex_m3_common_t *cortex_m3, in
 	armv7m_common_t *armv7m;
 	armv7m = &cortex_m3->armv7m;
 
-	arm_jtag_t * jtag_info = &cortex_m3->jtag_info;	
+	arm_jtag_t *jtag_info = &cortex_m3->jtag_info;	
 
 	/* prepare JTAG information for the new target */
 	cortex_m3->jtag_info.chain_pos = chain_pos;
diff --git a/src/target/cortex_m3.h b/src/target/cortex_m3.h
index 63a5c43b..7061d668 100644
--- a/src/target/cortex_m3.h
+++ b/src/target/cortex_m3.h
@@ -40,7 +40,6 @@ extern char* cortex_m3_state_strings[];
 #define DCB_DCRDR	0xE000EDF8
 #define DCB_DEMCR	0xE000EDFC
 
-
 #define DCRSR_WnR	(1<<16)	
 
 #define DWT_CTRL	0xE0001000
@@ -107,7 +106,6 @@ extern char* cortex_m3_state_strings[];
 #define DFSR_DWTTRAP	4
 #define DFSR_VCATCH	8
 
-
 #define FPCR_CODE 0
 #define FPCR_LITERAL 1
 #define FPCR_REPLACE_REMAP  (0<<30)
@@ -149,16 +147,16 @@ typedef struct cortex_m3_common_s
 	int fp_num_code;
 	int fp_code_available;
 	int auto_bp_type;
-	cortex_m3_fp_comparator_t * fp_comparator_list;
+	cortex_m3_fp_comparator_t *fp_comparator_list;
 	
 	/* DWT */
 	int dwt_num_comp;
 	int dwt_comp_available;
-	cortex_m3_dwt_comparator_t * dwt_comparator_list;
+	cortex_m3_dwt_comparator_t *dwt_comparator_list;
 	
 	/* Interrupts */
 	int intlinesnum;
-	u32 * intsetenable;
+	u32 *intsetenable;
 	
 /*
 	u32 arm_bkpt;
@@ -189,7 +187,6 @@ typedef struct cortex_m3_common_s
 	void *arch_info;
 } cortex_m3_common_t;
 
-
 extern void cortex_m3_build_reg_cache(target_t *target);
 
 enum target_state cortex_m3_poll(target_t *target);
diff --git a/src/target/cortex_swjdp.c b/src/target/cortex_swjdp.c
index e50c6843..0815f6ed 100644
--- a/src/target/cortex_swjdp.c
+++ b/src/target/cortex_swjdp.c
@@ -25,6 +25,9 @@
  * Cortex-M3™ TRM, ARM DDI 0337C                                            *
  *                                                                         *
 ***************************************************************************/
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 
 #include "replacements.h"
 
@@ -55,17 +58,17 @@ are immediatley available.
 ***************************************************************************/
 
 /* Scan out and in from target ordered u8 buffers */
-int swjdp_scan(arm_jtag_t *jtag_info, u8 chain, u8 reg_addr, u8 RnW, u8 *outvalue, u8 *invalue, u8 *ack)
+int swjdp_scan(arm_jtag_t *jtag_info, u8 instr, u8 reg_addr, u8 RnW, u8 *outvalue, u8 *invalue, u8 *ack)
 {
 	scan_field_t fields[2];
 	u8 out_addr_buf;
 	
 	jtag_add_end_state(TAP_RTI);
-	arm_jtag_set_instr(jtag_info, chain, NULL);
+	arm_jtag_set_instr(jtag_info, instr, NULL);
 
 	fields[0].device = jtag_info->chain_pos;
 	fields[0].num_bits = 3;
-	buf_set_u32(&out_addr_buf, 0, 3, ((reg_addr>>1)&0x6) | (RnW&0x1));
+	buf_set_u32(&out_addr_buf, 0, 3, ((reg_addr >> 1) & 0x6) | (RnW & 0x1));
 	fields[0].out_value = &out_addr_buf;
 	fields[0].out_mask = NULL;
 	fields[0].in_value = ack;
@@ -87,22 +90,21 @@ int swjdp_scan(arm_jtag_t *jtag_info, u8 chain, u8 reg_addr, u8 RnW, u8 *outvalu
 	jtag_add_dr_scan(2, fields, -1, NULL);
 
 	return ERROR_OK;
-
 }
 
 /* Scan out and in from host ordered u32 variables */
-int swjdp_scan_u32(arm_jtag_t *jtag_info, u8 chain, u8 reg_addr, u8 RnW, u32 outvalue, u32 *invalue, u8 *ack)
+int swjdp_scan_u32(arm_jtag_t *jtag_info, u8 instr, u8 reg_addr, u8 RnW, u32 outvalue, u32 *invalue, u8 *ack)
 {
 	scan_field_t fields[2];
 	u8 out_value_buf[4];
 	u8 out_addr_buf;
 	
 	jtag_add_end_state(TAP_RTI);
-	arm_jtag_set_instr(jtag_info, chain, NULL);
+	arm_jtag_set_instr(jtag_info, instr, NULL);
 
 	fields[0].device = jtag_info->chain_pos;
 	fields[0].num_bits = 3;
-	buf_set_u32(&out_addr_buf, 0, 3, ((reg_addr>>1)&0x6) | (RnW&0x1));
+	buf_set_u32(&out_addr_buf, 0, 3, ((reg_addr >> 1) & 0x6) | (RnW & 0x1));
 	fields[0].out_value = &out_addr_buf;
 	fields[0].out_mask = NULL;
 	fields[0].in_value = ack;
@@ -133,21 +135,19 @@ int swjdp_scan_u32(arm_jtag_t *jtag_info, u8 chain, u8 reg_addr, u8 RnW, u32 out
 	jtag_add_dr_scan(2, fields, -1, NULL);
 
 	return ERROR_OK;
-
 }
 
 /* scan_inout_check adds one extra inscan for DPAP_READ commands to read variables */ 
-int scan_inout_check(swjdp_common_t *swjdp, u8 chain, u8 reg_addr, u8 RnW, u8 *outvalue, u8 *invalue)
+int scan_inout_check(swjdp_common_t *swjdp, u8 instr, u8 reg_addr, u8 RnW, u8 *outvalue, u8 *invalue)
 {
-
-	swjdp_scan(swjdp->jtag_info, chain, reg_addr, RnW, outvalue, NULL, NULL);
-	if ((RnW==DPAP_READ) && (invalue != NULL))
+	swjdp_scan(swjdp->jtag_info, instr, reg_addr, RnW, outvalue, NULL, NULL);
+	if ((RnW == DPAP_READ) && (invalue != NULL))
 	{
 		swjdp_scan(swjdp->jtag_info, SWJDP_IR_DPACC, 0xC, DPAP_READ, 0, invalue, &swjdp->ack);
 	}
 	
 	/* In TRANS_MODE_ATOMIC all SWJDP_IR_APACC transactions wait for ack=OK/FAULT and the check CTRL_STAT */
-	if ((chain == SWJDP_IR_APACC)&&(swjdp->trans_mode == TRANS_MODE_ATOMIC))
+	if ((instr == SWJDP_IR_APACC) && (swjdp->trans_mode == TRANS_MODE_ATOMIC))
 	{
 		return swjdp_transaction_endcheck(swjdp);
 	}
@@ -155,17 +155,17 @@ int scan_inout_check(swjdp_common_t *swjdp, u8 chain, u8 reg_addr, u8 RnW, u8 *o
 	return ERROR_OK;
 }
 
-int scan_inout_check_u32(swjdp_common_t *swjdp, u8 chain, u8 reg_addr, u8 RnW, u32 outvalue, u32 *invalue)
+int scan_inout_check_u32(swjdp_common_t *swjdp, u8 instr, u8 reg_addr, u8 RnW, u32 outvalue, u32 *invalue)
 {
 
-	swjdp_scan_u32(swjdp->jtag_info, chain, reg_addr, RnW, outvalue, NULL, NULL);
+	swjdp_scan_u32(swjdp->jtag_info, instr, reg_addr, RnW, outvalue, NULL, NULL);
 	if ((RnW==DPAP_READ) && (invalue != NULL))
 	{
 		swjdp_scan_u32(swjdp->jtag_info, SWJDP_IR_DPACC, 0xC, DPAP_READ, 0, invalue, &swjdp->ack);
 	}
 	
 	/* In TRANS_MODE_ATOMIC all SWJDP_IR_APACC transactions wait for ack=OK/FAULT and the check CTRL_STAT */
-	if ((chain == SWJDP_IR_APACC)&&(swjdp->trans_mode == TRANS_MODE_ATOMIC))
+	if ((instr == SWJDP_IR_APACC) && (swjdp->trans_mode == TRANS_MODE_ATOMIC))
 	{
 		return swjdp_transaction_endcheck(swjdp);
 	}
@@ -177,17 +177,18 @@ int swjdp_transaction_endcheck(swjdp_common_t *swjdp)
 {
 	int waitcount = 0;
 	u32 ctrlstat;
-	u8 ack=0;
 
 	scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
 	jtag_execute_queue();
-	swjdp->ack = swjdp->ack&0x7;
+	
+	swjdp->ack = swjdp->ack & 0x7;
+	
 	while (swjdp->ack != 2)
 	{
-		if (swjdp->ack==1)
+		if (swjdp->ack == 1)
 		{
 			waitcount++;
-			if (waitcount>100)
+			if (waitcount > 100)
 			{
 				WARNING("Timeout waiting for ACK = OK/FAULT in SWJDP transaction");
 				return ERROR_JTAG_DEVICE_ERROR;
@@ -200,42 +201,47 @@ int swjdp_transaction_endcheck(swjdp_common_t *swjdp)
 		}
 		scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
 		jtag_execute_queue();
-		swjdp->ack = swjdp->ack&0x7;
+		swjdp->ack = swjdp->ack & 0x7;
 	}
 
 	/* Check for STICKYERR and STICKYORUN */
-	if (ctrlstat & (SSTICKYORUN|SSTICKYERR))
+	if (ctrlstat & (SSTICKYORUN | SSTICKYERR))
 	{
-		DEBUG("      swjdp: CTRL/STAT error 0x%x",ctrlstat);
+		DEBUG("swjdp: CTRL/STAT error 0x%x", ctrlstat);
 		/* Check power to debug regions */
-		if ((ctrlstat&0xf0000000)!=0xf0000000)
+		if ((ctrlstat & 0xf0000000) != 0xf0000000)
 		{
 			 ahbap_debugport_init(swjdp);
 		}
 		else
 		{
 			u32 dcb_dhcsr,nvic_shcsr, nvic_bfar, nvic_cfsr;
-			if (ctrlstat&SSTICKYORUN) ERROR("SWJ-DP OVERRUN - check clock or reduce jtag speed");
-			if (ctrlstat&SSTICKYERR) ERROR("SWJ-DP STICKY ERROR");
+			
+			if (ctrlstat & SSTICKYORUN)
+				ERROR("SWJ-DP OVERRUN - check clock or reduce jtag speed");
+			
+			if (ctrlstat & SSTICKYERR)
+				ERROR("SWJ-DP STICKY ERROR");
+			
 			/* Clear Sticky Error Bits */
-			scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, DP_CTRL_STAT, DPAP_WRITE, swjdp->dp_ctrl_stat|SSTICKYORUN|SSTICKYERR, NULL);
+			scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, DP_CTRL_STAT, DPAP_WRITE, swjdp->dp_ctrl_stat | SSTICKYORUN | SSTICKYERR, NULL);
 			scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
 			jtag_execute_queue();
 
+			DEBUG("swjdp: status 0x%x", ctrlstat);
+			
 			/* Can we find out the reason for the error ?? */			
 			ahbap_read_system_atomic_u32(swjdp, DCB_DHCSR, &dcb_dhcsr);
 			ahbap_read_system_atomic_u32(swjdp, NVIC_SHCSR, &nvic_shcsr);
 			ahbap_read_system_atomic_u32(swjdp, NVIC_CFSR, &nvic_cfsr);
 			ahbap_read_system_atomic_u32(swjdp, NVIC_BFAR, &nvic_bfar);
-			//DEBUG("dcb_dhcsr %x,  nvic_shcsr %x,  nvic_cfsr %x,  nvic_bfar %x",dcb_dhcsr,nvic_shcsr,nvic_cfsr,nvic_bfar);
-			ERROR("dcb_dhcsr %x,  nvic_shcsr %x,  nvic_cfsr %x,  nvic_bfar %x",dcb_dhcsr,nvic_shcsr,nvic_cfsr,nvic_bfar);
+			ERROR("dcb_dhcsr 0x%x, nvic_shcsr 0x%x, nvic_cfsr 0x%x, nvic_bfar 0x%x", dcb_dhcsr, nvic_shcsr, nvic_cfsr, nvic_bfar);
 		}
 		jtag_execute_queue();
 		return ERROR_JTAG_DEVICE_ERROR;
 	}
 
 	return ERROR_OK;
-
 }
 
 /***************************************************************************
@@ -250,12 +256,10 @@ int swjdp_write_dpacc(swjdp_common_t *swjdp, u32 value, u8 reg_addr)
 	
 	buf_set_u32(out_value_buf, 0, 32, value);
 	return scan_inout_check(swjdp, SWJDP_IR_DPACC, reg_addr, DPAP_WRITE, out_value_buf, NULL);
-
 }
 
 int swjdp_read_dpacc(swjdp_common_t *swjdp, u32 *value, u8 reg_addr)
 {
-
 	scan_inout_check_u32(swjdp, SWJDP_IR_DPACC, reg_addr, DPAP_READ, 0, value);
 
     return ERROR_OK;
@@ -264,9 +268,9 @@ int swjdp_read_dpacc(swjdp_common_t *swjdp, u32 *value, u8 reg_addr)
 int swjdp_bankselect_apacc(swjdp_common_t *swjdp,u32 reg_addr)
 {
 	u32 select;
-	select = (reg_addr&0xFF0000F0);
+	select = (reg_addr & 0xFF0000F0);
 
-	if ( select != swjdp->dp_select_value )
+	if (select != swjdp->dp_select_value)
 	{
 		swjdp_write_dpacc(swjdp, select, DP_SELECT);
 		swjdp->dp_select_value = select;
@@ -317,17 +321,16 @@ int ahbap_read_reg_u32(swjdp_common_t *swjdp, u32 reg_addr, u32 *value)
 
 int ahbap_setup_accessport(swjdp_common_t *swjdp, u32 csw, u32 tar)
 {
-
 	csw = csw | CSW_DBGSWENABLE | CSW_MASTER_DEBUG | CSW_HPROT;
-	if ( csw != swjdp->ap_csw_value )
+	if (csw != swjdp->ap_csw_value)
 	{
-	//DEBUG("swjdp : Set CSW %x",csw);
+		//DEBUG("swjdp : Set CSW %x",csw);
 		ahbap_write_reg_u32(swjdp, AHBAP_CSW, csw ); 
 		swjdp->ap_csw_value = csw;
 	}
-	if ( tar != swjdp->ap_tar_value )
+	if (tar != swjdp->ap_tar_value)
 	{
-	//DEBUG("swjdp : Set TAR %x",tar);
+		//DEBUG("swjdp : Set TAR %x",tar);
 		ahbap_write_reg_u32(swjdp, AHBAP_TAR, tar );
 		swjdp->ap_tar_value = tar;
 	}
@@ -349,11 +352,10 @@ int ahbap_setup_accessport(swjdp_common_t *swjdp, u32 csw, u32 tar)
 *****************************************************************************/
 int ahbap_read_system_u32(swjdp_common_t *swjdp, u32 address, u32 *value)
 {
-
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, address&0xFFFFFFF0);
-	ahbap_read_reg_u32(swjdp, AHBAP_BD0|address&0xC, value );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, address & 0xFFFFFFF0);
+	ahbap_read_reg_u32(swjdp, AHBAP_BD0 | (address & 0xC), value );
 	
 	return ERROR_OK;
 }
@@ -374,18 +376,16 @@ int ahbap_read_system_atomic_u32(swjdp_common_t *swjdp, u32 address, u32 *value)
 *****************************************************************************/
 int ahbap_write_system_u32(swjdp_common_t *swjdp, u32 address, u32 value)
 {
-
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, address&0xFFFFFFF0);
-	ahbap_write_reg_u32(swjdp, AHBAP_BD0|address&0xC, value );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, address & 0xFFFFFFF0);
+	ahbap_write_reg_u32(swjdp, AHBAP_BD0 | (address & 0xC), value );
 
 	return ERROR_OK;
 }
 
 int ahbap_write_system_atomic_u32(swjdp_common_t *swjdp, u32 address, u32 value)
 {
-
 	ahbap_write_system_u32(swjdp, address, value);
 	
 	return swjdp_transaction_endcheck(swjdp);
@@ -401,53 +401,53 @@ int ahbap_write_system_atomic_u32(swjdp_common_t *swjdp, u32 address, u32 value)
 int ahbap_write_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
 {
 	u32 outvalue;
-	int wcount, blocksize, writecount, errorcount=0, retval=ERROR_OK;
+	int wcount, blocksize, writecount, errorcount = 0, retval = ERROR_OK;
 
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 
-	while ( (address&0x3)&&(count>0) )
+	while ((address & 0x3) && (count > 0))
 	{
 		ahbap_setup_accessport(swjdp, CSW_8BIT | CSW_ADDRINC_SINGLE, address);
-		outvalue = (*buffer++)<<8*(address&0x3) ;
+		outvalue = (*buffer++) << 8 * (address & 0x3);
 		ahbap_write_reg_u32(swjdp, AHBAP_DRW, outvalue );
 		swjdp_transaction_endcheck(swjdp);
 		count--;
 		address++;
 	}
-	wcount = count>>2;
-	count = count-4*wcount;
-	while (wcount>0)
+	wcount = count >> 2;
+	count = count - 4 * wcount;
+	while (wcount > 0)
 	{
 		/* Adjust to read within 4K block boundaries */
-		blocksize = (0x1000-(0xFFF&address))>>2;
-		if (wcount<blocksize)
+		blocksize = (0x1000 - (0xFFF & address)) >> 2;
+		if (wcount < blocksize)
 			blocksize = wcount;
 		ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_SINGLE, address);
 		for (writecount=0; writecount<blocksize; writecount++)
 		{
-			ahbap_write_reg(swjdp, AHBAP_DRW, buffer+4*writecount );
+			ahbap_write_reg(swjdp, AHBAP_DRW, buffer + 4 * writecount );
 		}
-		if (swjdp_transaction_endcheck(swjdp)==ERROR_OK)
+		if (swjdp_transaction_endcheck(swjdp) == ERROR_OK)
 		{
-			wcount = wcount-blocksize;
-			address = address+4*blocksize;
-			buffer = buffer + 4*blocksize;
+			wcount = wcount - blocksize;
+			address = address + 4 * blocksize;
+			buffer = buffer + 4 * blocksize;
 		}
 		else
 		{
 			errorcount++;
 		}
-		if (errorcount>1)
+		if (errorcount > 1)
 		{
 			WARNING("Block read error address %x, count %x", address, count);
 			return ERROR_JTAG_DEVICE_ERROR;
 		}
 	}
-
-	while (count>0)
+	
+	while (count > 0)
 	{
 		ahbap_setup_accessport(swjdp, CSW_8BIT | CSW_ADDRINC_SINGLE, address);
-		outvalue = (*buffer++)<<8*(address&0x3) ;
+		outvalue = (*buffer++) << 8 * (address & 0x3);
 		ahbap_write_reg_u32(swjdp, AHBAP_DRW, outvalue );
 		retval = swjdp_transaction_endcheck(swjdp);
 		count--;
@@ -457,6 +457,27 @@ int ahbap_write_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
 	return retval;
 }
 
+int ahbap_write_buf_u16(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
+{
+	u32 outvalue;
+	int retval = ERROR_OK;
+	
+	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
+	
+	while (count > 0)
+	{
+		ahbap_setup_accessport(swjdp, CSW_16BIT | CSW_ADDRINC_SINGLE, address);
+		outvalue = *((u16*)buffer) << 8 * (address & 0x3);
+		ahbap_write_reg_u32(swjdp, AHBAP_DRW, outvalue );
+		retval = swjdp_transaction_endcheck(swjdp);
+		count -= 2;
+		address += 2;
+		buffer += 2;
+	}
+
+	return retval;
+}
+
 /*****************************************************************************
 *                                                                            *
 * ahbap_read_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)  *
@@ -467,60 +488,60 @@ int ahbap_write_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
 int ahbap_read_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
 {
 	u32 invalue;
-	int wcount, blocksize, readcount, errorcount=0, retval=ERROR_OK;
+	int wcount, blocksize, readcount, errorcount = 0, retval = ERROR_OK;
 
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 
-	while ( (address&0x3)&&(count>0) )
+	while ((address & 0x3) && (count > 0))
 	{
 		ahbap_setup_accessport(swjdp, CSW_8BIT | CSW_ADDRINC_SINGLE, address);
-		ahbap_read_reg_u32(swjdp, AHBAP_DRW, &invalue );
+		ahbap_read_reg_u32(swjdp, AHBAP_DRW, &invalue);
 		swjdp_transaction_endcheck(swjdp);
-		*buffer++ = (invalue>>8*(address&0x3))&0xFF;
+		*buffer++ = (invalue >> 8 * (address & 0x3)) & 0xFF;
 		count--;
 		address++;
 	}
-	wcount = count>>2;
-	count = count-4*wcount;
-	while (wcount>0)
+	wcount = count >> 2;
+	count = count - 4 * wcount;
+	while (wcount > 0)
 	{
 		/* Adjust to read within 4K block boundaries */
-		blocksize = (0x1000-(0xFFF&address))>>2;
-		if (wcount<blocksize)
+		blocksize = (0x1000 - (0xFFF & address)) >> 2;
+		if (wcount < blocksize)
 			blocksize = wcount;
 		ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_SINGLE, address);
 		/* Scan out first read */
 		swjdp_scan(swjdp->jtag_info, SWJDP_IR_APACC, AHBAP_DRW, DPAP_READ, 0, NULL, NULL);
-		for (readcount=0; readcount<blocksize-1; readcount++)
+		for (readcount = 0; readcount < blocksize - 1; readcount++)
 		{
 			/* Scan out read instruction and scan in previous value */
-			swjdp_scan(swjdp->jtag_info, SWJDP_IR_APACC, AHBAP_DRW, DPAP_READ, 0, buffer+4*readcount, &swjdp->ack);
+			swjdp_scan(swjdp->jtag_info, SWJDP_IR_APACC, AHBAP_DRW, DPAP_READ, 0, buffer + 4 * readcount, &swjdp->ack);
 		}
 		/* Scan in last value */
-		swjdp_scan(swjdp->jtag_info, SWJDP_IR_DPACC, 0xC, DPAP_READ, 0, buffer+4*readcount, &swjdp->ack);
-		if (swjdp_transaction_endcheck(swjdp)==ERROR_OK)
+		swjdp_scan(swjdp->jtag_info, SWJDP_IR_DPACC, 0xC, DPAP_READ, 0, buffer + 4 * readcount, &swjdp->ack);
+		if (swjdp_transaction_endcheck(swjdp) == ERROR_OK)
 		{
-			wcount = wcount-blocksize;
-			address += 4*blocksize;
-			buffer += 4*blocksize; 
+			wcount = wcount - blocksize;
+			address += 4 * blocksize;
+			buffer += 4 * blocksize; 
 		}
 		else
 		{
 			errorcount++;
 		}
-		if (errorcount>1)
+		if (errorcount > 1)
 		{
 			WARNING("Block read error address %x, count %x", address, count);
 			return ERROR_JTAG_DEVICE_ERROR;
 		}
 	}
 
-	while (count>0)
+	while (count > 0)
 	{
 		ahbap_setup_accessport(swjdp, CSW_8BIT | CSW_ADDRINC_SINGLE, address);
 		ahbap_read_reg_u32(swjdp, AHBAP_DRW, &invalue );
 		retval = swjdp_transaction_endcheck(swjdp);
-		*buffer++ = (invalue>>8*(address&0x3))&0xFF;
+		*buffer++ = (invalue >> 8 * (address & 0x3)) & 0xFF;
 		count--;
 		address++;
 	}
@@ -528,35 +549,56 @@ int ahbap_read_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
 	return retval;
 }
 
+int ahbap_read_buf_u16(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address)
+{
+	u32 invalue;
+	int retval = ERROR_OK;
+	
+	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
+	
+	while (count > 0)
+	{
+		ahbap_setup_accessport(swjdp, CSW_16BIT | CSW_ADDRINC_SINGLE, address);
+		ahbap_read_reg_u32(swjdp, AHBAP_DRW, &invalue );
+		retval = swjdp_transaction_endcheck(swjdp);
+		*((u16*)buffer) = (invalue >> 8 * (address & 0x3));
+		count -= 2;
+		address += 2;
+		buffer += 2;
+	}
+
+	return retval;
+}
+
 int ahbap_block_read_u32(swjdp_common_t *swjdp, u32 *buffer, int count, u32 address)
 {
-	int readcount, errorcount=0;
+	int readcount, errorcount = 0;
 	u32 blockmax, blocksize;
 	
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 	
-	while (count>0)
+	while (count > 0)
 	{
 		/* Adjust to read within 4K block boundaries */
-		blocksize = (0x1000-(0xFFF&address))>>2;
-		if (count<blocksize)
+		blocksize = (0x1000 - (0xFFF & address)) >> 2;
+		if (count < blocksize)
 			blocksize = count;
 		ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_SINGLE, address);
-		for (readcount=0; readcount<blocksize; readcount++)
+		for (readcount = 0; readcount < blocksize; readcount++)
 		{
-			ahbap_read_reg_u32(swjdp, AHBAP_DRW, buffer+readcount );
+			ahbap_read_reg_u32(swjdp, AHBAP_DRW, buffer + readcount );
 		}
-		if (swjdp_transaction_endcheck(swjdp)==ERROR_OK)
+		if (swjdp_transaction_endcheck(swjdp) == ERROR_OK)
 		{
-			count = count-blocksize;
-			address = address+4*blocksize;
+			count = count - blocksize;
+			address = address + 4 * blocksize;
 			buffer = buffer + blocksize;
 		}
 		else
 		{
 			errorcount++;
 		}
-		if (errorcount>1)
+		if (errorcount > 1)
 		{
 			WARNING("Block read error address %x, count %x", address, count);
 			return ERROR_JTAG_DEVICE_ERROR;
@@ -571,12 +613,12 @@ int ahbap_read_coreregister_u32(swjdp_common_t *swjdp, u32 *value, int regnum)
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 
 	/* ahbap_write_system_u32(swjdp, DCB_DCRSR, regnum); */
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRSR&0xFFFFFFF0);
-	ahbap_write_reg_u32(swjdp, AHBAP_BD0|DCB_DCRSR&0xC, regnum );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRSR & 0xFFFFFFF0);
+	ahbap_write_reg_u32(swjdp, AHBAP_BD0 | (DCB_DCRSR & 0xC), regnum );
 
 	/* ahbap_read_system_u32(swjdp, DCB_DCRDR, value); */
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRDR&0xFFFFFFF0);
-	ahbap_read_reg_u32(swjdp, AHBAP_BD0|DCB_DCRDR&0xC, value );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRDR & 0xFFFFFFF0);
+	ahbap_read_reg_u32(swjdp, AHBAP_BD0 | (DCB_DCRDR & 0xC), value );
 	
 	return swjdp_transaction_endcheck(swjdp);
 }
@@ -586,65 +628,64 @@ int ahbap_write_coreregister_u32(swjdp_common_t *swjdp, u32 value, int regnum)
 	swjdp->trans_mode = TRANS_MODE_COMPOSITE;
 	
 	/* ahbap_write_system_u32(swjdp, DCB_DCRDR, core_regs[i]); */
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRDR&0xFFFFFFF0);
-	ahbap_write_reg_u32(swjdp, AHBAP_BD0|DCB_DCRDR&0xC, value );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRDR & 0xFFFFFFF0);
+	ahbap_write_reg_u32(swjdp, AHBAP_BD0 | (DCB_DCRDR & 0xC), value );
 
 	/* ahbap_write_system_u32(swjdp, DCB_DCRSR, i | DCRSR_WnR	); */
-	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRSR&0xFFFFFFF0);
-	ahbap_write_reg_u32(swjdp, AHBAP_BD0|DCB_DCRSR&0xC, regnum | DCRSR_WnR );
+	ahbap_setup_accessport(swjdp, CSW_32BIT | CSW_ADDRINC_OFF, DCB_DCRSR & 0xFFFFFFF0);
+	ahbap_write_reg_u32(swjdp, AHBAP_BD0 | (DCB_DCRSR & 0xC), regnum | DCRSR_WnR );
 
 	return swjdp_transaction_endcheck(swjdp);
 }
 
 int ahbap_debugport_init(swjdp_common_t *swjdp)
 {
-
-u32 idreg, romaddr, dummy;
-u32 ctrlstat;
-int cnt=0;
-DEBUG("");
+	u32 idreg, romaddr, dummy;
+	u32 ctrlstat;
+	int cnt = 0;
+	
+	DEBUG(" ");
+	
 	swjdp->ap_csw_value = -1;
 	swjdp->ap_tar_value = -1;
 	swjdp->trans_mode = TRANS_MODE_ATOMIC;
 	swjdp_read_dpacc(swjdp, &dummy, DP_CTRL_STAT);
 	swjdp_write_dpacc(swjdp, SSTICKYERR, DP_CTRL_STAT);
 	swjdp_read_dpacc(swjdp, &dummy, DP_CTRL_STAT);
-
-	swjdp->dp_ctrl_stat = CDBGPWRUPREQ|CSYSPWRUPREQ;
+	
+	swjdp->dp_ctrl_stat = CDBGPWRUPREQ | CSYSPWRUPREQ;
 
 	swjdp_write_dpacc(swjdp, swjdp->dp_ctrl_stat, DP_CTRL_STAT);
 	swjdp_read_dpacc(swjdp, &ctrlstat, DP_CTRL_STAT);
 	jtag_execute_queue();
 
 	/* Check that we have debug power domains activated */
-	while (!(ctrlstat & CDBGPWRUPACK) && (cnt++<10))
+	while (!(ctrlstat & CDBGPWRUPACK) && (cnt++ < 10))
 	{
-		DEBUG("      swjdp: wait CDBGPWRUPACK");
+		DEBUG("swjdp: wait CDBGPWRUPACK");
 		swjdp_read_dpacc(swjdp, &ctrlstat, DP_CTRL_STAT);
 		jtag_execute_queue();
-
 		usleep(10000);
 	}
 
-	while (!(ctrlstat & CSYSPWRUPACK) && (cnt++<10))
+	while (!(ctrlstat & CSYSPWRUPACK) && (cnt++ < 10))
 	{
-		DEBUG("      swjdp: wait CSYSPWRUPACK");
+		DEBUG("swjdp: wait CSYSPWRUPACK");
 		swjdp_read_dpacc(swjdp, &ctrlstat, DP_CTRL_STAT);
 		jtag_execute_queue();
 		usleep(10000);
 	}
 
-
 	swjdp_read_dpacc(swjdp, &dummy, DP_CTRL_STAT);
 	/* With debug power on we can activate OVERRUN checking */
-	swjdp->dp_ctrl_stat = CDBGPWRUPREQ|CSYSPWRUPREQ|CORUNDETECT;
+	swjdp->dp_ctrl_stat = CDBGPWRUPREQ | CSYSPWRUPREQ | CORUNDETECT;
 	swjdp_write_dpacc(swjdp, swjdp->dp_ctrl_stat , DP_CTRL_STAT);
 	swjdp_read_dpacc(swjdp, &dummy, DP_CTRL_STAT);
 	
-	ahbap_read_reg_u32(swjdp, 0xFC, &idreg );
-	ahbap_read_reg_u32(swjdp, 0xF8, &romaddr );
+	ahbap_read_reg_u32(swjdp, 0xFC, &idreg);
+	ahbap_read_reg_u32(swjdp, 0xF8, &romaddr);
 	
-DEBUG("AHB-AP ID Register 0x%x, Debug ROM Address 0x%x",idreg,romaddr);	
+	DEBUG("AHB-AP ID Register 0x%x, Debug ROM Address 0x%x", idreg, romaddr);	
 	
 	return ERROR_OK;
 }
diff --git a/src/target/cortex_swjdp.h b/src/target/cortex_swjdp.h
index 093f2ed5..ad6c69a6 100644
--- a/src/target/cortex_swjdp.h
+++ b/src/target/cortex_swjdp.h
@@ -44,8 +44,6 @@
 #define CSYSPWRUPREQ	(1<<30)
 #define CSYSPWRUPACK	(1<<31)
 
-
-
 #define	AHBAP_CSW	0x00
 #define AHBAP_TAR	0x04
 #define AHBAP_DRW	0x0C
@@ -56,7 +54,6 @@
 #define AHBAP_DBGROMA	0xF8
 #define AHBAP_IDR	0xFC
 
-
 #define CSW_8BIT		0
 #define CSW_16BIT		1
 #define CSW_32BIT		2
@@ -65,17 +62,17 @@
 #define CSW_ADDRINC_OFF		0
 #define CSW_ADDRINC_SINGLE	(1<<4)
 #define CSW_ADDRINC_PACKED	(2<<4)
-#define CSW_HPROT	(1<<25)
+#define CSW_HPROT			(1<<25)
 #define CSW_MASTER_DEBUG	(1<<29)
-#define CSW_DBGSWENABLE	(1<<31)
-#define TRANS_MODE_NONE	0
+#define CSW_DBGSWENABLE		(1<<31)
 
+/* transaction mode */
+#define TRANS_MODE_NONE		0
 /* Transaction waits for previous to complete */
 #define TRANS_MODE_ATOMIC	1
 /* Freerunning transactions with delays and overrun checking */
 #define TRANS_MODE_COMPOSITE	2
 
-
 typedef struct swjdp_reg_s
 {
 	int addr;
@@ -96,7 +93,7 @@ typedef struct swjdp_common_s
 	u8  trans_mode;
 	u8  trans_rw;
 	u8  ack;
-	u32 * trans_value;
+	u32 *trans_value;
 } swjdp_common_t;
 
 /* Internal functions used in the module, partial transactions, use with caution */
@@ -112,7 +109,6 @@ extern int ahbap_read_system_u32(swjdp_common_t *swjdp, u32 address, u32 *value)
 extern int ahbap_write_system_u32(swjdp_common_t *swjdp, u32 address, u32 value);
 extern int swjdp_transaction_endcheck(swjdp_common_t *swjdp);
 
-
 /* External interface, complete atomic operations  */
 /* Host endian word transfer of single memory and system registers */
 extern int ahbap_read_system_atomic_u32(swjdp_common_t *swjdp, u32 address, u32 *value);
@@ -124,6 +120,11 @@ extern int ahbap_write_block(swjdp_common_t *swjdp, u8 *buffer, int bytecount, u
 extern int ahbap_read_coreregister_u32(swjdp_common_t *swjdp, u32 *value, int regnum);
 extern int ahbap_write_coreregister_u32(swjdp_common_t *swjdp, u32 value, int regnum);
 
+extern int ahbap_read_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address);
+extern int ahbap_read_buf_u16(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address);
+extern int ahbap_write_buf(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address);
+extern int ahbap_write_buf_u16(swjdp_common_t *swjdp, u8 *buffer, int count, u32 address);
+
 /* Initialisation of the debug system, power domains and registers */
 extern int ahbap_debugport_init(swjdp_common_t *swjdp);
 
diff --git a/src/target/target.c b/src/target/target.c
index 2eeb2bfc..07c450cd 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -653,6 +653,7 @@ int target_write_buffer(struct target_s *target, u32 address, u32 size, u8 *buff
 	{
 		if ((retval = target->type->write_memory(target, address, 1, size, buffer)) != ERROR_OK)
 			return retval;
+		return ERROR_OK;
 	}
 	
 	/* handle unaligned head bytes */
@@ -711,6 +712,7 @@ int target_read_buffer(struct target_s *target, u32 address, u32 size, u8 *buffe
 	{
 		if ((retval = target->type->read_memory(target, address, 1, size, buffer)) != ERROR_OK)
 			return retval;
+		return ERROR_OK;
 	}
 	
 	/* handle unaligned head bytes */