diff options
author | Spencer Oliver <spen@spen-soft.co.uk> | 2012-08-16 11:05:47 +0100 |
---|---|---|
committer | Freddie Chopin <freddie.chopin@gmail.com> | 2012-08-24 16:52:52 +0000 |
commit | 63a23e6fc862b94f00e0833ab474bd02901a019f (patch) | |
tree | 0da76d75abb9eaadbe155be452cb7f47c87a1d76 /src/target/target.c | |
parent | 349b459e98920cb3bd5770eae88c99e7780dd64e (diff) |
target: remove unused working area 'user' field
working_area::user has never been used so lets remove it.
Change-Id: I1200311b34248549c1fe30c9f675e6129b7bebee
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/781
Tested-by: jenkins
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
Diffstat (limited to 'src/target/target.c')
-rw-r--r-- | src/target/target.c | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/src/target/target.c b/src/target/target.c index b8e4c2ff..c71c5360 100644 --- a/src/target/target.c +++ b/src/target/target.c @@ -1405,7 +1405,6 @@ static void target_split_working_area(struct working_area *area, uint32_t size) new_wa->size = area->size - size; new_wa->address = area->address + size; new_wa->backup = NULL; - new_wa->user = NULL; new_wa->free = true; area->next = new_wa; @@ -1494,7 +1493,6 @@ int target_alloc_working_area_try(struct target *target, uint32_t size, struct w new_wa->size = target->working_area_size & ~3UL; /* 4-byte align */ new_wa->address = target->working_area; new_wa->backup = NULL; - new_wa->user = NULL; new_wa->free = true; } @@ -1538,9 +1536,6 @@ int target_alloc_working_area_try(struct target *target, uint32_t size, struct w c->free = false; *area = c; - /* user pointer */ - c->user = area; - print_wa_layout(target); return ERROR_OK; @@ -1591,13 +1586,6 @@ static int target_free_working_area_restore(struct target *target, struct workin LOG_DEBUG("freed %"PRIu32" bytes of working area at address 0x%08"PRIx32, area->size, area->address); - /* mark user pointer invalid */ - /* TODO: Is this really safe? It points to some previous caller's memory. - * How could we know that the area pointer is still in that place and not - * some other vital data? What's the purpose of this, anyway? */ - *area->user = NULL; - area->user = NULL; - target_merge_working_areas(target); print_wa_layout(target); @@ -1625,8 +1613,6 @@ static void target_free_all_working_areas_restore(struct target *target, int res if (restore) target_restore_working_area(target, c); c->free = true; - *c->user = NULL; /* Same as above */ - c->user = NULL; } c = c->next; } |