flashrom May 2010

flashrom@flashrom.org

64 participants
169 discussions

[PROPOSAL] support for NIC RTL8139 onboard flashing
by Guenter 18 May '10

18 May '10

Hi all, a lot of Realtek RTL8139 NICs have a DIL-32 bootrom socket, and it is possible from DOS to flash PXE bootcode directly onboard provided you use ATMEL AT29C512 or AT29C010A flashs; here's the download of the Realtek flash program (see DOS section): http://www.realtek.com.tw/downloads/downloadsView.aspx?Langid=1&PNid=6&PFid… unfortunately Realtek's flash utility only supports ATMEL flashs, and even more worse is the trap that you can only flash the chip if it is detected by the BIOS - if you just insert an empty chip then it didnt work for me; the chip needs at least the 55AA code at the beginning so that its memory is mapped into adapter space by the MB BIOS. Do you think that it is possible to add support for flashing these cards (omst likely the RTL8169 GB NICs work same) to flashrom? Günter.

5 6

[commit] r999 - trunk
by repository service 18 May '10

18 May '10

Author: mkarcher Date: Tue May 18 01:19:22 2010 New Revision: 999 URL: http://flashrom.org/trac/coreboot/changeset/999 Log: msr_t memory layout may depend on compiler; with optimizations this may lead to writing incorrect data to MSR. Create a temporary buffer with correct layout to avoid this problem. Signed-off-by: Anti Sullin <anti.sullin(a)artecdesign.ee> Acked-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> Modified: trunk/physmap.c Modified: trunk/physmap.c ============================================================================== --- trunk/physmap.c Sat May 15 17:04:37 2010 (r998) +++ trunk/physmap.c Tue May 18 01:19:22 2010 (r999) @@ -282,13 +282,17 @@ int wrmsr(int addr, msr_t msr) { + uint32_t buf[2]; + buf[0] = msr.lo; + buf[1] = msr.hi; + if (lseek(fd_msr, (off_t) addr, SEEK_SET) == -1) { perror("Could not lseek() to MSR"); close(fd_msr); exit(1); } - if (write(fd_msr, &msr, 8) != 8 && errno != EIO) { + if (write(fd_msr, buf, 8) != 8 && errno != EIO) { perror("Could not write() MSR"); close(fd_msr); exit(1);

1 0

flashrom command line syntax change
by Carl-Daniel Hailfinger 15 May '10

15 May '10

Hi, [please followup to flashrom(a)flashrom.org] since I didn't get any answers except "hmm..." on IRC, I hope that list feedback will be more verbose. In the past, flashrom served one purpose (developers writing LinuxBIOS images or parts thereof to onboard flash), but nowadays it can handle external programmers, graphics/network/storage cards and other hardware as well. Given that the flashrom user base is growing rapidly especially among less technically inclined people, we changed the flashrom command line interface in various ways to eliminate nasty surprises. The flashrom 0.9.2 release is impending (only one small bugfix and one big frontend/documentation patch left), and this is why I'd like comments about the following change: -r and -w and -v currently take a filename argument, but that filename can be anywhere on the command line. Examples: flashrom -p dummy backup.rom -r flashrom -rp dummy backup.rom flashrom -rV backup.rom flashrom -p dummy -r backup.rom This flashrom behaviour is confusing and contradicts the man page which says that -r/-w/-v are followed by a filename. Besides that, all other options with arguments want their arguments directly after the option. I posted a patch which will make the accepted command line syntax conform to the man page, but I'd like to hear about any problems this change may cause. The patch is at http://patchwork.coreboot.org/patch/1298/ (click on "patch" to download it in raw form) and applies cleanly against latest flashrom from svn. Tests of current svn flashrom with the patch applied are really appreciated, and feedback about the interface change will be taken seriously. Regards, Carl-Daniel -- http://www.hailfinger.org/

3 3

[PATCH] Unlocking for W39V040FA
by Michael Karcher 15 May '10

15 May '10

