diff options
author | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2005-12-18 16:41:10 +0000 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2005-12-18 16:41:10 +0000 |
commit | e3705282c7be0bc88089d877ba904053eb7fa8f3 (patch) | |
tree | a9bf154f3d56d27069c1e1aff5567ba3d139af86 | |
parent | 53e96253a4482072b2b39df1ebc2bd4c5d012581 (diff) | |
download | flashrom-e3705282c7be0bc88089d877ba904053eb7fa8f3.tar.gz flashrom-e3705282c7be0bc88089d877ba904053eb7fa8f3.tar.bz2 flashrom-e3705282c7be0bc88089d877ba904053eb7fa8f3.zip |
Multiple unrelated changes
* make -v switch print debug messages.
* do case insensitive comparison of mainboards, as wished on the
mailinglist
Corresponding to flashrom svn r37 and coreboot v2 svn r2146.
-rw-r--r-- | debug.h | 5 | ||||
-rw-r--r-- | flash_rom.c | 11 | ||||
-rw-r--r-- | layout.c | 10 |
3 files changed, 16 insertions, 10 deletions
@@ -2,6 +2,9 @@ #define __DEBUG_H__ 1 //#define printf_debug(x...) printf(x) -#define printf_debug(x...) { } + +extern int verbose; + +#define printf_debug(x...) { if(verbose) printf(x); } #endif diff --git a/flash_rom.c b/flash_rom.c index 7b38c8a3..fb7d0300 100644 --- a/flash_rom.c +++ b/flash_rom.c @@ -40,6 +40,9 @@ char *chip_to_probe = NULL; +int exclude_start_page, exclude_end_page; +int force=0, verbose=0; + struct flashchip *probe_flash(struct flashchip *flash) { int fd_mem; @@ -86,7 +89,7 @@ struct flashchip *probe_flash(struct flashchip *flash) return NULL; } -int verify_flash(struct flashchip *flash, uint8_t *buf, int verbose) +int verify_flash(struct flashchip *flash, uint8_t *buf) { int i; int total_size = flash->total_size * 1024; @@ -132,9 +135,6 @@ void usage(const char *name) exit(1); } -int exclude_start_page, exclude_end_page; -int force=0; - int main(int argc, char *argv[]) { uint8_t *buf; @@ -147,7 +147,6 @@ int main(int argc, char *argv[]) write_it = 0, erase_it = 0, verify_it = 0; - int verbose = 0; static struct option long_options[]= { { "read", 0, 0, 'r' }, @@ -340,7 +339,7 @@ int main(int argc, char *argv[]) flash->write(flash, buf); if (verify_it) - verify_flash(flash, buf, verbose); + verify_flash(flash, buf); return 0; } @@ -50,14 +50,18 @@ int show_id(uint8_t *bios, int size) return 0; } - printf("LinuxBIOS image size=%d\n", *walk); + printf("LinuxBIOS last image size (not rom size) is %d bytes.\n", *walk); walk--; mainboard_part=strdup((const char *)(bios+size-*walk)); walk--; mainboard_vendor=strdup((const char *)(bios+size-*walk)); printf("MANUFACTURER: %s\n", mainboard_vendor); printf("MAINBOARD ID: %s\n", mainboard_part); - if(lb_vendor && !strcmp(mainboard_vendor, lb_vendor) && - lb_part && !strcmp(mainboard_part, lb_part)) { + + /* These comparisons are case insensitive to make things + * a little less user^Werror prone. + */ + if(lb_vendor && !strcasecmp(mainboard_vendor, lb_vendor) && + lb_part && !strcasecmp(mainboard_part, lb_part)) { printf ("This firmware image matches " "this motherboard.\n"); } else { |