X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Ftarget%2Farm11.c;h=59b6c4fe7c85032515b3da368ffe7ebf5558e46d;hb=b8387ea272771accd2681b75b805043ae9d2b82a;hp=7c947fc2e7ce5ce98d90f915d5fe5467905b19c3;hpb=f370d70670bd5e30befe6fbfbc8d472e760f032b;p=fw%2Fopenocd diff --git a/src/target/arm11.c b/src/target/arm11.c index 7c947fc2e..59b6c4fe7 100644 --- a/src/target/arm11.c +++ b/src/target/arm11.c @@ -1,5 +1,7 @@ /*************************************************************************** * Copyright (C) 2008 digenius technology GmbH. * + * + * Copyright (C) 2008 Oyvind Harboe oyvind.harboe@zylin.com * * * * 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 * @@ -783,10 +785,10 @@ int arm11_resume(struct target_s *target, int current, u32 address, int handle_b LOG_DEBUG("target->state: %s", target_state_strings[target->state]); if (target->state != TARGET_HALTED) - { - LOG_WARNING("target was not halted"); - return ERROR_TARGET_NOT_HALTED; - } + { + LOG_ERROR("Target not halted"); + return ERROR_TARGET_NOT_HALTED; + } if (!current) R(PC) = address; @@ -1237,7 +1239,7 @@ int arm11_write_memory(struct target_s *target, u32 address, u32 size, u32 count LOG_ERROR("use 'arm11 memwrite burst disable' to disable fast burst mode"); if (arm11_config_memwrite_error_fatal) - exit(-1); + return ERROR_FAIL; } } #endif @@ -1349,8 +1351,7 @@ int arm11_target_command(struct command_context_s *cmd_ctx, char *cmd, char **ar if (argc < 4) { - LOG_ERROR("'target arm11' 4th argument "); - exit(-1); + return ERROR_COMMAND_SYNTAX_ERROR; } int chain_pos = strtoul(args[3], NULL, 0); @@ -1369,8 +1370,8 @@ int arm11_target_command(struct command_context_s *cmd_ctx, char *cmd, char **ar if (device->ir_length != 5) { - LOG_ERROR("'target arm11' expects 'jtag_device 5 0x01 0x1F 0x1E'"); - exit(-1); + LOG_ERROR("'target arm11' expects 'jtag_device 5 0x01 0x1F 0x1E'"); + return ERROR_COMMAND_SYNTAX_ERROR; } target->arch_info = arm11; @@ -1424,8 +1425,8 @@ int arm11_examine(struct target_s *target) case 0x07B76000: LOG_INFO("found ARM1176"); break; default: { - LOG_ERROR("'target arm11' expects IDCODE 0x*7B*7****"); - exit(-1); + LOG_ERROR("'target arm11' expects IDCODE 0x*7B*7****"); + return ERROR_FAIL; } } @@ -1435,7 +1436,7 @@ int arm11_examine(struct target_s *target) arm11->debug_version != ARM11_DEBUG_V61) { LOG_ERROR("Only ARMv6 v6 and v6.1 architectures supported."); - exit(-1); + return ERROR_FAIL; } @@ -1543,7 +1544,7 @@ void arm11_build_reg_cache(target_t *target) ARM11_REGCACHE_COUNT != asizeof(arm11_reg_defs) || ARM11_REGCACHE_COUNT != ARM11_RC_MAX) { - LOG_ERROR("arm11->reg_values inconsistent (%d " ZU " " ZU " %d)", ARM11_REGCACHE_COUNT, asizeof(arm11->reg_values), asizeof(arm11_reg_defs), ARM11_RC_MAX); + LOG_ERROR("BUG: arm11->reg_values inconsistent (%d " ZU " " ZU " %d)", ARM11_REGCACHE_COUNT, asizeof(arm11->reg_values), asizeof(arm11_reg_defs), ARM11_RC_MAX); exit(-1); }