X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=blobdiff_plain;f=src%2Ftarget%2Fstm32_stlink.c;h=d2f38e5af384d4eb8194977ac01022472baf30ae;hp=dec2b24bf1ac9a670aad91642cbb4759311cc297;hb=dbb8de15e39390bf5f323dddb8e5f090515977d1;hpb=c2ab3b4d5efba0201d88d899dc0fd310574ef5b9 diff --git a/src/target/stm32_stlink.c b/src/target/stm32_stlink.c index dec2b24bf1..d2f38e5af3 100644 --- a/src/target/stm32_stlink.c +++ b/src/target/stm32_stlink.c @@ -2,6 +2,9 @@ * Copyright (C) 2011 by Mathias Kuester * * Mathias Kuester * * * + * Copyright (C) 2011 by Spencer Oliver * + * spen@spen-soft.co.uk * + * * * 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 * @@ -17,11 +20,13 @@ * Free Software Foundation, Inc., * * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. * ***************************************************************************/ + #ifdef HAVE_CONFIG_H #include "config.h" #endif #include "jtag/jtag.h" +#include "jtag/stlink/stlink_transport.h" #include "jtag/stlink/stlink_interface.h" #include "jtag/stlink/stlink_layout.h" #include "register.h" @@ -31,6 +36,10 @@ #include "target_type.h" #include "armv7m.h" #include "cortex_m.h" +#include "arm_semihosting.h" + +#define ARMV7M_SCS_DCRSR 0xe000edf4 +#define ARMV7M_SCS_DCRDR 0xe000edf8 static inline struct stlink_interface_s *target_to_stlink(struct target *target) { @@ -38,8 +47,8 @@ static inline struct stlink_interface_s *target_to_stlink(struct target *target) } static int stm32_stlink_load_core_reg_u32(struct target *target, - enum armv7m_regtype type, - uint32_t num, uint32_t *value) + enum armv7m_regtype type, + uint32_t num, uint32_t *value) { int retval; struct stlink_interface_s *stlink_if = target_to_stlink(target); @@ -53,15 +62,25 @@ static int stm32_stlink_load_core_reg_u32(struct target *target, switch (num) { case 0 ... 18: /* read a normal core register */ - retval = - stlink_if->layout->api->read_reg(stlink_if->fd, num, value); + retval = stlink_if->layout->api->read_reg(stlink_if->fd, num, value); if (retval != ERROR_OK) { LOG_ERROR("JTAG failure %i", retval); return ERROR_JTAG_DEVICE_ERROR; } - LOG_DEBUG("load from core reg %i value 0x%" PRIx32 "", - (int)num, *value); + LOG_DEBUG("load from core reg %i value 0x%" PRIx32 "", (int)num, *value); + break; + + case 33: + case 64 ... 96: + /* Floating-point Status and Registers */ + retval = target_write_u32(target, ARMV7M_SCS_DCRSR, num); + if (retval != ERROR_OK) + return retval; + retval = target_read_u32(target, ARMV7M_SCS_DCRDR, value); + if (retval != ERROR_OK) + return retval; + LOG_DEBUG("load from core reg %i value 0x%" PRIx32 "", (int)num, *value); break; case ARMV7M_PRIMASK: @@ -72,8 +91,7 @@ static int stm32_stlink_load_core_reg_u32(struct target *target, * in one Debug Core register. So say r0 and r2 docs; * it was removed from r1 docs, but still works. */ - retval = - stlink_if->layout->api->read_reg(stlink_if->fd, 20, value); + retval = stlink_if->layout->api->read_reg(stlink_if->fd, 20, value); switch (num) { case ARMV7M_PRIMASK: @@ -105,8 +123,8 @@ static int stm32_stlink_load_core_reg_u32(struct target *target, } static int stm32_stlink_store_core_reg_u32(struct target *target, - enum armv7m_regtype type, - uint32_t num, uint32_t value) + enum armv7m_regtype type, + uint32_t num, uint32_t value) { int retval; uint32_t reg; @@ -133,9 +151,7 @@ static int stm32_stlink_store_core_reg_u32(struct target *target, */ switch (num) { case 0 ... 18: - retval = - stlink_if->layout->api->write_reg(stlink_if->fd, num, - value); + retval = stlink_if->layout->api->write_reg(stlink_if->fd, num, value); if (retval != ERROR_OK) { struct reg *r; @@ -145,8 +161,19 @@ static int stm32_stlink_store_core_reg_u32(struct target *target, r->dirty = r->valid; return ERROR_JTAG_DEVICE_ERROR; } - LOG_DEBUG("write core reg %i value 0x%" PRIx32 "", (int)num, - value); + LOG_DEBUG("write core reg %i value 0x%" PRIx32 "", (int)num, value); + break; + + case 33: + case 64 ... 96: + /* Floating-point Status and Registers */ + retval = target_write_u32(target, ARMV7M_SCS_DCRDR, value); + if (retval != ERROR_OK) + return retval; + retval = target_write_u32(target, ARMV7M_SCS_DCRSR, num | (1<<16)); + if (retval != ERROR_OK) + return retval; + LOG_DEBUG("write core reg %i value 0x%" PRIx32 "", (int)num, value); break; case ARMV7M_PRIMASK: @@ -157,7 +184,8 @@ static int stm32_stlink_store_core_reg_u32(struct target *target, * in one Debug Core register. So say r0 and r2 docs; * it was removed from r1 docs, but still works. */ - /* cortexm3_dap_read_coreregister_u32(swjdp, ®, 20); */ + + stlink_if->layout->api->read_reg(stlink_if->fd, 20, ®); switch (num) { case ARMV7M_PRIMASK: @@ -177,10 +205,9 @@ static int stm32_stlink_store_core_reg_u32(struct target *target, break; } - /* cortexm3_dap_write_coreregister_u32(swjdp, reg, 20); */ + stlink_if->layout->api->write_reg(stlink_if->fd, 20, reg); - LOG_DEBUG("write special reg %i value 0x%" PRIx32 " ", (int)num, - value); + LOG_DEBUG("write special reg %i value 0x%" PRIx32 " ", (int)num, value); break; default: @@ -190,6 +217,16 @@ static int stm32_stlink_store_core_reg_u32(struct target *target, return ERROR_OK; } +static int stm32_stlink_examine_debug_reason(struct target *target) +{ + if ((target->debug_reason != DBG_REASON_DBGRQ) + && (target->debug_reason != DBG_REASON_SINGLESTEP)) { + target->debug_reason = DBG_REASON_BREAKPOINT; + } + + return ERROR_OK; +} + static int stm32_stlink_init_arch_info(struct target *target, struct cortex_m3_common *cortex_m3, struct jtag_tap *tap) @@ -204,6 +241,8 @@ static int stm32_stlink_init_arch_info(struct target *target, armv7m->load_core_reg_u32 = stm32_stlink_load_core_reg_u32; armv7m->store_core_reg_u32 = stm32_stlink_store_core_reg_u32; + armv7m->examine_debug_reason = stm32_stlink_examine_debug_reason; + return ERROR_OK; } @@ -218,7 +257,7 @@ static int stm32_stlink_init_target(struct command_context *cmd_ctx, } static int stm32_stlink_target_create(struct target *target, - Jim_Interp *interp) + Jim_Interp *interp) { LOG_DEBUG("%s", __func__); @@ -232,80 +271,68 @@ static int stm32_stlink_target_create(struct target *target, return ERROR_OK; } -static int stm32_stlink_poll(struct target *target); - -static int stm32_stlink_examine(struct target *target) +static int stm32_stlink_load_context(struct target *target) { - int retval, i; - uint32_t cpuid, fpcr; - struct cortex_m3_common *cortex_m3 = target_to_cm3(target); - - LOG_DEBUG("%s", __func__); - - if (target->tap->hasidcode == false) { - LOG_ERROR("no IDCODE present on device"); - - return ERROR_COMMAND_SYNTAX_ERROR; - } - - if (!target_was_examined(target)) { - target_set_examined(target); - - stm32_stlink_poll(target); - - LOG_INFO("IDCODE %x", target->tap->idcode); - - /* Read from Device Identification Registers */ - retval = target_read_u32(target, CPUID, &cpuid); - if (retval != ERROR_OK) - return retval; - - if (((cpuid >> 4) & 0xc3f) == 0xc23) - LOG_DEBUG("Cortex-M3 r%" PRId8 "p%" PRId8 " processor detected", - (uint8_t)((cpuid >> 20) & 0xf), (uint8_t)((cpuid >> 0) & 0xf)); - LOG_DEBUG("cpuid: 0x%8.8" PRIx32 "", cpuid); - - /* Setup FPB */ - target_read_u32(target, FP_CTRL, &fpcr); - cortex_m3->auto_bp_type = 1; - cortex_m3->fp_num_code = ((fpcr >> 8) & 0x70) | - ((fpcr >> 4) & 0xF); /* bits [14:12] and [7:4] */ - 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(struct cortex_m3_fp_comparator)); - cortex_m3->fpb_enabled = fpcr & 1; - 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; - } - LOG_DEBUG("FPB fpcr 0x%" PRIx32 ", numcode %i, numlit %i", fpcr, - cortex_m3->fp_num_code, cortex_m3->fp_num_lit); - - /* Setup DWT */ - cortex_m3_dwt_setup(cortex_m3, target); + struct armv7m_common *armv7m = target_to_armv7m(target); + int num_regs = armv7m->core_cache->num_regs; - /* These hardware breakpoints only work for code in flash! */ - LOG_INFO("%s: hardware has %d breakpoints, %d watchpoints", - target_name(target), - cortex_m3->fp_num_code, - cortex_m3->dwt_num_comp); + for (int i = 0; i < num_regs; i++) { + if (!armv7m->core_cache->reg_list[i].valid) + armv7m->read_core_reg(target, i); } return ERROR_OK; } -static int stm32_stlink_load_context(struct target *target) +static int stlink_debug_entry(struct target *target) { struct armv7m_common *armv7m = target_to_armv7m(target); + struct arm *arm = &armv7m->arm; + struct reg *r; + uint32_t xPSR; + int retval; - for (unsigned i = 0; i < 23; i++) { - if (!armv7m->core_cache->reg_list[i].valid) - armv7m->read_core_reg(target, i); + retval = armv7m->examine_debug_reason(target); + if (retval != ERROR_OK) + return retval; + + stm32_stlink_load_context(target); + + r = armv7m->core_cache->reg_list + ARMV7M_xPSR; + xPSR = buf_get_u32(r->value, 0, 32); + + /* Are we in an exception handler */ + if (xPSR & 0x1FF) { + armv7m->core_mode = ARMV7M_MODE_HANDLER; + armv7m->exception_number = (xPSR & 0x1FF); + + arm->core_mode = ARM_MODE_HANDLER; + arm->map = armv7m_msp_reg_map; + } else { + unsigned control = buf_get_u32(armv7m->core_cache + ->reg_list[ARMV7M_CONTROL].value, 0, 2); + + /* is this thread privileged? */ + armv7m->core_mode = control & 1; + arm->core_mode = armv7m->core_mode + ? ARM_MODE_USER_THREAD + : ARM_MODE_THREAD; + + /* which stack is it using? */ + if (control & 2) + arm->map = armv7m_psp_reg_map; + else + arm->map = armv7m_msp_reg_map; + + armv7m->exception_number = 0; } - return ERROR_OK; + LOG_DEBUG("entered debug state in core mode: %s at PC 0x%" PRIx32 ", target->state: %s", + armv7m_mode_strings[armv7m->core_mode], + *(uint32_t *)(arm->pc->value), + target_state_name(target)); + + return retval; } static int stm32_stlink_poll(struct target *target) @@ -317,8 +344,7 @@ static int stm32_stlink_poll(struct target *target) state = stlink_if->layout->api->state(stlink_if->fd); if (state == TARGET_UNKNOWN) { - LOG_ERROR - ("jtag status contains invalid mode value - communication failure"); + LOG_ERROR("jtag status contains invalid mode value - communication failure"); return ERROR_TARGET_FAILURE; } @@ -326,20 +352,19 @@ static int stm32_stlink_poll(struct target *target) return ERROR_OK; if (state == TARGET_HALTED) { - target_call_event_callbacks(target, TARGET_EVENT_DEBUG_HALTED); target->state = state; - stm32_stlink_load_context(target); + int retval = stlink_debug_entry(target); + if (retval != ERROR_OK) + return retval; - LOG_INFO("halted: PC: 0x%x", buf_get_u32(armv7m->arm.pc->value, 0, 32)); - } + if (arm_semihosting(target, &retval) != 0) + return retval; - return ERROR_OK; -} + target_call_event_callbacks(target, TARGET_EVENT_HALTED); + LOG_DEBUG("halted: PC: 0x%x", buf_get_u32(armv7m->arm.pc->value, 0, 32)); + } -static int stm32_stlink_arch_state(struct target *target) -{ - LOG_DEBUG("%s", __func__); return ERROR_OK; } @@ -364,9 +389,12 @@ static int stm32_stlink_assert_reset(struct target *target) /* registers are now invalid */ register_cache_invalidate(armv7m->core_cache); - stm32_stlink_load_context(target); - - target->state = TARGET_HALTED; + if (target->reset_halt) { + target->state = TARGET_RESET; + target->debug_reason = DBG_REASON_DBGRQ; + } else { + target->state = TARGET_HALTED; + } return ERROR_OK; } @@ -410,10 +438,8 @@ static int stm32_stlink_halt(struct target *target) return ERROR_OK; } - if (target->state == TARGET_UNKNOWN) { - LOG_WARNING - ("target was in unknown state when halt was requested"); - } + if (target->state == TARGET_UNKNOWN) + LOG_WARNING("target was in unknown state when halt was requested"); res = stlink_if->layout->api->halt(stlink_if->fd); @@ -426,8 +452,8 @@ static int stm32_stlink_halt(struct target *target) } static int stm32_stlink_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 res; struct stlink_interface_s *stlink_if = target_to_stlink(target); @@ -437,7 +463,7 @@ static int stm32_stlink_resume(struct target *target, int current, struct reg *pc; LOG_DEBUG("%s %d %x %d %d", __func__, current, address, - handle_breakpoints, debug_execution); + handle_breakpoints, debug_execution); if (target->state != TARGET_HALTED) { LOG_WARNING("target not halted"); @@ -469,8 +495,8 @@ static int stm32_stlink_resume(struct target *target, int current, breakpoint = breakpoint_find(target, resume_pc); if (breakpoint) { LOG_DEBUG("unset breakpoint at 0x%8.8" PRIx32 " (ID: %d)", - breakpoint->address, - breakpoint->unique_id); + breakpoint->address, + breakpoint->unique_id); cortex_m3_unset_breakpoint(target, breakpoint); res = stlink_if->layout->api->step(stlink_if->fd); @@ -489,13 +515,13 @@ static int stm32_stlink_resume(struct target *target, int current, target->state = TARGET_RUNNING; - target_call_event_callbacks(target, TARGET_EVENT_DEBUG_RESUMED); + target_call_event_callbacks(target, TARGET_EVENT_RESUMED); return ERROR_OK; } static int stm32_stlink_step(struct target *target, int current, - uint32_t address, int handle_breakpoints) + uint32_t address, int handle_breakpoints) { int res; struct stlink_interface_s *stlink_if = target_to_stlink(target); @@ -545,19 +571,17 @@ static int stm32_stlink_step(struct target *target, int current, if (breakpoint) cortex_m3_set_breakpoint(target, breakpoint); - target->debug_reason = DBG_REASON_SINGLESTEP; + stlink_debug_entry(target); target_call_event_callbacks(target, TARGET_EVENT_HALTED); - stm32_stlink_load_context(target); - LOG_INFO("halted: PC: 0x%x", buf_get_u32(armv7m->arm.pc->value, 0, 32)); return ERROR_OK; } static int stm32_stlink_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 res; uint32_t buffer_threshold = 128; @@ -587,13 +611,11 @@ static int stm32_stlink_read_memory(struct target *target, uint32_t address, c = count; if (size != 4) - res = - stlink_if->layout->api->read_mem8(stlink_if->fd, address, - c, dst); + res = stlink_if->layout->api->read_mem8(stlink_if->fd, + address, c, dst); else - res = - stlink_if->layout->api->read_mem32(stlink_if->fd, address, - c, (uint32_t *)dst); + res = stlink_if->layout->api->read_mem32(stlink_if->fd, + address, c, (uint32_t *)dst); if (res != ERROR_OK) return res; @@ -607,8 +629,8 @@ static int stm32_stlink_read_memory(struct target *target, uint32_t address, } static int stm32_stlink_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 res; uint32_t buffer_threshold = 128; @@ -638,13 +660,11 @@ static int stm32_stlink_write_memory(struct target *target, uint32_t address, c = count; if (size != 4) - res = - stlink_if->layout->api->write_mem8(stlink_if->fd, address, - c, dst); + res = stlink_if->layout->api->write_mem8(stlink_if->fd, + address, c, dst); else - res = - stlink_if->layout->api->write_mem32(stlink_if->fd, address, - c, (uint32_t *)dst); + res = stlink_if->layout->api->write_mem32(stlink_if->fd, + address, c, (uint32_t *)dst); if (res != ERROR_OK) return res; @@ -658,8 +678,8 @@ static int stm32_stlink_write_memory(struct target *target, uint32_t address, } static int stm32_stlink_bulk_write_memory(struct target *target, - uint32_t address, uint32_t count, - const uint8_t *buffer) + uint32_t address, uint32_t count, + const uint8_t *buffer) { return stm32_stlink_write_memory(target, address, 4, count, buffer); } @@ -669,10 +689,10 @@ struct target_type stm32_stlink_target = { .init_target = stm32_stlink_init_target, .target_create = stm32_stlink_target_create, - .examine = stm32_stlink_examine, + .examine = cortex_m3_examine, .poll = stm32_stlink_poll, - .arch_state = stm32_stlink_arch_state, + .arch_state = armv7m_arch_state, .assert_reset = stm32_stlink_assert_reset, .deassert_reset = stm32_stlink_deassert_reset, @@ -687,6 +707,8 @@ struct target_type stm32_stlink_target = { .read_memory = stm32_stlink_read_memory, .write_memory = stm32_stlink_write_memory, .bulk_write_memory = stm32_stlink_bulk_write_memory, + .checksum_memory = armv7m_checksum_memory, + .blank_check_memory = armv7m_blank_check_memory, .run_algorithm = armv7m_run_algorithm, .start_algorithm = armv7m_start_algorithm,