Now added the missing "REGISTERMAP" feature. --- flashchips.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/flashchips.c b/flashchips.c index 10c0989..d51c2bf 100644 --- a/flashchips.c +++ b/flashchips.c @@ -6080,35 +6080,37 @@ struct flashchip flashchips[] = { { .vendor = "Winbond", .name = "W39V040FA", .bustype = CHIP_BUSTYPE_FWH, .manufacture_id = WINBOND_ID, .model_id = W_39V040FA, .total_size = 512, .page_size = 64 * 1024, - .feature_bits = FEATURE_EITHER_RESET, + .feature_bits = FEATURE_REGISTERMAP | FEATURE_EITHER_RESET, .tested = TEST_UNTESTED, .probe = probe_jedec, .probe_timing = 10, .block_erasers = { { .eraseblocks = { {4 * 1024, 128} }, .block_erase = erase_block_jedec, }, { .eraseblocks = { {64 * 1024, 8} }, .block_erase = erase_sector_jedec, }, { .eraseblocks = { {512 * 1024, 1} }, .block_erase = erase_chip_block_jedec, } }, + .printlock = printlock_sst_fwhub, + .unlock = unlock_sst_fwhub, .write = write_jedec_1, .read = read_memmapped, }, { .vendor = "Winbond", .name = "W39V080A", .bustype = CHIP_BUSTYPE_LPC, .manufacture_id = WINBOND_ID, -- 1.7.1

1 0

[PATCH] Unlocking for W39V040FA
by Michael Karcher 15 May '10

15 May '10

Should fix the current issue, probably obsoleted by the pending FWH unification. Signed-off-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> --- flashchips.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/flashchips.c b/flashchips.c index 10c0989..bc96a4a 100644 --- a/flashchips.c +++ b/flashchips.c @@ -6097,18 +6097,20 @@ struct flashchip flashchips[] = { .block_erase = erase_block_jedec, }, { .eraseblocks = { {64 * 1024, 8} }, .block_erase = erase_sector_jedec, }, { .eraseblocks = { {512 * 1024, 1} }, .block_erase = erase_chip_block_jedec, } }, + .printlock = printlock_sst_fwhub, + .unlock = unlock_sst_fwhub, .write = write_jedec_1, .read = read_memmapped, }, { .vendor = "Winbond", .name = "W39V080A", .bustype = CHIP_BUSTYPE_LPC, .manufacture_id = WINBOND_ID, -- 1.7.1

1 0

[PATCH] flashrom 0.9.2
by Carl-Daniel Hailfinger 15 May '10

15 May '10

No, 0.9.2 is not yet out. This patch should be committed in r999 so that the 0.9.2 tag in svn will be r1000. Patches I absolutely want to get in before 0.9.2: AT49F002(N)(T) eraseblock fix (needs re-review due to datasheet contradictions) http://patchwork.coreboot.org/patch/833/ drkaiser: MEM BAR fixes (needs update to fix gfxnvidia MEM BAR as well) http://patchwork.coreboot.org/patch/1077/ msg_* conversions (two patches, preferably as one commit, needs review) http://patchwork.coreboot.org/patch/1167/ http://patchwork.coreboot.org/patch/1166/ Fix delay loop (needs review) http://patchwork.coreboot.org/patch/1172/ This means we can fit 5 additional patches in flashrom to reach the desired commit number. Candidates for such patches (desirable, but not absolutely must-have): flashrom dependencies (needs fixing) http://patchwork.coreboot.org/patch/1131/ Merge print/print_wiki tables (needs review) http://patchwork.coreboot.org/patch/1044/ Move Gigabyte it87spi boards to 'no enable' section (conflicts with 1044) http://patchwork.coreboot.org/patch/1151/ Fix Kontron 986lcd-m (needs cleanup and retesting) http://patchwork.coreboot.org/patch/871/ Add missing Intel/VIA chipset IDs (needs finishing, Idwer made some progress) http://patchwork.coreboot.org/patch/1169/ Suggestions for other patches? If I overlooked any of your patches, I'm sorry. Feel free to point me to them and/or push them for inclusion. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: flashrom-0.9.2/Makefile =================================================================== --- flashrom-0.9.2/Makefile (Revision 989) +++ flashrom-0.9.2/Makefile (Arbeitskopie) @@ -73,7 +73,7 @@ # will not require subversion. The downloadable snapshots are already exported. SVNVERSION := $(shell LC_ALL=C svnversion -cn . | sed -e "s/.*://" -e "s/$[0-9]*$.*/\1/" | grep "[0-9]" || LC_ALL=C svn info . | grep ^Revision | sed "s/.*[[:blank:]]\+$[0-9]*$[^0-9]*/\1/" | grep "[0-9]" || echo unknown) -RELEASE := 0.9.1 +RELEASE := 0.9.2 VERSION := $(RELEASE)-r$(SVNVERSION) RELEASENAME ?= $(VERSION) -- http://www.hailfinger.org/

3 10

[commit] r998 - trunk
by repository service 15 May '10

15 May '10

Author: hailfinger Date: Sat May 15 17:04:37 2010 New Revision: 998 URL: http://flashrom.org/trac/coreboot/changeset/998 Log: Fix assorted documentation, frontend and printing bugs. Change the command line interface to make file names positional. Add more sanity checks to the command line parser. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Acked-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> Modified: trunk/Makefile trunk/cli_classic.c trunk/flash.h trunk/flashrom.8 trunk/flashrom.c trunk/print.c trunk/print_wiki.c Modified: trunk/Makefile ============================================================================== --- trunk/Makefile Fri May 7 22:09:04 2010 (r997) +++ trunk/Makefile Sat May 15 17:04:37 2010 (r998) @@ -71,7 +71,7 @@ # of the checked out flashrom files. # Note to packagers: Any tree exported with "make export" or "make tarball" # will not require subversion. The downloadable snapshots are already exported. -SVNVERSION := $(shell LC_ALL=C svnversion -cn . | sed -e "s/.*://" -e "s/$[0-9]*$.*/\1/" | grep "[0-9]" || LC_ALL=C svn info . | grep ^Revision | sed "s/.*[[:blank:]]\+$[0-9]*$[^0-9]*/\1/" | grep "[0-9]" || echo unknown) +SVNVERSION := $(shell LC_ALL=C svnversion -cn . 2>/dev/null | sed -e "s/.*://" -e "s/$[0-9]*$.*/\1/" | grep "[0-9]" || LC_ALL=C svn info . 2>/dev/null | awk '/^Revision:/ {print $$2 }' | grep "[0-9]" || LC_ALL=C git svn info . 2>/dev/null | awk '/^Revision:/ {print $$2 }' | grep "[0-9]" || echo unknown) RELEASE := 0.9.1 VERSION := $(RELEASE)-r$(SVNVERSION) Modified: trunk/cli_classic.c ============================================================================== --- trunk/cli_classic.c Fri May 7 22:09:04 2010 (r997) +++ trunk/cli_classic.c Sat May 15 17:04:37 2010 (r998) @@ -37,33 +37,53 @@ int remaining = 0; enum programmer p; - printf("Usage: %s [-VfLzhR] [-E|-r file|-w file|-v file] [-c chipname]\n" - " [-m [vendor:]part] [-l file] [-i image] [-p programmer]\n\n", name); - - printf("Please note that the command line interface for flashrom will " - "change before\nflashrom 1.0. Do not use flashrom in scripts " - "or other automated tools without\nchecking that your flashrom" - " version won't interpret options in a different way.\n\n"); - - printf - (" -r | --read: read flash and save into file\n" - " -w | --write: write file into flash\n" - " -v | --verify: verify flash against file\n" - " -n | --noverify: don't verify flash against file\n" - " -E | --erase: erase flash device\n" - " -V | --verbose: more verbose output\n" - " -c | --chip <chipname>: probe only for specified flash chip\n" + printf("Usage: %s [-n] [-V] [-f] [-h|-R|-L|" +#if PRINT_WIKI_SUPPORT == 1 + "-z|" +#endif + "-E|-r <file>|-w <file>|-v <file>]\n" + " [-c <chipname>] [-m [<vendor>:]<part>] [-l <file>]\n" + " [-i <image>] [-p <programmername>[:<parameters>]]\n", + name); + + printf("Please note that the command line interface for flashrom has " + "changed between\n" + "0.9.1 and 0.9.2 and will change again before flashrom 1.0.\n" + "Do not use flashrom in scripts or other automated tools " + "without checking\n" + "that your flashrom version won't interpret options in a " + "different way.\n\n"); + + printf(" -h | --help print this help text\n" + " -R | --version print version (release)\n" + " -r | --read <file> read flash and save to " + "<file>\n" + " -w | --write <file> write <file> to flash\n" + " -v | --verify <file> verify flash against " + "<file>\n" + " -E | --erase erase flash device\n" + " -V | --verbose more verbose output\n" + " -c | --chip <chipname> probe only for specified " + "flash chip\n" #if INTERNAL_SUPPORT == 1 - " -m | --mainboard <[vendor:]part>: override mainboard settings\n" + /* FIXME: --mainboard should be a programmer parameter */ + " -m | --mainboard <[vendor:]part> override mainboard " + "detection\n" #endif - " -f | --force: force write without checking image\n" - " -l | --layout <file.layout>: read ROM layout from file\n" - " -i | --image <name>: only flash image name from flash layout\n" - " -L | --list-supported: print supported devices\n" + " -f | --force force specific operations " + "(see man page)\n" + " -n | --noverify don't auto-verify\n" + " -l | --layout <file> read ROM layout from " + "<file>\n" + " -i | --image <name> only flash image <name> " + "from flash layout\n" + " -L | --list-supported print supported devices\n" #if PRINT_WIKI_SUPPORT == 1 - " -z | --list-supported-wiki: print supported devices in wiki syntax\n" + " -z | --list-supported-wiki print supported devices " + "in wiki syntax\n" #endif - " -p | --programmer <name>: specify the programmer device"); + " -p | --programmer <name>[:<param>] specify the programmer " + "device"); for (p = 0; p < PROGRAMMER_INVALID; p++) { pname = programmer_table[p].name; @@ -88,12 +108,19 @@ printf(")\n"); } } - - printf( - " -h | --help: print this help text\n" - " -R | --version: print the version (release)\n" - "\nYou can specify one of -E, -r, -w, -v or no operation. If no operation is\n" - "specified, then all that happens is that flash info is dumped.\n\n"); + + printf("\nYou can specify one of -h, -R, -L, " +#if PRINT_WIKI_SUPPORT == 1 + "-z, " +#endif + "-E, -r, -w, -v or no operation.\n" + "If no operation is specified, flashrom will only probe for " + "flash chips.\n\n"); +} + +void cli_classic_abort_usage(const char *name) +{ + printf("Please run \"%s --help\" for usage info.\n", name); exit(1); } @@ -115,16 +142,12 @@ int operation_specified = 0; int i; -#if PRINT_WIKI_SUPPORT == 1 - const char *optstring = "rRwvnVEfc:m:l:i:p:Lzh"; -#else - const char *optstring = "rRwvnVEfc:m:l:i:p:Lh"; -#endif + const char *optstring = "r:Rw:v:nVEfc:m:l:i:p:Lzh"; static struct option long_options[] = { - {"read", 0, 0, 'r'}, - {"write", 0, 0, 'w'}, + {"read", 1, 0, 'r'}, + {"write", 1, 0, 'w'}, {"erase", 0, 0, 'E'}, - {"verify", 0, 0, 'v'}, + {"verify", 1, 0, 'v'}, {"noverify", 0, 0, 'n'}, {"chip", 1, 0, 'c'}, {"mainboard", 1, 0, 'm'}, @@ -133,9 +156,7 @@ {"layout", 1, 0, 'l'}, {"image", 1, 0, 'i'}, {"list-supported", 0, 0, 'L'}, -#if PRINT_WIKI_SUPPORT == 1 {"list-supported-wiki", 0, 0, 'z'}, -#endif {"programmer", 1, 0, 'p'}, {"help", 0, 0, 'h'}, {"version", 0, 0, 'R'}, @@ -147,18 +168,15 @@ char *tempstr = NULL; print_version(); - - if (argc > 1) { - /* Yes, print them. */ - printf_debug("The arguments are:\n"); - for (i = 1; i < argc; ++i) - printf_debug("%s\n", argv[i]); - } + print_banner(); if (selfcheck()) exit(1); setbuf(stdout, NULL); + /* FIXME: Delay all operation_specified checks until after command + * line parsing to allow --help overriding everything else. + */ while ((opt = getopt_long(argc, argv, optstring, long_options, &option_index)) != EOF) { switch (opt) { @@ -166,16 +184,18 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } + filename = strdup(optarg); read_it = 1; break; case 'w': if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } + filename = strdup(optarg); write_it = 1; break; case 'v': @@ -183,20 +203,21 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } if (dont_verify_it) { fprintf(stderr, "--verify and --noverify are" "mutually exclusive. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } + filename = strdup(optarg); verify_it = 1; break; case 'n': if (verify_it) { fprintf(stderr, "--verify and --noverify are" "mutually exclusive. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } dont_verify_it = 1; break; @@ -210,36 +231,55 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - exit(1); + cli_classic_abort_usage(argv[0]); } erase_it = 1; break; -#if INTERNAL_SUPPORT == 1 case 'm': +#if INTERNAL_SUPPORT == 1 tempstr = strdup(optarg); lb_vendor_dev_from_string(tempstr); - break; +#else + fprintf(stderr, "Error: Internal programmer support " + "was not compiled in and --mainboard only\n" + "applies to the internal programmer. Aborting.\n"); + cli_classic_abort_usage(argv[0]); #endif + break; case 'f': force = 1; break; case 'l': tempstr = strdup(optarg); if (read_romlayout(tempstr)) - exit(1); + cli_classic_abort_usage(argv[0]); break; case 'i': tempstr = strdup(optarg); find_romentry(tempstr); break; case 'L': + if (++operation_specified > 1) { + fprintf(stderr, "More than one operation " + "specified. Aborting.\n"); + cli_classic_abort_usage(argv[0]); + } list_supported = 1; break; -#if PRINT_WIKI_SUPPORT == 1 case 'z': +#if PRINT_WIKI_SUPPORT == 1 + if (++operation_specified > 1) { + fprintf(stderr, "More than one operation " + "specified. Aborting.\n"); + cli_classic_abort_usage(argv[0]); + } list_supported_wiki = 1; - break; +#else + fprintf(stderr, "Error: Wiki output was not compiled " + "in. Aborting.\n"); + cli_classic_abort_usage(argv[0]); #endif + break; case 'p': for (programmer = 0; programmer < PROGRAMMER_INVALID; programmer++) { name = programmer_table[programmer].name; @@ -267,21 +307,37 @@ } } if (programmer == PROGRAMMER_INVALID) { - printf("Error: Unknown programmer %s.\n", optarg); - exit(1); + fprintf(stderr, "Error: Unknown programmer " + "%s.\n", optarg); + cli_classic_abort_usage(argv[0]); } break; case 'R': /* print_version() is always called during startup. */ + if (++operation_specified > 1) { + fprintf(stderr, "More than one operation " + "specified. Aborting.\n"); + cli_classic_abort_usage(argv[0]); + } exit(0); break; case 'h': - default: + if (++operation_specified > 1) { + fprintf(stderr, "More than one operation " + "specified. Aborting.\n"); + cli_classic_abort_usage(argv[0]); + } cli_classic_usage(argv[0]); + exit(0); + break; + default: + cli_classic_abort_usage(argv[0]); break; } } + /* FIXME: Print the actions flashrom will take. */ + if (list_supported) { print_supported(); exit(0); @@ -294,18 +350,18 @@ } #endif - if (read_it && write_it) { - printf("Error: -r and -w are mutually exclusive.\n"); - cli_classic_usage(argv[0]); + if (optind < argc) { + fprintf(stderr, "Error: Extra parameter found.\n"); + cli_classic_abort_usage(argv[0]); } - if (optind < argc) - filename = argv[optind++]; - - if (optind < argc) { - printf("Error: Extra parameter found.\n"); - cli_classic_usage(argv[0]); +#if INTERNAL_SUPPORT == 1 + if ((programmer != PROGRAMMER_INTERNAL) && (lb_part || lb_vendor)) { + fprintf(stderr, "Error: --mainboard requires the internal " + "programmer. Aborting.\n"); + cli_classic_abort_usage(argv[0]); } +#endif if (chip_to_probe) { for (flash = flashchips; flash && flash->name; flash++) @@ -321,13 +377,15 @@ /* Clean up after the check. */ flash = NULL; } - + + msg_pdbg("Initializing %s programmer\n", + programmer_table[programmer].name); if (programmer_init()) { fprintf(stderr, "Error: Programmer initialization failed.\n"); exit(1); } - // FIXME: Delay calibration should happen in programmer code. + /* FIXME: Delay calibration should happen in programmer code. */ myusec_calibrate_delay(); for (i = 0; i < ARRAY_SIZE(flashes); i++) { @@ -383,7 +441,7 @@ printf("No operations were specified.\n"); // FIXME: flash writes stay enabled! programmer_shutdown(); - exit(1); + exit(0); } if (!filename && !erase_it) { Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Fri May 7 22:09:04 2010 (r997) +++ trunk/flash.h Sat May 15 17:04:37 2010 (r998) @@ -548,6 +548,7 @@ int need_erase(uint8_t *have, uint8_t *want, int len, enum write_granularity gran); char *strcat_realloc(char *dest, const char *src); void print_version(void); +void print_banner(void); int selfcheck(void); int doit(struct flashchip *flash, int force, char *filename, int read_it, int write_it, int erase_it, int verify_it); Modified: trunk/flashrom.8 ============================================================================== --- trunk/flashrom.8 Fri May 7 22:09:04 2010 (r997) +++ trunk/flashrom.8 Sat May 15 17:04:37 2010 (r998) @@ -1,9 +1,13 @@ -.TH FLASHROM 8 "May 21, 2009" +.TH FLASHROM 8 "Apr 29, 2010" .SH NAME flashrom \- detect, read, write, verify and erase flash chips .SH SYNOPSIS -.B flashrom \fR[\fB\-VfLzhRn\fR] [\fB\-E\fR|\fB\-r\fR file|\fB\-w\fR file|\fB\-v\fR file] [\fB\-c\fR chipname] - [\fB\-m\fR [vendor:]part] [\fB\-l\fR file] [\fB\-i\fR image] [\fB\-p\fR programmer] +.B flashrom \fR[\fB\-n\fR] [\fB\-V\fR] [\fB\-f\fR] [\fB\-h\fR|\fB\-R\fR|\ +\fB\-L\fR|\fB\-z\fR|\fB\-E\fR|\fB\-r\fR <file>|\fB\-w\fR <file>|\ +\fB\-v\fR <file>] + [\fB\-c\fR <chipname>] [\fB\-m\fR [<vendor>:]<part>] \ +[\fB\-l\fR <file>] + [\fB\-i\fR <image>] [\fB\-p\fR <programmername>[:<parameters>]] .SH DESCRIPTION .B flashrom is a utility for detecting, reading, writing, verifying and erasing flash @@ -20,9 +24,8 @@ flashrom 1.0. Do not use flashrom in scripts or other automated tools without checking that your flashrom version won't interpret options in a different way. .PP -You can specify one of \-E, \-r, \-w, \-v or no operation. -If no operation is specified, then all that happens -is that flash info is dumped and the flash chip is set to writable. It is +You can specify one of \-h, \-R, \-L, \-z, \-E, \-r, \-w, \-v or no operation. +If no operation is specified, flashrom will only probe for flash chips. It is recommended that if you try flashrom the first time on a system, you run it in probe only mode and check the output. Also you are advised to make a backup of your current ROM contents with \-r before you try to write a new @@ -33,19 +36,19 @@ .BR <file> . .TP .B "\-w, \-\-write <file>" -Write file into flash ROM. +Write +.B <file> +into flash ROM. .TP .B "\-n, \-\-noverify" -Do -.B not -verify the flash ROM contents after writing them to the chip. Using this +Skip the automatic verification of flash ROM contents after writing. Using this option is .B not -recommended, you should only use it if you know what you are doing and you +recommended, you should only use it if you know what you are doing and if you feel that the time for verification takes too long. .sp Typical usage is: -.B "flashrom -wn file" +.B "flashrom -n -w file" .sp This option is only useful in combination with .BR \-\-write . @@ -66,14 +69,14 @@ .B "flashrom \-L" without the vendor name. Please note that the chip name is case sensitive. .TP -.B "\-m, \-\-mainboard" <[vendor:]part> +.B "\-m, \-\-mainboard" [<vendor>:]<part> Override mainboard settings. .sp flashrom reads the coreboot table to determine the current mainboard. If no coreboot table could be read or if you want to override these values, you can specify \-m, e.g.: .sp -.B " flashrom -w --mainboard AGAMI:ARUMA agami_aruma.rom" +.B " flashrom --mainboard AGAMI:ARUMA -w agami_aruma.rom" .sp See the 'Supported mainboards' section in the output of 'flashrom \-L' for a list of boards which require the specification of the board name, if no @@ -108,19 +111,15 @@ All addresses are offsets within the file, not absolute addresses! If you only want to update the normal image in a ROM you can say: .sp -.B " flashrom -w --layout rom.layout --image normal agami_aruma.rom" +.B " flashrom --layout rom.layout --image normal -w agami_aruma.rom" .sp To update normal and fallback but leave the VGA BIOS alone, say: .sp -.B " flashrom -w -l rom.layout -i normal \" +.B " flashrom -l rom.layout -i normal \" .br -.B " -i fallback agami_aruma.rom" +.B " -i fallback -w agami_aruma.rom" .sp Currently overlapping sections are not supported. -.sp -ROM layouts should replace the \-s and \-e option since they are more -flexible and they should lead to a ROM update file format with the -ROM layout and the ROM image in one file (cpio, zip or something?). .TP .B "\-i, \-\-image <name>" Only flash image @@ -141,7 +140,8 @@ Same as .BR \-\-list\-supported , but outputs the supported hardware in MediaWiki syntax, so that it can be -easily pasted into the wiki page at http://www.flashrom.org/. +easily pasted into the wiki page at http://www.flashrom.org/. Please note +that MediaWiki output is not compiled in by default. .TP .B "\-p, \-\-programmer <name>[:parameter[,parameter[,parameter]]]" Specify the programmer device. Currently supported are: @@ -237,6 +237,7 @@ .sp If your mainboard uses an ITE IT87 series Super I/O for LPC<->SPI flash bus translation, flashrom should autodetect that configuration. You can use +.sp .B "flashrom -p internal:it87spiport=portnum" syntax as explained in the .B it87spi @@ -382,6 +383,8 @@ .br Markus Boas <ryven(a)ryven.de> .br +Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> +.br Nikolay Petukhov <nikolay.petukhov(a)gmail.com> .br Peter Stuge <peter(a)stuge.se> @@ -392,6 +395,8 @@ .br Ronald Hoogenboom <ronald(a)zonnet.nl> .br +Sean Nelson <audiohacked(a)gmail.com> +.br Stefan Reinauer <stepan(a)coresystems.de> .br Stefan Wildemann <stefan.wildemann(a)kontron.com> @@ -406,5 +411,6 @@ .br some others .PP -This manual page was written by Uwe Hermann <uwe(a)hermann-uwe.de>. +This manual page was written by Uwe Hermann <uwe(a)hermann-uwe.de> and Carl-Daniel +Hailfinger. It is licensed under the terms of the GNU GPL (version 2 or later). Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Fri May 7 22:09:04 2010 (r997) +++ trunk/flashrom.c Sat May 15 17:04:37 2010 (r998) @@ -1130,8 +1130,9 @@ { msg_gerr("Your flash chip is in an unknown state.\n" "Get help on IRC at irc.freenode.net (channel #flashrom) or\n" - "mail flashrom(a)flashrom.org!\n--------------------" - "-----------------------------------------------------------\n" + "mail flashrom(a)flashrom.org!\n" + "-------------------------------------------------------------" + "------------------\n" "DO NOT REBOOT OR POWEROFF!\n"); } @@ -1183,10 +1184,17 @@ void print_version(void) { - msg_ginfo("flashrom v%s\n", flashrom_version); + msg_ginfo("flashrom v%s", flashrom_version); print_sysinfo(); } +void print_banner(void) +{ + msg_ginfo("flashrom is free software, get the source code at " + "http://www.flashrom.org\n"); + msg_ginfo("\n"); +} + int selfcheck(void) { int ret = 0; @@ -1247,13 +1255,20 @@ msg_cinfo("\n"); } /* FIXME: This message is designed towards CLI users. */ - msg_cinfo("Please email a report to flashrom(a)flashrom.org if any " - "of the above operations\nwork correctly for you with " - "this flash part. Please include the flashrom\noutput " - "with the additional -V option for all operations you " - "tested (-V, -rV,\n-wV, -EV), and mention which " - "mainboard or programmer you tested.\nThanks for your " - "help!\n===\n"); + msg_cinfo("The test status of this chip may have been updated " + "in the latest development\n" + "version of flashrom. If you are running the latest " + "development version,\n" + "please email a report to flashrom(a)flashrom.org if " + "any of the above operations\n" + "work correctly for you with this flash part. Please " + "include the flashrom\n" + "output with the additional -V option for all " + "operations you tested (-V, -Vr,\n" + "-Vw, -VE), and mention which mainboard or " + "programmer you tested.\n" + "Thanks for your help!\n" + "===\n"); } } Modified: trunk/print.c ============================================================================== --- trunk/print.c Fri May 7 22:09:04 2010 (r997) +++ trunk/print.c Sat May 15 17:04:37 2010 (r998) @@ -247,6 +247,9 @@ #if SATASII_SUPPORT == 1 print_supported_pcidevs(satas_sii); #endif +#if ATAHPT_SUPPORT == 1 + print_supported_pcidevs(ata_hpt); +#endif } Modified: trunk/print_wiki.c ============================================================================== --- trunk/print_wiki.c Fri May 7 22:09:04 2010 (r997) +++ trunk/print_wiki.c Sat May 15 17:04:37 2010 (r998) @@ -25,6 +25,7 @@ #include "flash.h" #include "flashchips.h" +#if INTERNAL_SUPPORT == 1 struct board_info_url { const char *vendor; const char *name; @@ -36,6 +37,7 @@ const char *name; const char *note; }; +#endif const char *wiki_header = "= Supported devices =\n\n\ <div style=\"margin-top:0.5em; padding:0.5em 0.5em 0.5em 0.5em; \ @@ -44,6 +46,7 @@ generated by pasting '''flashrom -z''' output.<br />\ '''Last update:''' %s(generated by flashrom %s)\n</small></div>\n"; +#if INTERNAL_SUPPORT == 1 const char *chipset_th = "{| border=\"0\" style=\"font-size: smaller\"\n\ |- bgcolor=\"#6699dd\"\n! align=\"left\" | Vendor\n\ ! align=\"left\" | Southbridge\n! align=\"left\" | PCI IDs\n\ @@ -69,6 +72,7 @@ Mainboards which don't appear in the list may or may not work (we don't \ know, someone has to give it a try). Please report any further verified \ mainboards on the [[Mailinglist|mailing list]].\n"; +#endif const char *chip_th = "{| border=\"0\" style=\"font-size: smaller\" \ valign=\"top\"\n|- bgcolor=\"#6699dd\"\n! align=\"left\" | Vendor\n\ @@ -85,6 +89,7 @@ ! align=\"left\" | Device\n! align=\"left\" | PCI IDs\n\ ! align=\"left\" | Status\n\n"; +#if INTERNAL_SUPPORT == 1 const char *laptop_intro = "\n== Supported laptops/notebooks ==\n\n\ In general, flashing laptops is more difficult because laptops\n\n\ * often use the flash chip for stuff besides the BIOS,\n\ @@ -505,6 +510,7 @@ wiki_helper("Known good (worked out of the box)", "OK", 1, laptops_ok); wiki_helper("Not supported (yet)", "No", 1, laptops_bad); } +#endif void print_supported_chips_wiki(void) { @@ -576,9 +582,11 @@ time_t t = time(NULL); printf(wiki_header, ctime(&t), flashrom_version); +#if INTERNAL_SUPPORT == 1 print_supported_chips_wiki(); print_supported_chipsets_wiki(); print_supported_boards_wiki(); +#endif printf("%s", programmer_section); #if NIC3COM_SUPPORT == 1 print_supported_pcidevs_wiki(nics_3com);

1 0

[patch] unhide the flash chip on QDI KD7G
by Idwer Vollering 15 May '10

15 May '10

Only works with "Flash Write Protect" set to "Disabled", in the BIOS menu. With the attached patch write/erase work without the need of extra code. Signed-off-by: Idwer Vollering <vidwer(a)gmail.com>

3 2

[PATCH] Don't ignore wait_82802ab() return value
by Stefan Reinauer 14 May '10

14 May '10

See patch. -- coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br. Tel.: +49 761 7668825 • Fax: +49 761 7664613 Email: info(a)coresystems.de • http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg • HRB 7656 Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

3 2

[Patch] RFC: Set status register option
by David Hendricks 14 May '10

14 May '10

Hello all, I'd like to know what people's thoughts are on setting the status register for SPI flash memories using Flashrom. It seems like a good fit for Flashrom and would go a long way in helping OEMs/ODMs migrate away from crappy DOS utilities to using Flashrom for mass production. I've attached a very lame patch that adds a command-line option (-s or --set-status) to do this for SPI chips. -- David Hendricks (dhendrix) Systems Software Engineer, Google Inc.

2 3

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom May 2010