From 2a4d3c03cd9f6ec1d761a6fb8795d6aac95e9fe3 Mon Sep 17 00:00:00 2001 From: David Brownell Date: Tue, 10 Nov 2009 02:01:20 -0800 Subject: [PATCH] Target: minor cleanup - improve some names -- a "default" prefix is not descriptive - add doxygen @todo entries for some issues - avr8 isn't ever going to need those MMU hooks Signed-off-by: David Brownell --- src/target/avrt.c | 4 ---- src/target/target.c | 30 +++++++++++++++++++++--------- 2 files changed, 21 insertions(+), 13 deletions(-) diff --git a/src/target/avrt.c b/src/target/avrt.c index d3e6282bbf..98433cf71c 100644 --- a/src/target/avrt.c +++ b/src/target/avrt.c @@ -94,10 +94,6 @@ target_type_t avr_target = .register_commands = avr_register_commands, .target_create = avr_target_create, .init_target = avr_init_target, -/* - .virt2phys = avr_virt2phys, - .mmu = avr_mmu -*/ }; int avr_register_commands(struct command_context_s *cmd_ctx) diff --git a/src/target/target.c b/src/target/target.c index b7df1d8841..5cdfa5dc00 100644 --- a/src/target/target.c +++ b/src/target/target.c @@ -460,13 +460,14 @@ int target_process_reset(struct command_context_s *cmd_ctx, enum target_reset_mo return retval; } -static int default_virt2phys(struct target_s *target, uint32_t virtual, uint32_t *physical) +static int identity_virt2phys(struct target_s *target, + uint32_t virtual, uint32_t *physical) { *physical = virtual; return ERROR_OK; } -static int default_mmu(struct target_s *target, int *enabled) +static int no_mmu(struct target_s *target, int *enabled) { *enabled = 0; return ERROR_OK; @@ -738,7 +739,7 @@ int target_mcr(struct target_s *target, int cpnum, uint32_t op1, uint32_t op2, u } static int -default_read_phys_memory(struct target_s *target, uint32_t address, +err_read_phys_memory(struct target_s *target, uint32_t address, uint32_t size, uint32_t count, uint8_t *buffer) { LOG_ERROR("Not implemented: %s", __func__); @@ -746,7 +747,7 @@ default_read_phys_memory(struct target_s *target, uint32_t address, } static int -default_write_phys_memory(struct target_s *target, uint32_t address, +err_write_phys_memory(struct target_s *target, uint32_t address, uint32_t size, uint32_t count, uint8_t *buffer) { LOG_ERROR("Not implemented: %s", __func__); @@ -775,19 +776,23 @@ int target_init(struct command_context_s *cmd_ctx) /* Set up default functions if none are provided by target */ if (target->type->virt2phys == NULL) { - target->type->virt2phys = default_virt2phys; + target->type->virt2phys = identity_virt2phys; } if (target->type->read_phys_memory == NULL) { - target->type->read_phys_memory = default_read_phys_memory; + target->type->read_phys_memory = err_read_phys_memory; } if (target->type->write_phys_memory == NULL) { - target->type->write_phys_memory = default_write_phys_memory; + target->type->write_phys_memory = err_write_phys_memory; } + /** + * @todo MCR/MRC are ARM-specific; don't require them in + * all targets, or for ARMs without coprocessors. + */ if (target->type->mcr == NULL) { target->type->mcr = default_mcr; @@ -810,6 +815,12 @@ int target_init(struct command_context_s *cmd_ctx) } + /** + * @todo get rid of those *memory_imp() methods, now that all + * callers are using target_*_memory() accessors ... and make + * sure the "physical" paths handle the same issues. + */ + /* a non-invasive way(in terms of patches) to add some code that * runs before the type->write/read_memory implementation */ @@ -824,7 +835,7 @@ int target_init(struct command_context_s *cmd_ctx) if (target->type->mmu == NULL) { - target->type->mmu = default_mmu; + target->type->mmu = no_mmu; } target = target->next; } @@ -4780,10 +4791,11 @@ int target_register_user_commands(struct command_context_s *cmd_ctx) "loads active fast load image to current target " "- mainly for profiling purposes"); - + /** @todo don't register virt2phys() unless target supports it */ register_command(cmd_ctx, NULL, "virt2phys", handle_virt2phys_command, COMMAND_ANY, "translate a virtual address into a physical address"); + register_command(cmd_ctx, NULL, "reg", handle_reg_command, COMMAND_EXEC, "display or set a register"); -- 2.30.2