From: Jim Paris <jim@jtan.com>
Date: Fri, 31 Jan 2014 21:26:35 +0000 (-0500)
Subject: When ignoring the end of a file, make sure we don't ignore partial words.
X-Git-Url: https://git.gag.com/?a=commitdiff_plain;h=5dcd45d8a48aaead2a2d1d1b9c36ab14f507628b;p=fw%2Fstlink

When ignoring the end of a file, make sure we don't ignore partial words.

Consider a 128-byte write to the chip.  If the last 2 bytes are
considered "empty", then len is adjusted to 126, and run_flash_loader
will only copy 126 bytes to RAM.  However, run_flash_loader then
proceeds to round up to 32 words (128 bytes) when flashing, which has
the effect of clobbering those last two "empty" bytes with junk data.
---

diff --git a/src/stlink-common.c b/src/stlink-common.c
index b7645fe..b942dea 100644
--- a/src/stlink-common.c
+++ b/src/stlink-common.c
@@ -1752,8 +1752,10 @@ int stlink_fwrite_flash(stlink_t *sl, const char* path, stm32_addr_t addr) {
 	else
 	    num_empty = 0;
     }
+    /* Round down to words */
+    num_empty -= (num_empty & 3);
     if(num_empty != 0) {
-	ILOG("Ignoring %d bytes of Zeros at end of file\n",num_empty);
+	ILOG("Ignoring %d bytes of 0x%02x at end of file\n", num_empty, erased_pattern);
 	mf.len -= num_empty;
     }
     err = stlink_write_flash(sl, addr, mf.base, mf.len);