diff options
author | Claus Gindhart <claus.gindhart@kontron.com> | 2008-04-24 09:07:57 +0000 |
---|---|---|
committer | Stefan Reinauer <stepan@openbios.org> | 2008-04-24 09:07:57 +0000 |
commit | 20ba8eb7abc973ad70844b29412dc958f0629866 (patch) | |
tree | db682d174409b0a1e02777e223f1b8997ed196be | |
parent | a1491314150eac4494d551917ad5e705977c7603 (diff) | |
download | coreboot-20ba8eb7abc973ad70844b29412dc958f0629866.tar.xz |
Flash pages, which where excluded from updating using the exclude or the
layout option, as well as areas, whose flash contents already contain the
desired data, will be skipped.
These ensures absolute data security of critical areas (BIOS boot block),
e.g. against a sudden power off or a CPU hangup during flashing. As a
nice side effect, it speeds up the flash process, if the BIOS to be flashed
is very similar to the version in flash.
Signed-off-by: Claus Gindhart <claus.gindhart@kontron.com>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3260 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
-rw-r--r-- | util/flashrom/82802ab.c | 34 |
1 files changed, 27 insertions, 7 deletions
diff --git a/util/flashrom/82802ab.c b/util/flashrom/82802ab.c index 38a5ce295d..a191093ccf 100644 --- a/util/flashrom/82802ab.c +++ b/util/flashrom/82802ab.c @@ -27,6 +27,7 @@ */ #include <stdio.h> +#include <string.h> #include <stdint.h> #include "flash.h" @@ -110,6 +111,7 @@ int erase_82802ab_block(struct flashchip *flash, int offset) { volatile uint8_t *bios = flash->virtual_memory + offset; volatile uint8_t *wrprotect = flash->virtual_registers + offset + 2; + int j; uint8_t status; // clear status register @@ -128,6 +130,12 @@ int erase_82802ab_block(struct flashchip *flash, int offset) // now let's see what the register is status = wait_82802ab(flash->virtual_memory); //print_82802ab_status(status); + for (j = 0; j < flash->page_size; j++) { + if (*(bios + j) != 0xFF) { + printf("BLOCK ERASE failed at 0x%x\n", offset); + return -1; + } + } printf("DONE BLOCK 0x%x\n", offset); return 0; @@ -167,17 +175,29 @@ int write_82802ab(struct flashchip *flash, uint8_t *buf) int page_size = flash->page_size; volatile uint8_t *bios = flash->virtual_memory; - erase_82802ab(flash); - if (*bios != 0xff) { - printf("ERASE FAILED\n"); - return -1; - } - printf("Programming page: "); + printf("Programming page: \n"); for (i = 0; i < total_size / page_size; i++) { + printf + ("\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b"); printf("%04d at address: 0x%08x", i, i * page_size); + + /* Auto Skip Blocks, which already contain the desired data + * Faster, because we only write, what has changed + * More secure, because blocks, which are excluded + * (with the exclude or layout feature) + * or not erased and rewritten; their data is retained also in + * sudden power off situations + */ + if (!memcmp((void *)(buf + i * page_size), + (void *)(bios + i * page_size), page_size)) { + printf("SKIPPED\n"); + continue; + } + + /* erase block by block and write block by block; this is the most secure way */ + erase_82802ab_block(flash, i * page_size); write_page_82802ab(bios, buf + i * page_size, bios + i * page_size, page_size); - printf("\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b"); } printf("\n"); protect_jedec(bios); |