flashrom: Beautify flash chip ID verbose printout a little, always use %02x.

[coreboot.git] / util / flashrom / 82802ab.c

diff --git a/util/flashrom/82802ab.c b/util/flashrom/82802ab.c
index 2784d7ae5932f579865aeac0abb93633aefcd766..5e5b1a59add70c75195c9db93bc98afb99004ca4 100644 (file)
--- a/util/flashrom/82802ab.c
+++ b/util/flashrom/82802ab.c
@@ -1,29 +1,33 @@
 /*
- * 82802ab.c: driver for programming JEDEC standard flash parts
+ * This file is part of the flashrom project.
  *
+ * Copyright (C) 2000 Silicon Integrated System Corporation
  *
- * Copyright 2000 Silicon Integrated System Corporation
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
  *
- *     This program is free software; you can redistribute it and/or modify
- *     it under the terms of the GNU General Public License as published by
- *     the Free Software Foundation; either version 2 of the License, or
- *     (at your option) any later version.
- *
- *     This program is distributed in the hope that it will be useful,
- *     but WITHOUT ANY WARRANTY; without even the implied warranty of
- *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *     GNU General Public License for more details.
- *
- *     You should have received a copy of the GNU General Public License
- *     along with this program; if not, write to the Free Software
- *     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
- *
- *
- * Reference: http://www.intel.com/design/chipsets/datashts/290658.htm
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
  *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ */
+
+/*
+ * Datasheet:
+ *  - Name: Intel 82802AB/82802AC Firmware Hub (FWH)
+ *  - URL: http://www.intel.com/design/chipsets/datashts/290658.htm
+ *  - PDF: http://download.intel.com/design/chipsets/datashts/29065804.pdf
+ *  - Order number: 290658-004
  */
 
 #include <stdio.h>
+#include <string.h>
 #include <stdint.h>
 #include "flash.h"
 
@@ -65,7 +69,7 @@ int probe_82802ab(struct flashchip *flash)
 
        myusec_delay(10);
 
-       printf_debug("%s: id1 0x%x, id2 0x%x\n", __FUNCTION__, id1, id2);
+       printf_debug("%s: id1 0x%02x, id2 0x%02x\n", __FUNCTION__, id1, id2);
 
        if (id1 != flash->manufacture_id || id2 != flash->model_id)
                return 0;
@@ -107,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
@@ -125,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;
@@ -164,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);