summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorØyvind Harboe <oyvind.harboe@zylin.com>2010-04-29 03:49:32 +0200
committerØyvind Harboe <oyvind.harboe@zylin.com>2010-04-29 03:54:43 +0200
commit78248f1df67d1b4feefd8ac4e459acd7599d6af2 (patch)
treefc9b7ea431d6f9e13ca135fa7b3c175cf7549a48
parent49b7905cae66ee9e011c71aff758fafba823f87f (diff)
downloadopenocd+libswd-78248f1df67d1b4feefd8ac4e459acd7599d6af2.tar.gz
openocd+libswd-78248f1df67d1b4feefd8ac4e459acd7599d6af2.tar.bz2
openocd+libswd-78248f1df67d1b4feefd8ac4e459acd7599d6af2.tar.xz
openocd+libswd-78248f1df67d1b4feefd8ac4e459acd7599d6af2.zip
flash: write_image will now pad erase to nearest sector
this is done for unlocking and it is a simple omission that it wasn't done for sectors. The unnerving thing is that nobody has complained about this until now.... Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
-rw-r--r--src/flash/nor/core.c10
1 files changed, 2 insertions, 8 deletions
diff --git a/src/flash/nor/core.c b/src/flash/nor/core.c
index b8dda96b..1ff41937 100644
--- a/src/flash/nor/core.c
+++ b/src/flash/nor/core.c
@@ -1,6 +1,6 @@
/***************************************************************************
* Copyright (C) 2005 by Dominic Rath <Dominic.Rath@gmx.de> *
- * Copyright (C) 2007,2008 Øyvind Harboe <oyvind.harboe@zylin.com> *
+ * Copyright (C) 2007-2010 Øyvind Harboe <oyvind.harboe@zylin.com> *
* Copyright (C) 2008 by Spencer Oliver <spen@spen-soft.co.uk> *
* Copyright (C) 2009 Zachary T Welch <zw@superlucidity.net> *
* *
@@ -519,12 +519,6 @@ int flash_write_unlock(struct target *target, struct image *image,
struct flash_bank *c;
int *padding;
- /* REVISIT do_pad should perhaps just be another parameter.
- * GDB wouldn't ever need it, since it erases separately.
- * But "flash write_image" commands might want that option.
- */
- bool do_pad = false;
-
section = 0;
section_offset = 0;
@@ -694,7 +688,7 @@ int flash_write_unlock(struct target *target, struct image *image,
{
/* calculate and erase sectors */
retval = flash_erase_address_range(target,
- do_pad, run_address, run_size);
+ true, run_address, run_size);
}
}