- flashrom - mail.coreboot.org

Bug report - Thinkpad X60s
by Ted 05 Sep '14

05 Sep '14

x60s@x60s-ThinkPad-X60s:~/Downloads/libreboot_bin/flashrom$ sudo ./flashrom_lenovobios_sst -p internal -r factory.bin flashrom v0.9.7-unknown on Linux 3.2.0-67-generic (i686) flashrom is free software, get the source code at http://www.flashrom.org Calibrating delay loop... delay loop is unreliable, trying to continue OK. Found chipset "Intel ICH7M". Enabling flash write... WARNING: SPI Configuration Lockdown activated. OK. Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org No EEPROM/flash device found. Note: flashrom can never write if the flash chip isn't found automatically. x60s@x60s-ThinkPad-X60s:~/Downloads/libreboot_bin/flashrom$ sudo ./flashrom_lenovobios_macronix -p internal -r factory.bin flashrom v0.9.7-unknown on Linux 3.2.0-67-generic (i686) flashrom is free software, get the source code at http://www.flashrom.org Calibrating delay loop... delay loop is unreliable, trying to continue OK. Found chipset "Intel ICH7M". Enabling flash write... WARNING: SPI Configuration Lockdown activated. OK. Found Macronix flash chip "MX25L1605D/MX25L1608D/MX25L1673E" (2048 kB, SPI) at physical address 0xffe00000. Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Trying to unmap a nonexisting mapping! Please report a bug at flashrom(a)flashrom.org Reading flash... done. x60s@x60s-ThinkPad-X60s:~/Downloads/libreboot_bin/flashrom$

2 1

[commit] r1849 - trunk
by repository service 05 Sep '14

05 Sep '14

Author: stefanct Date: Fri Sep 5 18:14:11 2014 New Revision: 1849 URL: http://flashrom.org/trac/flashrom/changeset/1849 Log: dmi.c: Duplicate returned strings because they are meant to be freed. Without this patch dmi_shutdown calls free() on read-only strings. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Modified: trunk/dmi.c Modified: trunk/dmi.c ============================================================================== --- trunk/dmi.c Sun Aug 31 02:09:21 2014 (r1848) +++ trunk/dmi.c Fri Sep 5 18:14:11 2014 (r1849) @@ -120,18 +120,18 @@ size_t i, len; if (string_id == 0) - return "Not Specified"; + return strdup("Not Specified"); while (string_id > 1 && string_id--) { if (buf >= limit) { msg_perr("DMI table is broken (string portion out of bounds)!\n"); - return "<OUT OF BOUNDS>"; + return strdup("<OUT OF BOUNDS>"); } buf += strnlen(buf, limit - buf) + 1; } if (!*buf) /* as long as the current byte we're on isn't null */ - return "<BAD INDEX>"; + return strdup("<BAD INDEX>"); len = strnlen(buf, limit - buf); char *newbuf = malloc(len + 1);

1 0

Question about flashrom
by false 03 Sep '14

03 Sep '14

To whom it may concern : Hello, I read some command code about using the software but I didn't understand clearly how I can use it to read/write memory chips through FT2232D SPI port! Would you tell me how I can use it? or send an useful manual ... Thanks a lot...

1 0

[PATCH] Add support for AMD Bolton chipset
by Martin Roth 31 Aug '14

31 Aug '14

Add support for AMD's Bolton chipset. The SPI controller on the bolton chipset uses the same 3-bit speed settings as yangtze, but is otherwise the same as the hudson chips. Note that the Bolton RRG doesn't specify a speed setting for the bit setting of 0b111, so I'm assuming that it's the same setting as yangtze. Signed-off-by: Martin Roth <martin.roth(a)se-eng.com>

3 4

[commit] r1848 - trunk
by repository service 30 Aug '14

30 Aug '14

Author: stefanct Date: Sun Aug 31 02:09:21 2014 New Revision: 1848 URL: http://flashrom.org/trac/flashrom/changeset/1848 Log: Introduce generic shutdown_free() and remove redundant internal_shutdown(). The former will be useful in cases where cleanup equals a simple call to free(). Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Modified: trunk/flash.h trunk/flashrom.c trunk/internal.c Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Sun Aug 31 01:39:51 2014 (r1847) +++ trunk/flash.h Sun Aug 31 02:09:21 2014 (r1848) @@ -54,6 +54,7 @@ #define PRIuCHIPSIZE PRIu32 int register_shutdown(int (*function) (void *data), void *data); +int shutdown_free(void *data); void *programmer_map_flash_region(const char *descr, uintptr_t phys_addr, size_t len); void programmer_unmap_flash_region(void *virt_addr, size_t len); void programmer_delay(unsigned int usecs); Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Sun Aug 31 01:39:51 2014 (r1847) +++ trunk/flashrom.c Sun Aug 31 02:09:21 2014 (r1848) @@ -373,6 +373,12 @@ static int check_block_eraser(const struct flashctx *flash, int k, int log); +int shutdown_free(void *data) +{ + free(data); + return 0; +} + /* Register a function to be executed on programmer shutdown. * The advantage over atexit() is that you can supply a void pointer which will * be used as parameter to the registered function upon programmer shutdown. Modified: trunk/internal.c ============================================================================== --- trunk/internal.c Sun Aug 31 01:39:51 2014 (r1847) +++ trunk/internal.c Sun Aug 31 02:09:21 2014 (r1848) @@ -158,11 +158,6 @@ enum chipbustype internal_buses_supported = BUS_NONE; -static int internal_shutdown(void *data) -{ - return 0; -} - int internal_init(void) { #if __FLASHROM_LITTLE_ENDIAN__ @@ -237,8 +232,6 @@ if (rget_io_perms()) return 1; - if (register_shutdown(internal_shutdown, NULL)) - return 1; /* Default to Parallel/LPC/FWH flash devices. If a known host controller * is found, the host controller init routine sets the

1 0

Re: [flashrom] [PATCH] Refine physical address mapping of flash chips.
by Stefan Tauner 30 Aug '14

30 Aug '14

On Wed, 20 Aug 2014 01:15:43 +0200 Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> wrote: > + msg_pdbg2("Successfully mapped flash chip \"%s\" at physical address 0x%0*" PRIxPTR ".\n", > + fill_flash->chip->name, PRIxPTR_WIDTH, fill_flash->physical_memory); > + Forgot to remove these lines, please ignore. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

2 2

[commit] r1847 - trunk
by repository service 30 Aug '14

30 Aug '14

Author: stefanct Date: Sun Aug 31 01:39:51 2014 New Revision: 1847 URL: http://flashrom.org/trac/flashrom/changeset/1847 Log: Refine physical address mapping of flash chips. - Create distinct functions for mapping and unmapping for flash chips. - Map only when needed: map before probing and unmap immediately after it. Map again when a single chip was probed successfully before taking any actual actions and clean up afterwards. - Map special function chip registers centrally together with flash space instead of within (some) probing methods after successful probes. - Save the used base addresses of the mappings in struct flashctx as well. - Do not try to (un)map the zero-sized chip definitions that are merely hacks. This also fixes the printing of wrong warnings for these chip definitions introduced in r1765. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Modified: trunk/82802ab.c trunk/cli_classic.c trunk/flash.h trunk/flashrom.c trunk/jedec.c Modified: trunk/82802ab.c ============================================================================== --- trunk/82802ab.c Wed Aug 20 20:56:35 2014 (r1846) +++ trunk/82802ab.c Sun Aug 31 01:39:51 2014 (r1847) @@ -83,9 +83,6 @@ if (id1 != flash->chip->manufacture_id || id2 != flash->chip->model_id) return 0; - if (flash->chip->feature_bits & FEATURE_REGISTERMAP) - map_flash_registers(flash); - return 1; } Modified: trunk/cli_classic.c ============================================================================== --- trunk/cli_classic.c Wed Aug 20 20:56:35 2014 (r1846) +++ trunk/cli_classic.c Sun Aug 31 01:39:51 2014 (r1847) @@ -481,8 +481,14 @@ ret = 1; goto out_shutdown; } + if (map_flash(&flashes[0]) != 0) { + free(flashes[0].chip); + ret = 1; + goto out_shutdown; + } msg_cinfo("Please note that forced reads most likely contain garbage.\n"); ret = read_flash_to_file(&flashes[0], filename); + unmap_flash(&flashes[0]); free(flashes[0].chip); goto out_shutdown; } @@ -525,6 +531,12 @@ if (write_it && !dont_verify_it) verify_it = 1; + /* Map the selected flash chip again. */ + if (map_flash(fill_flash) != 0) { + ret = 1; + goto out_shutdown; + } + /* FIXME: We should issue an unconditional chip reset here. This can be * done once we have a .reset function in struct flashchip. * Give the chip time to settle. @@ -532,6 +544,7 @@ programmer_delay(100000); ret |= doit(fill_flash, force, filename, read_it, write_it, erase_it, verify_it); + unmap_flash(fill_flash); out_shutdown: programmer_shutdown(); out: Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Wed Aug 20 20:56:35 2014 (r1846) +++ trunk/flash.h Sun Aug 31 01:39:51 2014 (r1847) @@ -48,9 +48,9 @@ /* Types and macros regarding the maximum flash space size supported by generic code. */ typedef uint32_t chipoff_t; /* Able to store any addressable offset within a supported flash memory. */ typedef uint32_t chipsize_t; /* Able to store the number of bytes of any supported flash memory. */ -#define FL_MAX_CHIPADDR_BITS (24) -#define FL_MAX_CHIPADDR ((chipoff_t)(1ULL<<FL_MAX_CHIPADDR_BITS)-1) -#define PRIxCHIPADDR "06"PRIx32 +#define FL_MAX_CHIPOFF_BITS (24) +#define FL_MAX_CHIPOFF ((chipoff_t)(1ULL<<FL_MAX_CHIPOFF_BITS)-1) +#define PRIxCHIPOFF "06"PRIx32 #define PRIuCHIPSIZE PRIu32 int register_shutdown(int (*function) (void *data), void *data); @@ -209,8 +209,14 @@ struct flashctx { struct flashchip *chip; + /* FIXME: The memory mappings should be saved in a more structured way. */ + /* The physical_* fields store the respective addresses in the physical address space of the CPU. */ + uintptr_t physical_memory; + /* The virtual_* fields store where the respective physical address is mapped into flashrom's address + * space. A value equivalent to (chipaddr)ERROR_PTR indicates an invalid mapping (or none at all). */ chipaddr virtual_memory; - /* Some flash devices have an additional register space. */ + /* Some flash devices have an additional register space; semantics are like above. */ + uintptr_t physical_registers; chipaddr virtual_registers; struct registered_master *mst; }; @@ -252,7 +258,8 @@ /* flashrom.c */ extern const char flashrom_version[]; extern const char *chip_to_probe; -void map_flash_registers(struct flashctx *flash); +int map_flash(struct flashctx *flash); +void unmap_flash(struct flashctx *flash); int read_memmapped(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len); int erase_flash(struct flashctx *flash); int probe_flash(struct registered_master *mst, int startchip, struct flashctx *fill_flash, int force); Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Wed Aug 20 20:56:35 2014 (r1846) +++ trunk/flashrom.c Sun Aug 31 01:39:51 2014 (r1847) @@ -479,6 +479,7 @@ void programmer_unmap_flash_region(void *virt_addr, size_t len) { programmer_table[programmer].unmap_flash_region(virt_addr, len); + msg_gspew("%s: unmapped 0x%0*" PRIxPTR "\n", __func__, PRIxPTR_WIDTH, (uintptr_t)virt_addr); } void chip_writeb(const struct flashctx *flash, uint8_t val, chipaddr addr) @@ -528,14 +529,6 @@ programmer_table[programmer].delay(usecs); } -void map_flash_registers(struct flashctx *flash) -{ - size_t size = flash->chip->total_size * 1024; - /* Flash registers live 4 MByte below the flash. */ - /* FIXME: This is incorrect for nonstandard flashbase. */ - flash->virtual_registers = (chipaddr)programmer_map_flash_region("flash chip registers", (0xFFFFFFFF - 0x400000 - size + 1), size); -} - int read_memmapped(struct flashctx *flash, uint8_t *buf, unsigned int start, int unsigned len) { @@ -1052,12 +1045,64 @@ return limitexceeded; } +void unmap_flash(struct flashctx *flash) +{ + if (flash->virtual_registers != (chipaddr)ERROR_PTR) { + programmer_unmap_flash_region((void *)flash->virtual_registers, flash->chip->total_size * 1024); + flash->physical_registers = 0; + flash->virtual_registers = (chipaddr)ERROR_PTR; + } + + if (flash->virtual_memory != (chipaddr)ERROR_PTR) { + programmer_unmap_flash_region((void *)flash->virtual_memory, flash->chip->total_size * 1024); + flash->physical_memory = 0; + flash->virtual_memory = (chipaddr)ERROR_PTR; + } +} + +int map_flash(struct flashctx *flash) +{ + /* Init pointers to the fail-safe state to distinguish them later from legit values. */ + flash->virtual_memory = (chipaddr)ERROR_PTR; + flash->virtual_registers = (chipaddr)ERROR_PTR; + + /* FIXME: This avoids mapping (and unmapping) of flash chip definitions with size 0. + * These are used for various probing-related hacks that would not map successfully anyway and should be + * removed ASAP. */ + if (flash->chip->total_size == 0) + return 0; + + const chipsize_t size = flash->chip->total_size * 1024; + uintptr_t base = flashbase ? flashbase : (0xffffffff - size + 1); + void *addr = programmer_map_flash_region(flash->chip->name, base, size); + if (addr == ERROR_PTR) { + msg_perr("Could not map flash chip %s at 0x%0*" PRIxPTR ".\n", + flash->chip->name, PRIxPTR_WIDTH, base); + return 1; + } + flash->physical_memory = base; + flash->virtual_memory = (chipaddr)addr; + + /* FIXME: Special function registers normally live 4 MByte below flash space, but it might be somewhere + * completely different on some chips and programmers, or not mappable at all. + * Ignore these problems for now and always report success. */ + if (flash->chip->feature_bits & FEATURE_REGISTERMAP) { + base = 0xffffffff - size - 0x400000 + 1; + addr = programmer_map_flash_region("flash chip registers", base, size); + if (addr == ERROR_PTR) { + msg_pdbg2("Could not map flash chip registers %s at 0x%0*" PRIxPTR ".\n", + flash->chip->name, PRIxPTR_WIDTH, base); + return 0; + } + flash->physical_registers = base; + flash->virtual_registers = (chipaddr)addr; + } + return 0; +} + int probe_flash(struct registered_master *mst, int startchip, struct flashctx *flash, int force) { const struct flashchip *chip; - unsigned long base = 0; - char location[64]; - uint32_t size; enum chipbustype buses_common; char *tmp; @@ -1082,9 +1127,8 @@ memcpy(flash->chip, chip, sizeof(struct flashchip)); flash->mst = mst; - size = flash->chip->total_size * 1024; - base = flashbase ? flashbase : (0xffffffff - size + 1); - flash->virtual_memory = (chipaddr)programmer_map_flash_region("flash chip", base, size); + if (map_flash(flash) != 0) + return -1; /* We handle a forced match like a real match, we just avoid probing. Note that probe_flash() * is only called with force=1 after normal probing failed. @@ -1133,8 +1177,7 @@ break; /* Not the first flash chip detected on this bus, and it's just a generic match. Ignore it. */ notfound: - programmer_unmap_flash_region((void *)flash->virtual_memory, size); - flash->virtual_memory = (chipaddr)NULL; + unmap_flash(flash); free(flash->chip); flash->chip = NULL; } @@ -1142,25 +1185,28 @@ if (!flash->chip) return -1; + + tmp = flashbuses_to_text(flash->chip->bustype); + msg_cinfo("%s %s flash chip \"%s\" (%d kB, %s) ", force ? "Assuming" : "Found", + flash->chip->vendor, flash->chip->name, flash->chip->total_size, tmp); + free(tmp); #if CONFIG_INTERNAL == 1 if (programmer_table[programmer].map_flash_region == physmap) - snprintf(location, sizeof(location), "at physical address 0x%lx", base); + msg_cinfo("mapped at physical address 0x%0*" PRIxPTR ".\n", + PRIxPTR_WIDTH, flash->physical_memory); else #endif - snprintf(location, sizeof(location), "on %s", programmer_table[programmer].name); + msg_cinfo("on %s.\n", programmer_table[programmer].name); - tmp = flashbuses_to_text(flash->chip->bustype); - msg_cinfo("%s %s flash chip \"%s\" (%d kB, %s) %s.\n", force ? "Assuming" : "Found", - flash->chip->vendor, flash->chip->name, flash->chip->total_size, tmp, location); - free(tmp); - - /* Flash registers will not be mapped if the chip was forced. Lock info - * may be stored in registers, so avoid lock info printing. - */ + /* Flash registers may more likely not be mapped if the chip was forced. + * Lock info may be stored in registers, so avoid lock info printing. */ if (!force) if (flash->chip->printlock) flash->chip->printlock(flash); + /* Get out of the way for later runs. */ + unmap_flash(flash); + /* Return position of matching chip. */ return chip - flashchips; } Modified: trunk/jedec.c ============================================================================== --- trunk/jedec.c Wed Aug 20 20:56:35 2014 (r1846) +++ trunk/jedec.c Sun Aug 31 01:39:51 2014 (r1847) @@ -166,9 +166,6 @@ if (man_id != chip->manufacture_id || dev_id != chip->model_id) return 0; - if (chip->feature_bits & FEATURE_REGISTERMAP) - map_flash_registers(flash); - return 1; } @@ -287,9 +284,6 @@ if (largeid1 != chip->manufacture_id || largeid2 != chip->model_id) return 0; - if (chip->feature_bits & FEATURE_REGISTERMAP) - map_flash_registers(flash); - return 1; }

1 0

Re: [flashrom] failed stumpy
by Stefan Tauner 29 Aug '14

29 Aug '14

On Thu, 28 Aug 2014 21:47:32 -0700 Maison Pulaski <maisonpulaski(a)gmail.com> wrote: > Hello John (if this is John?), Hi, no, we are the flashrom mailing list. Flashrom is the tool underneath John's script that does the actual flash handling. John is subscribed to the list though and should answer in more detail soon-ish, I presume. > I got a scary warning error when trying to flash the Stumpy Rom... 25 bytes of the whole image could not be written. It is probably harmless and just because John's image differs in some write-protected regions which are benign, but without knowing exactly where the differences are you can not be sure. If you don't get a reply from John in a timely manner, I would suggest that you write your backup and we investigate it further when your laptop is in a safe state. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

2 1

Re: [flashrom] failed stumpy
by John Lewis 29 Aug '14

29 Aug '14

No, this isn't me. Could you please keep to the G+ community and/or website. Flashrom is a utility used as part of the script, not written or maintained by me, so don't direct queries at Flashrom devs. On 29/08/14 05:47, Maison Pulaski wrote: > Hello John (if this is John?), > > I got a scary warning error when trying to flash the Stumpy Rom... > > "About to flash new ROM, repeat 'If this bricks my Chromebook/box, on my > head be it!' observing exact case and punctuation: If this bricks my > Chromebook/box, on my head be it! > > 1+0 records in > 1+0 records out > 4096 bytes (4.1 kB) copied, 0.00286514 s, 1.4 MB/s > flashrom v0.9.4 : cac0a4a : May 09 2014 16:55:24 UTC on Linux 3.4.0 > (x86_64), built with libpci 3.1.10, GCC 4.8.x-google 20131105 > (prerelease), little endian > VERIFY FAILED at 0x00408fbc! Expected=0x66, Read=0xff, failed byte count > from 0x00000000-0x007fffff: 0x19 > Your flash chip is in an unknown state. > Get help on IRC at irc.freenode.net <http://irc.freenode.net> (channel > #flashrom) or > mail flashrom(a)flashrom.org <mailto:flashrom@flashrom.org> with FAILED: > your board name in the subject line! > ------------------------------------------------------------------------------- > DO NOT REBOOT OR POWEROFF! > Erasing and writing flash chip... Verifying flash... FAILED" > > So, I am not exactly sure what to do now? > > Sorry. > > I did send you a little something (92R63514PY632983E). I know that it is > not payment for services rendered and if you can't help me I totally > understand. It is just a little something for the help you have already > given. > > Thanks, > Maison Pulaski > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom >

1 0

failed stumpy
by Maison Pulaski 28 Aug '14

28 Aug '14

Hello John (if this is John?), I got a scary warning error when trying to flash the Stumpy Rom... "About to flash new ROM, repeat 'If this bricks my Chromebook/box, on my head be it!' observing exact case and punctuation: If this bricks my Chromebook/box, on my head be it! 1+0 records in 1+0 records out 4096 bytes (4.1 kB) copied, 0.00286514 s, 1.4 MB/s flashrom v0.9.4 : cac0a4a : May 09 2014 16:55:24 UTC on Linux 3.4.0 (x86_64), built with libpci 3.1.10, GCC 4.8.x-google 20131105 (prerelease), little endian VERIFY FAILED at 0x00408fbc! Expected=0x66, Read=0xff, failed byte count from 0x00000000-0x007fffff: 0x19 Your flash chip is in an unknown state. Get help on IRC at irc.freenode.net (channel #flashrom) or mail flashrom(a)flashrom.org with FAILED: your board name in the subject line! ------------------------------------------------------------------------------- DO NOT REBOOT OR POWEROFF! Erasing and writing flash chip... Verifying flash... FAILED" So, I am not exactly sure what to do now? Sorry. I did send you a little something (92R63514PY632983E). I know that it is not payment for services rendered and if you can't help me I totally understand. It is just a little something for the help you have already given. Thanks, Maison Pulaski

1 0

Re: [flashrom] [RFC] split code into sub-directories
by Carl-Daniel Hailfinger 28 Aug '14

28 Aug '14

Hi Sean, nice to see you're back. Am 28.08.2014 00:57 schrieb Sean Nelson: > Move files into a directory structure, and edit Makefiles to facilitate > future compilation granularity. For Example, compile a focused version of > flashrom for a specific coreboot motherboard. > > subversion operations: > svn mkdir {chips,programmers,libraries} > > svn move 82802ab.c chips/ > svn move at45db.c chips/ > svn move chipdrivers.h chips/ > svn move en29lv640b.c chips/ > svn move flashchips.c chips/ > svn move flashchips.h chips/ > svn move jedec.c chips/ > svn move opaque.c chips/ > svn move sfdp.c chips/ > svn move spi25.c chips/ > svn move spi25_statusreg.c chips/ > svn move spi.c chips/ > svn move spi.h chips/ > svn move sst28sf040.c chips/ > svn move sst49lfxxxc.c chips/ > svn move sst_fwhub.c chips/ > svn move stm50.c chips/ > svn move w29ee011.c chips/ > svn move w39.c chips/ > > svn move flashrom.c libraries/ > svn move helpers.c libraries/ > svn move layout.c libraries/ > svn move programmer.c libraries/ > svn move programmer.h libraries/ > svn move udelay.c libraries/ > > svn move amd_imc.c programmers/ > svn move atahpt.c programmers/ > svn move atavia.c programmers/ > svn move bitbang_spi.c programmers/ > svn move board_enable.c programmers/ > svn move buspirate_spi.c programmers/ > svn move cbtable.c programmers/ > svn move chipset_enable.c programmers/ > svn move coreboot_tables.h programmers/ > svn move dediprog.c programmers/ > svn move dmi.c programmers/ > svn move drkaiser.c programmers/ > svn move dummyflasher.c programmers/ > svn move ft2232_spi.c programmers/ > svn move gfxnvidia.c programmers/ > svn move hwaccess.c programmers/ > svn move hwaccess.h programmers/ > svn move ich_descriptors.c programmers/ > svn move ich_descriptors.h programmers/ > svn move ichspi.c programmers/ > svn move internal.c programmers/ > svn move it8212.c programmers/ > svn move it85spi.c programmers/ > svn move it87spi.c programmers/ > svn move linux_spi.c programmers/ > svn move mcp6x_spi.c programmers/ > svn move nic3com.c programmers/ > svn move nicintel.c programmers/ > svn move nicintel_eeprom.c programmers/ > svn move nicintel_spi.c programmers/ > svn move nicnatsemi.c programmers/ > svn move nicrealtek.c programmers/ > svn move ogp_spi.c programmers/ > svn move pcidev.c programmers/ > svn move physmap.c programmers/ > svn move pony_spi.c programmers/ > svn move processor_enable.c programmers/ > svn move rayer_spi.c programmers/ > svn move satamv.c programmers/ > svn move satasii.c programmers/ > svn move sb600spi.c programmers/ > svn move serial.c programmers/ > svn move serprog.c programmers/ > svn move serprog.h programmers/ > svn move usbblaster_spi.c programmers/ > svn move wbsio_spi.c programmers/ Good idea, I think we had a similar suggestion already in the past, but it went nowhere because the patch queue was too big. I would like to get the probing stuff in first, though. Rebasing that would be too painful. Regards, Carl-Daniel

1 0

[RFC] split code into sub-directories
by Sean Nelson 27 Aug '14

27 Aug '14

Move files into a directory structure, and edit Makefiles to facilitate future compilation granularity. For Example, compile a focused version of flashrom for a specific coreboot motherboard. subversion operations: svn mkdir {chips,programmers,libraries} svn move 82802ab.c chips/ svn move at45db.c chips/ svn move chipdrivers.h chips/ svn move en29lv640b.c chips/ svn move flashchips.c chips/ svn move flashchips.h chips/ svn move jedec.c chips/ svn move opaque.c chips/ svn move sfdp.c chips/ svn move spi25.c chips/ svn move spi25_statusreg.c chips/ svn move spi.c chips/ svn move spi.h chips/ svn move sst28sf040.c chips/ svn move sst49lfxxxc.c chips/ svn move sst_fwhub.c chips/ svn move stm50.c chips/ svn move w29ee011.c chips/ svn move w39.c chips/ svn move flashrom.c libraries/ svn move helpers.c libraries/ svn move layout.c libraries/ svn move programmer.c libraries/ svn move programmer.h libraries/ svn move udelay.c libraries/ svn move amd_imc.c programmers/ svn move atahpt.c programmers/ svn move atavia.c programmers/ svn move bitbang_spi.c programmers/ svn move board_enable.c programmers/ svn move buspirate_spi.c programmers/ svn move cbtable.c programmers/ svn move chipset_enable.c programmers/ svn move coreboot_tables.h programmers/ svn move dediprog.c programmers/ svn move dmi.c programmers/ svn move drkaiser.c programmers/ svn move dummyflasher.c programmers/ svn move ft2232_spi.c programmers/ svn move gfxnvidia.c programmers/ svn move hwaccess.c programmers/ svn move hwaccess.h programmers/ svn move ich_descriptors.c programmers/ svn move ich_descriptors.h programmers/ svn move ichspi.c programmers/ svn move internal.c programmers/ svn move it8212.c programmers/ svn move it85spi.c programmers/ svn move it87spi.c programmers/ svn move linux_spi.c programmers/ svn move mcp6x_spi.c programmers/ svn move nic3com.c programmers/ svn move nicintel.c programmers/ svn move nicintel_eeprom.c programmers/ svn move nicintel_spi.c programmers/ svn move nicnatsemi.c programmers/ svn move nicrealtek.c programmers/ svn move ogp_spi.c programmers/ svn move pcidev.c programmers/ svn move physmap.c programmers/ svn move pony_spi.c programmers/ svn move processor_enable.c programmers/ svn move rayer_spi.c programmers/ svn move satamv.c programmers/ svn move satasii.c programmers/ svn move sb600spi.c programmers/ svn move serial.c programmers/ svn move serprog.c programmers/ svn move serprog.h programmers/ svn move usbblaster_spi.c programmers/ svn move wbsio_spi.c programmers/

1 0

Re: [flashrom] [PATCH 1/3] Refine Flash Component descriptor handling.
by Stefan Tauner 27 Aug '14

27 Aug '14

On Wed, 27 Aug 2014 16:44:23 -0400 "Wen Wang" <wen.wang(a)adiengineering.com> wrote: > Hi Stefan, > > Looks like my problem is in the flash descriptor that I received from intel. We generated our own descriptor and I am now able to read and write flash. I will see if I could figure out the difference in the descriptor. > Hi, please see http://www.flashrom.org/pipermail/flashrom/2014-August/012856.html I have explained the problem back then already. Apparently you are not the only one with the problem. Maybe we should add a workaround for it in one way or another... -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

[PATCH v2] Enable Dediprog SF100 programmers with FW 5.5.1
by Pablo Cases 26 Aug '14

26 Aug '14

v2 changes: * Major v1 bug fixed: Partial reads and writes are now supported. Read, Erase, Write tested up to 8MiB on Winbond W25Q64BV part. * Trivial v1 bug fixed: Block and Sector erase commands functional (previously only Chip Erase commands were functional). Signed-off-by: Pablo Cases <pablo.cases at flatfrog.com> Index: dediprog.c =================================================================== --- dediprog.c (revision 1846) +++ dediprog.c (working copy) @@ -101,8 +101,13 @@ target_leds = leds; } - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, target_leds, + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + ret = usb_control_msg(dediprog_handle, 0x42, 0x07, (target_leds << 8) | 0x09, 0x0, NULL, 0x0, DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, target_leds, + NULL, 0x0, DEFAULT_TIMEOUT); + if (ret != 0x0) { msg_perr("Command Set LED 0x%x failed (%s)!\n", leds, usb_strerror()); @@ -216,8 +221,28 @@ const char count_and_chunk[] = {count & 0xff, (count >> 8) & 0xff, chunksize & 0xff, - (chunksize >> 8) & 0xff}; + (chunksize >> 8) & 0xff, + 0xff, + 0x00, + (start % 0x10000) & 0xff, + ((start % 0x10000) >> 8) & 0xff, + (start / 0x10000) & 0xff, + ((start / 0x10000) >> 8) & 0xff}; + unsigned int count_and_chunk_size; + /* Reverse engineering note: + * ======================== + * Omitting the added 6 bytes to count_and_chunk[] on SF100 FW 5.5.1 + * results in SF100 to incorrectly issue 0x0B (Fast Read) command with + * start at SPI address 0xF30000 instead of the expected address 0x0. + * (On a 8MiB W25Q64.V chip this wraps around to physical address 0x730000). + * Possibly a result of SF100 FW 5.5.1 receive data buffer initialization. + */ + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + count_and_chunk_size = 10; + else + count_and_chunk_size = 4; + if ((start % chunksize) || (len % chunksize)) { msg_perr("%s: Unaligned start=%i, len=%i! Please report a bug " "at flashrom(a)flashrom.org\n", __func__, start, len); @@ -230,10 +255,15 @@ /* Command Read SPI Bulk. No idea which read command is used on the * SPI side. */ - ret = usb_control_msg(dediprog_handle, 0x42, 0x20, start % 0x10000, + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + ret = usb_control_msg(dediprog_handle, 0x42, 0x20, 0x00, + 0x00, (char *)count_and_chunk, + count_and_chunk_size, DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0x42, 0x20, start % 0x10000, start / 0x10000, (char *)count_and_chunk, - sizeof(count_and_chunk), DEFAULT_TIMEOUT); - if (ret != sizeof(count_and_chunk)) { + count_and_chunk_size, DEFAULT_TIMEOUT); + if (ret != count_and_chunk_size) { msg_perr("Command Read SPI Bulk failed, %i %s!\n", ret, usb_strerror()); return 1; @@ -316,9 +346,37 @@ * space in a USB bulk transfer must be filled with 0xff padding. */ const unsigned int count = len / chunksize; - const char count_and_cmd[] = {count & 0xff, (count >> 8) & 0xff, 0x00, dedi_spi_cmd}; + const char count_and_cmd[] = {count & 0xff, + (count >> 8) & 0xff, + 0x00, + dedi_spi_cmd, + 0xff, + 0x00, + (start % 0x10000) & 0xff, + ((start % 0x10000) >> 8) & 0xff, + (start / 0x10000) & 0xff, + ((start / 0x10000) >> 8) & 0xff}; + unsigned int count_and_cmd_size; char usbbuf[512]; - + + /* Reverse engineering note: + * ======================== + * Omitting the added 6 bytes to count_and_cmd[] on SF100 FW 5.5.1 + * results in SF100 to incorrectly issue 0x02 (Page Program) command with + * start at SPI address 0xF30000 instead of the expected address 0x0. + * (On a 8MiB W25Q64.V chip this wraps around to physical address 0x730000). + * Possibly a result of SF100 FW 5.5.1 receive data buffer initialization. + * + * Issuing a single spi_bulk_write() command with this appendage will + * correct the addressing so that all further spi_bulk_write() commands + * (with or without the appendage) will start at SPI address 0x0. + * Only a power cycle of the SF100 will reintroduce the error. + */ + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + count_and_cmd_size = 10; + else + count_and_cmd_size = 4; + /* * We should change this check to * chunksize > 512 @@ -342,9 +400,13 @@ /* Command Write SPI Bulk. No idea which write command is used on the * SPI side. */ - ret = usb_control_msg(dediprog_handle, 0x42, 0x30, start % 0x10000, start / 0x10000, - (char *)count_and_cmd, sizeof(count_and_cmd), DEFAULT_TIMEOUT); - if (ret != sizeof(count_and_cmd)) { + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + ret = usb_control_msg(dediprog_handle, 0x42, 0x30, 0x00, 0x00, + (char *)count_and_cmd, count_and_cmd_size, DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0x42, 0x30, start % 0x10000, start / 0x10000, + (char *)count_and_cmd, count_and_cmd_size, DEFAULT_TIMEOUT); + if (ret != count_and_cmd_size) { msg_perr("Command Write SPI Bulk failed, %i %s!\n", ret, usb_strerror()); return 1; @@ -447,10 +509,28 @@ msg_perr("Untested readcnt=%i, aborting.\n", readcnt); return 1; } - - ret = usb_control_msg(dediprog_handle, 0x42, 0x1, 0xff, - readcnt ? 0x1 : 0x0, (char *)writearr, writecnt, - DEFAULT_TIMEOUT); + + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + if (*writearr == 0x6 || // Write Enable + *writearr == 0x60 || *writearr == 0xc7 || // Chip Erase + *writearr == 0x52 || *writearr == 0xd8 || // Block Erase + *writearr == 0x20) // Sector Erase + /* These commands are treated differently otherwise + * Read Status Register-1 (05h) will return incorrect + * results for WEL and WIP bits + * (at least on Winbond W25Q64BV Flash) + */ + ret = usb_control_msg(dediprog_handle, 0x42, 0x1, 0x0, + 0x0, (char *)writearr, writecnt, + DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0x42, 0x1, 0x1, + 0x0, (char *)writearr, writecnt, + DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0x42, 0x1, 0xff, + readcnt ? 0x1 : 0x0, (char *)writearr, writecnt, + DEFAULT_TIMEOUT); if (ret != writecnt) { msg_perr("Send SPI failed, expected %i, got %i %s!\n", writecnt, ret, usb_strerror()); @@ -459,8 +539,13 @@ if (!readcnt) return 0; memset(readarr, 0, readcnt); - ret = usb_control_msg(dediprog_handle, 0xc2, 0x01, 0xbb8, 0x0000, - (char *)readarr, readcnt, DEFAULT_TIMEOUT); + + if (dediprog_firmwareversion >= FIRMWARE_VERSION(5,5,1)) + ret = usb_control_msg(dediprog_handle, 0xc2, 0x1, 0x1, 0x0, + (char *)readarr, readcnt, DEFAULT_TIMEOUT); + else + ret = usb_control_msg(dediprog_handle, 0xc2, 0x1, 0xbb8, 0x0000, + (char *)readarr, readcnt, DEFAULT_TIMEOUT); if (ret != readcnt) { msg_perr("Receive SPI failed, expected %i, got %i %s!\n", readcnt, ret, usb_strerror()); @@ -485,6 +570,12 @@ " String!\n"); return 1; } + /* Reverse engineering note: + * ======================== + * Dediprog Software SF6.0.4.33 application sends the devicestring + * command with wValue and wIndex values of 0x0 instead of 0xff. + * Both commands yield the same result, so nothing is changed for now. + */ /* Command Receive Device String. */ memset(buf, 0, sizeof(buf)); ret = usb_control_msg(dediprog_handle, 0xc2, 0x8, 0xff, 0xff, buf,

1 0

AT45DB011D read not working
by The Raven 25 Aug '14

25 Aug '14

Read is not working, other things (yet) not tested. See log for details. It ends with a "Speicherzugriffsfehler" which means "Segmentation fault". I can do more tests if needed. Tell me what you need. :-) THX

3 9

Re: [flashrom] [patch] exit when the same filename is used for reading/writing flash chip content and to store log output
by Daniele Forsi 25 Aug '14

25 Aug '14

2014-08-21 13:37 GMT+02:00 Idwer Vollering: > Tested on physical hardware, built on FreeBSD (less portability 'headaches'). > > Signed-off-By: Idwer Vollering <vidwer(a)gmail.com> > > Index: cli_output.c > =================================================================== > --- cli_output.c (revision 1846) > +++ cli_output.c (working copy) > @@ -52,8 +52,12 @@ > msg_gerr("No logfile name specified.\n"); > return 1; > } > - if ((logfile = fopen(filename, "w")) == NULL) { > + if ((logfile = fopen(filename, "xw")) == NULL) { > msg_gerr("Error: opening log file \"%s\" failed: > %s\n", filename, strerror(errno)); > + msg_gdbg("This means that flashrom can't write log output\n" \ > + "to the file you specified (\"%s\"),\n" \ > + "or you are using the same filename to read > from/write to (-r / -w)\n" \ > + "and write the log output to (-o).\n", filename); > return 1; > } > return 0; "xw" always fails if filename exists even if it's not the same used for read or write, yes strerror() will have printed "File exists" but having to delete the log file by hand is inconvenient (I always create a log file "just in case" but I rarely need it) -- Daniele Forsi

1 0

Asus x75vd laptop repaired, thanks !
by Alex 25 Aug '14

25 Aug '14

Hi, Thanks for this ! You save my x75vd laptop ! One arduino, one resistor on a breadboard. I use the improved 3.3v version by Simon Inns but without 4050n because I didn't have this. The 4050N is a non-inverting hex buffer IC: http://www.nxp.com/documents/data_sheet/HEF4050B.pdf See my post about it here : http://www.waitingforfriday.com/forum/viewtopic.php?f=10&t=3359 My Circuit : https://www.dropbox.com/s/95gb5gh0yp84hog/2014-08-23%2018.43.54.jpg?dl=0 I used : - Ubuntu to install and upload serprog-duino quickly - SystemRescueCd (live CD with flashrom) to run flashrom with command line I tried flashrom windows version withtout success (command line : mingw32-w64-flashrom-r1781.exe -p serprog:dev=COM3:115200 -r coreboot.rom) The hardest was to find a real rom extraction and not the bios given on the Asus website (wich is smaller than the EEPROM) Thanks a lot :)

1 0

Re: [flashrom] [PATCH] Add SUPPORT_EMAIL compiler flag.
by David Hendricks 21 Aug '14

21 Aug '14

Overall I like the concept of this patch, and I think it should get merged. I'm not entirely convinced that redirecting people using the chromiumos branch to a separate list will significantly reduce the traffic on this list though. AFAICT most people who report issues are trying to flash manually with upstream flashrom (or a version bundled with their distro) and encounter issues with the ME or some such. On Thu, Aug 21, 2014 at 1:16 AM, Stefan Tauner < stefan.tauner(a)alumni.tuwien.ac.at> wrote: > Use it throughout the source to allow for printing alternative support > email > addresses (e.g. for distributions, chromiumos or other 3rd parties). > Also, fix some indentions on the way. > > Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > --- > > This of course only makes sense if someone uses it (and in the case of > chromium port it to their tree... and set up some kind of community > mailing list or something?). Therefore I would like to hear your thoughts > about it. Would you use it? > > Makefile | 3 +++ > bitbang_spi.c | 2 +- > board_enable.c | 12 ++++++------ > chipset_enable.c | 16 ++++++---------- > cli_common.c | 2 +- > dediprog.c | 10 +++++----- > flashrom.c | 50 +++++++++++++++++++++----------------------------- > jedec.c | 2 +- > opaque.c | 2 +- > pcidev.c | 10 +++++----- > physmap.c | 6 ++---- > print.c | 2 +- > programmer.c | 2 +- > sb600spi.c | 8 ++++---- > sfdp.c | 5 ++--- > spi.c | 6 +++--- > spi25.c | 6 +++--- > 17 files changed, 66 insertions(+), 78 deletions(-) > > diff --git a/Makefile b/Makefile > index 98230a5..1e85f41 100644 > --- a/Makefile > +++ b/Makefile > @@ -19,6 +19,7 @@ > # > > PROGRAM = flashrom > +SUPPORT_EMAIL = flashrom(a)flashrom.org > > > ############################################################################### > # Defaults for the toolchain. > @@ -723,6 +724,8 @@ endif > > FEATURE_CFLAGS += $(shell LC_ALL=C grep -q "UTSNAME := yes" .features && > printf "%s" "-D'HAVE_UTSNAME=1'") > > +FEATURE_CFLAGS += -D'SUPPORT_EMAIL="$(SUPPORT_EMAIL)"' > + > # We could use PULLED_IN_LIBS, but that would be ugly. > FEATURE_LIBS += $(shell LC_ALL=C grep -q "NEEDLIBZ := yes" .libdeps && > printf "%s" "-lz") > > diff --git a/bitbang_spi.c b/bitbang_spi.c > index 0b27a67..9a0441c 100644 > --- a/bitbang_spi.c > +++ b/bitbang_spi.c > @@ -94,7 +94,7 @@ int register_spi_bitbang_master(const struct > bitbang_spi_master *master) > (master->request_bus && !master->release_bus) || > (!master->request_bus && master->release_bus)) { > msg_perr("Incomplete SPI bitbang master setting!\n" > - "Please report a bug at flashrom(a)flashrom.org > \n"); > + "Please report a bug at " SUPPORT_EMAIL "\n"); > return ERROR_FLASHROM_BUG; > } > > diff --git a/board_enable.c b/board_enable.c > index 1235bb8..6606a59 100644 > --- a/board_enable.c > +++ b/board_enable.c > @@ -335,7 +335,7 @@ void probe_superio_winbond(void) > } > msg_pinfo("Active config mode, unknown reg 0x20 > ID: %02x.\n", model); > msg_pinfo("Please send the output of \"flashrom -V > -p internal\" to \n" > - "flashrom(a)flashrom.org with W836xx: > your board name: flashrom -V\n" > + SUPPORT_EMAIL " with W836xx: your board > name: flashrom -V\n" > "as the subject to help us finish > support for your Super I/O. Thanks.\n"); > continue; > } > @@ -620,7 +620,7 @@ int it8705f_write_enable(uint8_t port) > /* The data sheet contradicts itself about max > size. */ > max_rom_decode.parallel = 1024 * 1024; > msg_pinfo("IT8705F with very unusual settings.\n" > - "Please send the output of \"flashrom -V > -p internal\" to flashrom(a)flashrom.org\n" > + "Please send the output of \"flashrom -V > -p internal\" to " SUPPORT_EMAIL "\n" > "with \"IT8705: your board name: > flashrom -V\" as the subject to help us finish\n" > "support for your Super I/O. Thanks.\n"); > ret = 1; > @@ -2479,7 +2479,7 @@ int selfcheck_board_enables(void) > const struct board_match *b = &board_matches[i]; > if (b->vendor_name == NULL || b->board_name == NULL) { > msg_gerr("ERROR: Board enable #%d does not define > a vendor and board name.\n" > - "Please report a bug at > flashrom(a)flashrom.org\n", i); > + "Please report a bug at " SUPPORT_EMAIL > "\n", i); > ret = 1; > continue; > } > @@ -2488,7 +2488,7 @@ int selfcheck_board_enables(void) > ((b->lb_vendor == NULL) ^ (b->lb_part == NULL)) || > (b->max_rom_decode_parallel == 0 && b->enable == > NULL)) { > msg_gerr("ERROR: Board enable for %s %s is > misdefined.\n" > - "Please report a bug at > flashrom(a)flashrom.org\n", > + "Please report a bug at " SUPPORT_EMAIL > "\n", > b->vendor_name, b->board_name); > ret = 1; > } > @@ -2553,7 +2553,7 @@ static const struct board_match > *board_match_name(const char *vendor, const char > if (partmatch) { > /* More than one entry has a matching name. */ > msg_perr("Board name \"%s\":\"%s\" and PCI IDs > matched more than one board enable " > - "entry. Please report a bug at > flashrom(a)flashrom.org\n", vendor, model); > + "entry. Please report a bug at " > SUPPORT_EMAIL "\n", vendor, model); > return NULL; > } > partmatch = board; > @@ -2636,7 +2636,7 @@ static int board_enable_safetycheck(const struct > board_match *board) > return 1; > } > msg_pwarn("NOTE: Running an untested board enable procedure.\n" > - "Please report success/failure to flashrom(a)flashrom.org > .\n"); > + "Please report success/failure to " SUPPORT_EMAIL ".\n"); > return 0; > } > > diff --git a/chipset_enable.c b/chipset_enable.c > index dd4e0ea..8334866 100644 > --- a/chipset_enable.c > +++ b/chipset_enable.c > @@ -1393,7 +1393,7 @@ static int enable_flash_mcp6x_7x(struct pci_dev > *dev, const char *name) > internal_buses_supported = BUS_NONE; > msg_pwarn("Flash bus type is unknown (none)\n"); > msg_pinfo("Please send the log files created by \"flashrom > -p internal -o logfile\" to \n" > - "flashrom(a)flashrom.org with \"your board name: > flashrom -V\" as the subject to\n" > + SUPPORT_EMAIL " with \"your board name: flashrom > -V\" as the subject to\n" > "help us finish support for your chipset. > Thanks.\n"); > return ERROR_NONFATAL; > } > @@ -1791,14 +1791,10 @@ int chipset_flash_enable(void) > if (!dev) > continue; > if (ret != -2) { > - msg_pwarn("Warning: unexpected second chipset > match: " > - "\"%s %s\"\n" > - "ignoring, please report lspci and board > URL " > - "to flashrom(a)flashrom.org\n" > - "with \'CHIPSET: your board name\' in > the " > - "subject line.\n", > - chipset_enables[i].vendor_name, > - chipset_enables[i].device_name); > + msg_pwarn("Warning: unexpected second chipset > match: \"%s %s\"\n" > + "ignoring, please report lspci and board > URL to " SUPPORT_EMAIL "\n" > + "with \'CHIPSET: your board name\' in > the subject line.\n", > + chipset_enables[i].vendor_name, > chipset_enables[i].device_name); > continue; > } > msg_pinfo("Found chipset \"%s %s\"", > @@ -1815,7 +1811,7 @@ int chipset_flash_enable(void) > "flashrom *and* were (not) able to " > "successfully update your firmware with > it,\n" > "then please email a report to " > - "flashrom(a)flashrom.org including a > verbose " > + SUPPORT_EMAIL " including a verbose " > "(-V) log.\nThank you!\n"); > } > msg_pinfo("Enabling flash write... "); > diff --git a/cli_common.c b/cli_common.c > index 71cc2dd..1cae546 100644 > --- a/cli_common.c > +++ b/cli_common.c > @@ -107,7 +107,7 @@ void print_chip_support_status(const struct flashchip > *chip) > } > 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" > + "please email a report to " SUPPORT_EMAIL " if > any of the above operations\n" > "work correctly for you with this flash chip. > Please include the flashrom log\n" > "file for all operations you tested (see the man > page for details), and mention\n" > "which mainboard or programmer you tested in the > subject line.\n" > diff --git a/dediprog.c b/dediprog.c > index 71a9f8b..676248b 100644 > --- a/dediprog.c > +++ b/dediprog.c > @@ -219,8 +219,8 @@ static int dediprog_spi_bulk_read(struct flashctx > *flash, uint8_t *buf, > (chunksize >> 8) & 0xff}; > > if ((start % chunksize) || (len % chunksize)) { > - msg_perr("%s: Unaligned start=%i, len=%i! Please report a > bug " > - "at flashrom(a)flashrom.org\n", __func__, start, > len); > + msg_perr("%s: Unaligned start=%i, len=%i! Please report a > bug at " SUPPORT_EMAIL "\n", > + __func__, start, len); > return 1; > } > > @@ -326,13 +326,13 @@ static int dediprog_spi_bulk_write(struct flashctx > *flash, const uint8_t *buf, u > */ > if (chunksize != 256) { > msg_perr("%s: Chunk sizes other than 256 bytes are > unsupported, chunksize=%u!\n" > - "Please report a bug at flashrom(a)flashrom.org\n", > __func__, chunksize); > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__, chunksize); > return 1; > } > > if ((start % chunksize) || (len % chunksize)) { > - msg_perr("%s: Unaligned start=%i, len=%i! Please report a > bug " > - "at flashrom(a)flashrom.org\n", __func__, start, > len); > + msg_perr("%s: Unaligned start=%i, len=%i!\nPlease report a > bug at " SUPPORT_EMAIL "\n", > + __func__, start, len); > return 1; > } > > diff --git a/flashrom.c b/flashrom.c > index 93b292b..1148fd4 100644 > --- a/flashrom.c > +++ b/flashrom.c > @@ -558,8 +558,7 @@ char *extract_param(const char *const *haystack, const > char *needle, const char > > needlelen = strlen(needle); > if (!needlelen) { > - msg_gerr("%s: empty needle! Please report a bug at " > - "flashrom(a)flashrom.org\n", __func__); > + msg_gerr("%s: empty needle! Please report a bug at " > SUPPORT_EMAIL "\n", __func__); > return NULL; > } > /* No programmer parameters given. */ > @@ -780,8 +779,7 @@ int need_erase(const uint8_t *have, const uint8_t > *want, unsigned int len, enum > result = 0; > break; > default: > - msg_cerr("%s: Unsupported granularity! Please report a bug > at " > - "flashrom(a)flashrom.org\n", __func__); > + msg_cerr("%s: Unsupported granularity! Please report a bug > at " SUPPORT_EMAIL "\n", __func__); > } > return result; > } > @@ -842,8 +840,7 @@ static unsigned int get_next_write(const uint8_t > *have, const uint8_t *want, uns > stride = 1056; > break; > default: > - msg_cerr("%s: Unsupported granularity! Please report a bug > at " > - "flashrom(a)flashrom.org\n", __func__); > + msg_cerr("%s: Unsupported granularity! Please report a bug > at " SUPPORT_EMAIL "\n", __func__); > /* Claim that no write was needed. A write with unknown > * granularity is too dangerous to try. > */ > @@ -1119,7 +1116,7 @@ int probe_flash(struct registered_master *mst, int > startchip, struct flashctx *f > > msg_cinfo(" we need to add them manually.\n" > "You can help us by mailing us the > output of the following command to " > - "flashrom@flashrom.org:\n" > + SUPPORT_EMAIL ":\n" > "'flashrom -VV [plus the -p/--programmer > parameter]'\n" > "Thanks for your help!\n" > "===\n"); > @@ -1278,19 +1275,17 @@ static int selfcheck_eraseblocks(const struct > flashchip *chip) > /* Blocks with zero size are bugs in flashchips.c. > */ > if (eraser.eraseblocks[i].count && > !eraser.eraseblocks[i].size) { > - msg_gerr("ERROR: Flash chip %s erase > function " > - "%i region %i has size 0. Please > report" > - " a bug at flashrom(a)flashrom.org > \n", > - chip->name, k, i); > + msg_gerr("ERROR: Flash chip %s erase > function %i region %i has size 0.\n" > + "Please report a bug at " > SUPPORT_EMAIL "\n", > + chip->name, k, i); > ret = 1; > } > /* Blocks with zero count are bugs in > flashchips.c. */ > if (!eraser.eraseblocks[i].count && > eraser.eraseblocks[i].size) { > - msg_gerr("ERROR: Flash chip %s erase > function " > - "%i region %i has count 0. Please > report" > - " a bug at flashrom(a)flashrom.org > \n", > - chip->name, k, i); > + msg_gerr("ERROR: Flash chip %s erase > function %i region %i has count 0.\n" > + "Please report a bug at " > SUPPORT_EMAIL "\n", > + chip->name, k, i); > ret = 1; > } > done += eraser.eraseblocks[i].count * > @@ -1303,11 +1298,9 @@ static int selfcheck_eraseblocks(const struct > flashchip *chip) > if (!done) > continue; > if (done != chip->total_size * 1024) { > - msg_gerr("ERROR: Flash chip %s erase function %i " > - "region walking resulted in 0x%06x bytes > total," > - " expected 0x%06x bytes. Please report a > bug at" > - " flashrom(a)flashrom.org\n", chip->name, k, > - done, chip->total_size * 1024); > + msg_gerr("ERROR: Flash chip %s erase function %i > region walking resulted in " > + "0x%06x bytes total, expected 0x%06x > bytes. Please report a bug at" > + SUPPORT_EMAIL "\n", chip->name, k, done, > chip->total_size * 1024); > ret = 1; > } > if (!eraser.block_erase) > @@ -1319,10 +1312,9 @@ static int selfcheck_eraseblocks(const struct > flashchip *chip) > for (j = k + 1; j < NUM_ERASEFUNCTIONS; j++) { > if (eraser.block_erase == > chip->block_erasers[j].block_erase) { > - msg_gerr("ERROR: Flash chip %s erase > function " > - "%i and %i are identical. Please > report" > - " a bug at flashrom(a)flashrom.org > \n", > - chip->name, k, j); > + msg_gerr("ERROR: Flash chip %s erase > function %i and %i are identical.\n" > + "Please report a bug at " > SUPPORT_EMAIL "\n", > + chip->name, k, j); > ret = 1; > } > } > @@ -1518,7 +1510,7 @@ static void nonfatal_help_message(void) > if (programmer == PROGRAMMER_INTERNAL) > msg_gerr("This means we have to add special support for > your board, programmer or flash\n" > "chip. Please report this on IRC at > chat.freenode.net (channel #flashrom) or\n" > - "mail flashrom(a)flashrom.org, thanks!\n" > + "mail " SUPPORT_EMAIL ", thanks!\n" > > "-------------------------------------------------------------------------------\n" > "You may now reboot or simply leave the machine > running.\n"); > else > @@ -1526,7 +1518,7 @@ static void nonfatal_help_message(void) > msg_gerr("Please check the connections (especially those > to write protection pins) between\n" > "the programmer and the flash chip. If you think > the error is caused by flashrom\n" > "please report this on IRC at chat.freenode.net > (channel #flashrom) or\n" > - "mail flashrom(a)flashrom.org, thanks!\n"); > + "mail " SUPPORT_EMAIL ", thanks!\n"); > } > > static void emergency_help_message(void) > @@ -1535,13 +1527,13 @@ static void emergency_help_message(void) > #if CONFIG_INTERNAL == 1 > if (programmer == PROGRAMMER_INTERNAL) > msg_gerr("Get help on IRC at chat.freenode.net (channel > #flashrom) or\n" > - "mail flashrom(a)flashrom.org with the subject > \"FAILED: <your board name>\"!\n" > + "mail " SUPPORT_EMAIL " with the subject \"FAILED: > <your board name>\"!\n" > > "-------------------------------------------------------------------------------\n" > "DO NOT REBOOT OR POWEROFF!\n"); > else > #endif > msg_gerr("Please report this on IRC at chat.freenode.net > (channel #flashrom) or\n" > - "mail flashrom(a)flashrom.org, thanks!\n"); > + "mail " SUPPORT_EMAIL ", thanks!\n"); > } > > /* The way to go if you want a delimited list of programmers */ > @@ -1755,7 +1747,7 @@ int selfcheck(void) > if (chip->vendor == NULL || chip->name == NULL || > chip->bustype == BUS_NONE) { > ret = 1; > msg_gerr("ERROR: Some field of flash chip > #%d (%s) is misconfigured.\n" > - "Please report a bug at > flashrom(a)flashrom.org\n", i, > + "Please report a bug at " > SUPPORT_EMAIL "\n", i, > chip->name == NULL ? "unnamed" : > chip->name); > } > if (selfcheck_eraseblocks(chip)) { > diff --git a/jedec.c b/jedec.c > index 358b850..fa02c7b 100644 > --- a/jedec.c > +++ b/jedec.c > @@ -669,7 +669,7 @@ static int changelock_regspace2_block(const struct > flashctx *flash, chipaddr off > > if (new_bits & ~REG2_MASK) { > msg_cerr("Invalid locking change 0x%02x requested at > 0x%0*" PRIxPTR "! " > - "Please report a bug at flashrom(a)flashrom.org\n", > + "Please report a bug at " SUPPORT_EMAIL "\n", > new_bits, PRIxPTR_WIDTH, offset); > return -1; > } > diff --git a/opaque.c b/opaque.c > index a5a829c..5bf6737 100644 > --- a/opaque.c > +++ b/opaque.c > @@ -56,7 +56,7 @@ int register_opaque_master(const struct opaque_master > *mst) > > if (!mst->probe || !mst->read || !mst->write || !mst->erase) { > msg_perr("%s called with incomplete master definition. " > - "Please report a bug at flashrom(a)flashrom.org\n", > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return ERROR_FLASHROM_BUG; > } > diff --git a/pcidev.c b/pcidev.c > index 2c78063..6023438 100644 > --- a/pcidev.c > +++ b/pcidev.c > @@ -146,7 +146,7 @@ uintptr_t pcidev_readbar(struct pci_dev *dev, int bar) > addr &= PCI_ROM_ADDRESS_MASK; > break; > case TYPE_UNKNOWN: > - msg_perr("BAR type unknown, please report a bug at > flashrom(a)flashrom.org\n"); > + msg_perr("BAR type unknown, please report a bug at " > SUPPORT_EMAIL "\n"); > } > > return (uintptr_t)addr; > @@ -156,7 +156,7 @@ static int pcidev_shutdown(void *data) > { > if (pacc == NULL) { > msg_perr("%s: Tried to cleanup an invalid PCI context!\n" > - "Please report a bug at flashrom(a)flashrom.org\n", > __func__); > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return 1; > } > pci_cleanup(pacc); > @@ -167,7 +167,7 @@ int pci_init_common(void) > { > if (pacc != NULL) { > msg_perr("%s: Tried to allocate a new PCI context, but > there is still an old one!\n" > - "Please report a bug at flashrom(a)flashrom.org\n", > __func__); > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return 1; > } > pacc = pci_alloc(); /* Get the pci_access structure */ > @@ -225,7 +225,7 @@ struct pci_dev *pcidev_init(const struct dev_entry > *devs, int bar) > if (devs[i].status == NT) > msg_pinfo("===\nThis PCI device is > UNTESTED. Please report the 'flashrom -p " > "xxxx' output \n" > - "to flashrom(a)flashrom.org if it > works for you. Please add the name " > + "to " SUPPORT_EMAIL " if it > works for you. Please add the name " > "of your\n" > "PCI device to the subject. > Thank you for your help!\n===\n"); > > @@ -274,7 +274,7 @@ int undo_pci_write(void *p) > struct undo_pci_write_data *data = p; > if (pacc == NULL) { > msg_perr("%s: Tried to undo PCI writes without a valid PCI > context!\n" > - "Please report a bug at flashrom(a)flashrom.org\n", > __func__); > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return 1; > } > msg_pdbg("Restoring PCI config space for %02x:%02x:%01x reg > 0x%02x\n", > diff --git a/physmap.c b/physmap.c > index ca5afd0..6fb5e3b 100644 > --- a/physmap.c > +++ b/physmap.c > @@ -308,8 +308,7 @@ void physunmap_unaligned(void *virt_addr, size_t len) > { > /* No need to check for zero size, such mappings would have > yielded ERROR_PTR. */ > if (virt_addr == ERROR_PTR) { > - msg_perr("Trying to unmap a nonexisting mapping!\n" > - "Please report a bug at flashrom(a)flashrom.org > \n"); > + msg_perr("Trying to unmap a nonexisting mapping!\nPlease > report a bug at " SUPPORT_EMAIL "\n"); > return; > } > > @@ -322,8 +321,7 @@ void physunmap(void *virt_addr, size_t len) > > /* No need to check for zero size, such mappings would have > yielded ERROR_PTR. */ > if (virt_addr == ERROR_PTR) { > - msg_perr("Trying to unmap a nonexisting mapping!\n" > - "Please report a bug at flashrom(a)flashrom.org > \n"); > + msg_perr("Trying to unmap a nonexisting mapping!\nPlease > report a bug at " SUPPORT_EMAIL "\n"); > return; > } > tmp = (uintptr_t)virt_addr; > diff --git a/print.c b/print.c > index 243aa49..de9f85f 100644 > --- a/print.c > +++ b/print.c > @@ -522,7 +522,7 @@ int print_supported(void) > break; > default: > msg_gerr("\n%s: %s: Uninitialized programmer type! > Please report a bug at " > - "flashrom(a)flashrom.org\n", __func__, > prog.name); > + SUPPORT_EMAIL "\n", __func__, prog.name); > break; > } > } > diff --git a/programmer.c b/programmer.c > index fd634b2..c88e9b5 100644 > --- a/programmer.c > +++ b/programmer.c > @@ -103,7 +103,7 @@ int register_par_master(const struct par_master *mst, > !mst->chip_writen || !mst->chip_readb || !mst->chip_readw || > !mst->chip_readl || !mst->chip_readn) { > msg_perr("%s called with incomplete master definition. " > - "Please report a bug at flashrom(a)flashrom.org\n", > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return ERROR_FLASHROM_BUG; > } > diff --git a/sb600spi.c b/sb600spi.c > index 69fad65..381254e 100644 > --- a/sb600spi.c > +++ b/sb600spi.c > @@ -106,8 +106,8 @@ static void determine_generation(struct pci_dev *dev) > msg_pdbg("SB8xx/SB9xx/Hudson-1 detected.\n"); > } else { > msg_pwarn("SB device found but SMBus revision > 0x%02x does not match known values.\n" > - "Assuming SB8xx/SB9xx/Hudson-1. Please > send a log to flashrom(a)flashrom.org\n", > - rev); > + "Assuming SB8xx/SB9xx/Hudson-1. Please > send a log to " SUPPORT_EMAIL "\n", > + rev); > amd_gen = CHIPSET_SB89XX; > } > } else if (dev->device_id == 0x780e) { > @@ -146,13 +146,13 @@ static void determine_generation(struct pci_dev *dev) > msg_pdbg("Yangtze detected.\n"); > } else { > msg_pwarn("FCH device found but SMBus revision > 0x%02x does not match known values.\n" > - "Please report this to > flashrom(a)flashrom.org and include this log and\n" > + "Please report this to " SUPPORT_EMAIL " > and include this log and\n" > "the output of lspci -nnvx, thanks!.\n", > rev); > } > #endif > } else > msg_pwarn("%s: Unknown LPC device %" PRIx16 ":%" PRIx16 > ".\n" > - "Please report this to flashrom(a)flashrom.org > and include this log and\n" > + "Please report this to " SUPPORT_EMAIL " and > include this log and\n" > "the output of lspci -nnvx, thanks!\n", > __func__, dev->vendor_id, dev->device_id); > } > diff --git a/sfdp.c b/sfdp.c > index bc69dd0..ed22392 100644 > --- a/sfdp.c > +++ b/sfdp.c > @@ -89,8 +89,7 @@ static int sfdp_add_uniform_eraser(struct flashchip > *chip, uint8_t opcode, uint3 > > if (erasefn == NULL || total_size == 0 || block_size == 0 || > total_size % block_size != 0) { > - msg_cdbg("%s: invalid input, please report to " > - "flashrom(a)flashrom.org\n", __func__); > + msg_cdbg("%s: invalid input, please report to " > SUPPORT_EMAIL "\n", __func__); > return 1; > } > > @@ -120,7 +119,7 @@ static int sfdp_add_uniform_eraser(struct flashchip > *chip, uint8_t opcode, uint3 > return 0; > } > msg_cinfo("%s: Not enough space to store another eraser (i=%d)." > - " Please report this at flashrom(a)flashrom.org\n", > + " Please report this at " SUPPORT_EMAIL "\n", > __func__, i); > return 1; > } > diff --git a/spi.c b/spi.c > index 894f73f..102e9b5 100644 > --- a/spi.c > +++ b/spi.c > @@ -82,7 +82,7 @@ int default_spi_read(struct flashctx *flash, uint8_t > *buf, unsigned int start, > if (max_data == MAX_DATA_UNSPECIFIED) { > msg_perr("%s called, but SPI read chunk size not defined " > "on this hardware. Please report a bug at " > - "flashrom(a)flashrom.org\n", __func__); > + SUPPORT_EMAIL "\n", __func__); > return 1; > } > return spi_read_chunked(flash, buf, start, len, max_data); > @@ -94,7 +94,7 @@ int default_spi_write_256(struct flashctx *flash, const > uint8_t *buf, unsigned i > if (max_data == MAX_DATA_UNSPECIFIED) { > msg_perr("%s called, but SPI write chunk size not defined " > "on this hardware. Please report a bug at " > - "flashrom(a)flashrom.org\n", __func__); > + "" SUPPORT_EMAIL "\n", __func__); > return 1; > } > return spi_write_chunked(flash, buf, start, len, max_data); > @@ -174,7 +174,7 @@ int register_spi_master(const struct spi_master *mst) > ((mst->command == default_spi_send_command) && > (mst->multicommand == default_spi_send_multicommand))) { > msg_perr("%s called with incomplete master definition. " > - "Please report a bug at flashrom(a)flashrom.org\n", > + "Please report a bug at " SUPPORT_EMAIL "\n", > __func__); > return ERROR_FLASHROM_BUG; > } > diff --git a/spi25.c b/spi25.c > index 673bdf5..5cb9632 100644 > --- a/spi25.c > +++ b/spi25.c > @@ -835,7 +835,7 @@ erasefunc_t *spi_get_erasefn_from_opcode(uint8_t > opcode) > return &spi_block_erase_db; > default: > msg_cinfo("%s: unknown erase opcode (0x%02x). Please > report " > - "this at flashrom(a)flashrom.org\n", __func__, > opcode); > + "this at " SUPPORT_EMAIL "\n", __func__, opcode); > return NULL; > } > } > @@ -1101,7 +1101,7 @@ int default_spi_write_aai(struct flashctx *flash, > const uint8_t *buf, unsigned i > /* The data sheet requires a start address with the low bit > cleared. */ > if (start % 2) { > msg_cerr("%s: start address not even! Please report a bug > at " > - "flashrom(a)flashrom.org\n", __func__); > + "" SUPPORT_EMAIL "\n", __func__); > if (spi_chip_write_1(flash, buf, start, start % 2)) > return SPI_GENERIC_ERROR; > pos += start % 2; > @@ -1119,7 +1119,7 @@ int default_spi_write_aai(struct flashctx *flash, > const uint8_t *buf, unsigned i > /* The data sheet requires total AAI write length to be even. */ > if (len % 2) { > msg_cerr("%s: total write length not even! Please report a > " > - "bug at flashrom(a)flashrom.org\n", __func__); > + "bug at " SUPPORT_EMAIL "\n", __func__); > /* Do not return an error for now. */ > //return SPI_GENERIC_ERROR; > } > -- > Kind regards, Stefan Tauner > > -- David Hendricks (dhendrix) Systems Software Engineer, Google Inc.

1 0

FAILED: peppy
by Gerard Looije 21 Aug '14

21 Aug '14

*** Already downloaded! *** *** Backing up existing ROM to current directory *** flashrom v0.9.4 : 14ce0cf : Oct 09 2013 22:16:28 UTC on Linux 3.16.1-1-ARCH (x86_64), built with libpci 3.1.10, GCC 4.7.x-google 20130114 (prerelease), little endian w25q_disable_writeprotect(): error=1. FAILED *** Downloading latest available custom ROM for Peppy *** *** Already downloaded! *** --2014-08-19 17:42:40-- https://johnlewis.ie/Chromebook-ROMs/md5sums.txt Resolving johnlewis.ie (johnlewis.ie)... 37.187.3.40 Connecting to johnlewis.ie (johnlewis.ie)|37.187.3.40|:443... connected. HTTP request sent, awaiting response... 200 OK Length: 4941 (4.8K) [text/plain] Saving to: ‘md5sums.txt’ 100%[==============================================================================================================================>] 4,941 --.-K/s in 0s 2014-08-19 17:42:41 (479 MB/s) - ‘md5sums.txt’ saved [4941/4941] About to flash new ROM, repeat 'If this bricks my Chromebook/box, on my head be it!' observing exact case and punctuation: If this bricks my Chromebook/box, on my head be it! dd: failed to open ‘backup-190814.rom’: No such file or directory flashrom v0.9.4 : 14ce0cf : Oct 09 2013 22:16:28 UTC on Linux 3.16.1-1-ARCH (x86_64), built with libpci 3.1.10, GCC 4.7.x-google 20130114 (prerelease), little endian Block protection could not be disabled! Erasing and writing flash chip... spi_block_erase_20 failed during command execution at address 0x0 ERASE FAILED at 0x00600000! Expected=0xff, Read=0x5f, failed byte count from 0x00600000-0x00600fff: 0x148 ERASE FAILED! Reading current flash chip contents... spi_block_erase_52 failed during command execution at address 0x0 ERASE FAILED! Reading current flash chip contents... spi_block_erase_d8 failed during command execution at address 0x0 ERASE FAILED at 0x00600000! Expected=0xff, Read=0x5f, failed byte count from 0x00600000-0x0060ffff: 0x148 ERASE FAILED! Reading current flash chip contents... spi_chip_erase_60 failed during command execution ERASE FAILED! Reading current flash chip contents... spi_chip_erase_c7 failed during command execution ERASE FAILED! FAILED! Uh oh. Erase/write failed. Checking if anything changed. Your flash chip is in an unknown state. Get help on IRC at irc.freenode.net (channel #flashrom) or mail flashrom(a)flashrom.org with FAILED: your board name in the subject line! ------------------------------------------------------------------------------- DO NOT REBOOT OR POWEROFF! FAILED *** Assuming you didn't get any errors beyond the ubiquitous 'spi_block_erase_20 failed during command execution at address 0×0 Verifying flash… VERIFY FAILED at 0×00000062! Expected=0xff, Read=0x0b, failed byte count from 0×00000000-0x007fffff: 0×4' you may reboot. Otherwise, do not reboot under ANY circumstances, and seek help! ***

3 4

[patch] exit when the same filename is used for reading/writing flash chip content and to store log output
by Idwer Vollering 21 Aug '14

21 Aug '14

Tested on physical hardware, built on FreeBSD (less portability 'headaches'). Signed-off-By: Idwer Vollering <vidwer(a)gmail.com> Index: cli_output.c =================================================================== --- cli_output.c (revision 1846) +++ cli_output.c (working copy) @@ -52,8 +52,12 @@ msg_gerr("No logfile name specified.\n"); return 1; } - if ((logfile = fopen(filename, "w")) == NULL) { + if ((logfile = fopen(filename, "xw")) == NULL) { msg_gerr("Error: opening log file \"%s\" failed: %s\n", filename, strerror(errno)); + msg_gdbg("This means that flashrom can't write log output\n" \ + "to the file you specified (\"%s\"),\n" \ + "or you are using the same filename to read from/write to (-r / -w)\n" \ + "and write the log output to (-o).\n", filename); return 1; } return 0;

1 0

[PATCH] Add SUPPORT_EMAIL compiler flag.
by Stefan Tauner 21 Aug '14

21 Aug '14

Use it throughout the source to allow for printing alternative support email addresses (e.g. for distributions, chromiumos or other 3rd parties). Also, fix some indentions on the way. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> --- This of course only makes sense if someone uses it (and in the case of chromium port it to their tree... and set up some kind of community mailing list or something?). Therefore I would like to hear your thoughts about it. Would you use it? Makefile | 3 +++ bitbang_spi.c | 2 +- board_enable.c | 12 ++++++------ chipset_enable.c | 16 ++++++---------- cli_common.c | 2 +- dediprog.c | 10 +++++----- flashrom.c | 50 +++++++++++++++++++++----------------------------- jedec.c | 2 +- opaque.c | 2 +- pcidev.c | 10 +++++----- physmap.c | 6 ++---- print.c | 2 +- programmer.c | 2 +- sb600spi.c | 8 ++++---- sfdp.c | 5 ++--- spi.c | 6 +++--- spi25.c | 6 +++--- 17 files changed, 66 insertions(+), 78 deletions(-) diff --git a/Makefile b/Makefile index 98230a5..1e85f41 100644 --- a/Makefile +++ b/Makefile @@ -19,6 +19,7 @@ # PROGRAM = flashrom +SUPPORT_EMAIL = flashrom(a)flashrom.org ############################################################################### # Defaults for the toolchain. @@ -723,6 +724,8 @@ endif FEATURE_CFLAGS += $(shell LC_ALL=C grep -q "UTSNAME := yes" .features && printf "%s" "-D'HAVE_UTSNAME=1'") +FEATURE_CFLAGS += -D'SUPPORT_EMAIL="$(SUPPORT_EMAIL)"' + # We could use PULLED_IN_LIBS, but that would be ugly. FEATURE_LIBS += $(shell LC_ALL=C grep -q "NEEDLIBZ := yes" .libdeps && printf "%s" "-lz") diff --git a/bitbang_spi.c b/bitbang_spi.c index 0b27a67..9a0441c 100644 --- a/bitbang_spi.c +++ b/bitbang_spi.c @@ -94,7 +94,7 @@ int register_spi_bitbang_master(const struct bitbang_spi_master *master) (master->request_bus && !master->release_bus) || (!master->request_bus && master->release_bus)) { msg_perr("Incomplete SPI bitbang master setting!\n" - "Please report a bug at flashrom(a)flashrom.org\n"); + "Please report a bug at " SUPPORT_EMAIL "\n"); return ERROR_FLASHROM_BUG; } diff --git a/board_enable.c b/board_enable.c index 1235bb8..6606a59 100644 --- a/board_enable.c +++ b/board_enable.c @@ -335,7 +335,7 @@ void probe_superio_winbond(void) } msg_pinfo("Active config mode, unknown reg 0x20 ID: %02x.\n", model); msg_pinfo("Please send the output of \"flashrom -V -p internal\" to \n" - "flashrom(a)flashrom.org with W836xx: your board name: flashrom -V\n" + SUPPORT_EMAIL " with W836xx: your board name: flashrom -V\n" "as the subject to help us finish support for your Super I/O. Thanks.\n"); continue; } @@ -620,7 +620,7 @@ int it8705f_write_enable(uint8_t port) /* The data sheet contradicts itself about max size. */ max_rom_decode.parallel = 1024 * 1024; msg_pinfo("IT8705F with very unusual settings.\n" - "Please send the output of \"flashrom -V -p internal\" to flashrom(a)flashrom.org\n" + "Please send the output of \"flashrom -V -p internal\" to " SUPPORT_EMAIL "\n" "with \"IT8705: your board name: flashrom -V\" as the subject to help us finish\n" "support for your Super I/O. Thanks.\n"); ret = 1; @@ -2479,7 +2479,7 @@ int selfcheck_board_enables(void) const struct board_match *b = &board_matches[i]; if (b->vendor_name == NULL || b->board_name == NULL) { msg_gerr("ERROR: Board enable #%d does not define a vendor and board name.\n" - "Please report a bug at flashrom(a)flashrom.org\n", i); + "Please report a bug at " SUPPORT_EMAIL "\n", i); ret = 1; continue; } @@ -2488,7 +2488,7 @@ int selfcheck_board_enables(void) ((b->lb_vendor == NULL) ^ (b->lb_part == NULL)) || (b->max_rom_decode_parallel == 0 && b->enable == NULL)) { msg_gerr("ERROR: Board enable for %s %s is misdefined.\n" - "Please report a bug at flashrom(a)flashrom.org\n", + "Please report a bug at " SUPPORT_EMAIL "\n", b->vendor_name, b->board_name); ret = 1; } @@ -2553,7 +2553,7 @@ static const struct board_match *board_match_name(const char *vendor, const char if (partmatch) { /* More than one entry has a matching name. */ msg_perr("Board name \"%s\":\"%s\" and PCI IDs matched more than one board enable " - "entry. Please report a bug at flashrom(a)flashrom.org\n", vendor, model); + "entry. Please report a bug at " SUPPORT_EMAIL "\n", vendor, model); return NULL; } partmatch = board; @@ -2636,7 +2636,7 @@ static int board_enable_safetycheck(const struct board_match *board) return 1; } msg_pwarn("NOTE: Running an untested board enable procedure.\n" - "Please report success/failure to flashrom(a)flashrom.org.\n"); + "Please report success/failure to " SUPPORT_EMAIL ".\n"); return 0; } diff --git a/chipset_enable.c b/chipset_enable.c index dd4e0ea..8334866 100644 --- a/chipset_enable.c +++ b/chipset_enable.c @@ -1393,7 +1393,7 @@ static int enable_flash_mcp6x_7x(struct pci_dev *dev, const char *name) internal_buses_supported = BUS_NONE; msg_pwarn("Flash bus type is unknown (none)\n"); msg_pinfo("Please send the log files created by \"flashrom -p internal -o logfile\" to \n" - "flashrom(a)flashrom.org with \"your board name: flashrom -V\" as the subject to\n" + SUPPORT_EMAIL " with \"your board name: flashrom -V\" as the subject to\n" "help us finish support for your chipset. Thanks.\n"); return ERROR_NONFATAL; } @@ -1791,14 +1791,10 @@ int chipset_flash_enable(void) if (!dev) continue; if (ret != -2) { - msg_pwarn("Warning: unexpected second chipset match: " - "\"%s %s\"\n" - "ignoring, please report lspci and board URL " - "to flashrom(a)flashrom.org\n" - "with \'CHIPSET: your board name\' in the " - "subject line.\n", - chipset_enables[i].vendor_name, - chipset_enables[i].device_name); + msg_pwarn("Warning: unexpected second chipset match: \"%s %s\"\n" + "ignoring, please report lspci and board URL to " SUPPORT_EMAIL "\n" + "with \'CHIPSET: your board name\' in the subject line.\n", + chipset_enables[i].vendor_name, chipset_enables[i].device_name); continue; } msg_pinfo("Found chipset \"%s %s\"", @@ -1815,7 +1811,7 @@ int chipset_flash_enable(void) "flashrom *and* were (not) able to " "successfully update your firmware with it,\n" "then please email a report to " - "flashrom(a)flashrom.org including a verbose " + SUPPORT_EMAIL " including a verbose " "(-V) log.\nThank you!\n"); } msg_pinfo("Enabling flash write... "); diff --git a/cli_common.c b/cli_common.c index 71cc2dd..1cae546 100644 --- a/cli_common.c +++ b/cli_common.c @@ -107,7 +107,7 @@ void print_chip_support_status(const struct flashchip *chip) } 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" + "please email a report to " SUPPORT_EMAIL " if any of the above operations\n" "work correctly for you with this flash chip. Please include the flashrom log\n" "file for all operations you tested (see the man page for details), and mention\n" "which mainboard or programmer you tested in the subject line.\n" diff --git a/dediprog.c b/dediprog.c index 71a9f8b..676248b 100644 --- a/dediprog.c +++ b/dediprog.c @@ -219,8 +219,8 @@ static int dediprog_spi_bulk_read(struct flashctx *flash, uint8_t *buf, (chunksize >> 8) & 0xff}; if ((start % chunksize) || (len % chunksize)) { - msg_perr("%s: Unaligned start=%i, len=%i! Please report a bug " - "at flashrom(a)flashrom.org\n", __func__, start, len); + msg_perr("%s: Unaligned start=%i, len=%i! Please report a bug at " SUPPORT_EMAIL "\n", + __func__, start, len); return 1; } @@ -326,13 +326,13 @@ static int dediprog_spi_bulk_write(struct flashctx *flash, const uint8_t *buf, u */ if (chunksize != 256) { msg_perr("%s: Chunk sizes other than 256 bytes are unsupported, chunksize=%u!\n" - "Please report a bug at flashrom(a)flashrom.org\n", __func__, chunksize); + "Please report a bug at " SUPPORT_EMAIL "\n", __func__, chunksize); return 1; } if ((start % chunksize) || (len % chunksize)) { - msg_perr("%s: Unaligned start=%i, len=%i! Please report a bug " - "at flashrom(a)flashrom.org\n", __func__, start, len); + msg_perr("%s: Unaligned start=%i, len=%i!\nPlease report a bug at " SUPPORT_EMAIL "\n", + __func__, start, len); return 1; } diff --git a/flashrom.c b/flashrom.c index 93b292b..1148fd4 100644 --- a/flashrom.c +++ b/flashrom.c @@ -558,8 +558,7 @@ char *extract_param(const char *const *haystack, const char *needle, const char needlelen = strlen(needle); if (!needlelen) { - msg_gerr("%s: empty needle! Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + msg_gerr("%s: empty needle! Please report a bug at " SUPPORT_EMAIL "\n", __func__); return NULL; } /* No programmer parameters given. */ @@ -780,8 +779,7 @@ int need_erase(const uint8_t *have, const uint8_t *want, unsigned int len, enum result = 0; break; default: - msg_cerr("%s: Unsupported granularity! Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + msg_cerr("%s: Unsupported granularity! Please report a bug at " SUPPORT_EMAIL "\n", __func__); } return result; } @@ -842,8 +840,7 @@ static unsigned int get_next_write(const uint8_t *have, const uint8_t *want, uns stride = 1056; break; default: - msg_cerr("%s: Unsupported granularity! Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + msg_cerr("%s: Unsupported granularity! Please report a bug at " SUPPORT_EMAIL "\n", __func__); /* Claim that no write was needed. A write with unknown * granularity is too dangerous to try. */ @@ -1119,7 +1116,7 @@ int probe_flash(struct registered_master *mst, int startchip, struct flashctx *f msg_cinfo(" we need to add them manually.\n" "You can help us by mailing us the output of the following command to " - "flashrom@flashrom.org:\n" + SUPPORT_EMAIL ":\n" "'flashrom -VV [plus the -p/--programmer parameter]'\n" "Thanks for your help!\n" "===\n"); @@ -1278,19 +1275,17 @@ static int selfcheck_eraseblocks(const struct flashchip *chip) /* Blocks with zero size are bugs in flashchips.c. */ if (eraser.eraseblocks[i].count && !eraser.eraseblocks[i].size) { - msg_gerr("ERROR: Flash chip %s erase function " - "%i region %i has size 0. Please report" - " a bug at flashrom(a)flashrom.org\n", - chip->name, k, i); + msg_gerr("ERROR: Flash chip %s erase function %i region %i has size 0.\n" + "Please report a bug at " SUPPORT_EMAIL "\n", + chip->name, k, i); ret = 1; } /* Blocks with zero count are bugs in flashchips.c. */ if (!eraser.eraseblocks[i].count && eraser.eraseblocks[i].size) { - msg_gerr("ERROR: Flash chip %s erase function " - "%i region %i has count 0. Please report" - " a bug at flashrom(a)flashrom.org\n", - chip->name, k, i); + msg_gerr("ERROR: Flash chip %s erase function %i region %i has count 0.\n" + "Please report a bug at " SUPPORT_EMAIL "\n", + chip->name, k, i); ret = 1; } done += eraser.eraseblocks[i].count * @@ -1303,11 +1298,9 @@ static int selfcheck_eraseblocks(const struct flashchip *chip) if (!done) continue; if (done != chip->total_size * 1024) { - msg_gerr("ERROR: Flash chip %s erase function %i " - "region walking resulted in 0x%06x bytes total," - " expected 0x%06x bytes. Please report a bug at" - " flashrom(a)flashrom.org\n", chip->name, k, - done, chip->total_size * 1024); + msg_gerr("ERROR: Flash chip %s erase function %i region walking resulted in " + "0x%06x bytes total, expected 0x%06x bytes. Please report a bug at" + SUPPORT_EMAIL "\n", chip->name, k, done, chip->total_size * 1024); ret = 1; } if (!eraser.block_erase) @@ -1319,10 +1312,9 @@ static int selfcheck_eraseblocks(const struct flashchip *chip) for (j = k + 1; j < NUM_ERASEFUNCTIONS; j++) { if (eraser.block_erase == chip->block_erasers[j].block_erase) { - msg_gerr("ERROR: Flash chip %s erase function " - "%i and %i are identical. Please report" - " a bug at flashrom(a)flashrom.org\n", - chip->name, k, j); + msg_gerr("ERROR: Flash chip %s erase function %i and %i are identical.\n" + "Please report a bug at " SUPPORT_EMAIL "\n", + chip->name, k, j); ret = 1; } } @@ -1518,7 +1510,7 @@ static void nonfatal_help_message(void) if (programmer == PROGRAMMER_INTERNAL) msg_gerr("This means we have to add special support for your board, programmer or flash\n" "chip. Please report this on IRC at chat.freenode.net (channel #flashrom) or\n" - "mail flashrom(a)flashrom.org, thanks!\n" + "mail " SUPPORT_EMAIL ", thanks!\n" "-------------------------------------------------------------------------------\n" "You may now reboot or simply leave the machine running.\n"); else @@ -1526,7 +1518,7 @@ static void nonfatal_help_message(void) msg_gerr("Please check the connections (especially those to write protection pins) between\n" "the programmer and the flash chip. If you think the error is caused by flashrom\n" "please report this on IRC at chat.freenode.net (channel #flashrom) or\n" - "mail flashrom(a)flashrom.org, thanks!\n"); + "mail " SUPPORT_EMAIL ", thanks!\n"); } static void emergency_help_message(void) @@ -1535,13 +1527,13 @@ static void emergency_help_message(void) #if CONFIG_INTERNAL == 1 if (programmer == PROGRAMMER_INTERNAL) msg_gerr("Get help on IRC at chat.freenode.net (channel #flashrom) or\n" - "mail flashrom(a)flashrom.org with the subject \"FAILED: <your board name>\"!\n" + "mail " SUPPORT_EMAIL " with the subject \"FAILED: <your board name>\"!\n" "-------------------------------------------------------------------------------\n" "DO NOT REBOOT OR POWEROFF!\n"); else #endif msg_gerr("Please report this on IRC at chat.freenode.net (channel #flashrom) or\n" - "mail flashrom(a)flashrom.org, thanks!\n"); + "mail " SUPPORT_EMAIL ", thanks!\n"); } /* The way to go if you want a delimited list of programmers */ @@ -1755,7 +1747,7 @@ int selfcheck(void) if (chip->vendor == NULL || chip->name == NULL || chip->bustype == BUS_NONE) { ret = 1; msg_gerr("ERROR: Some field of flash chip #%d (%s) is misconfigured.\n" - "Please report a bug at flashrom(a)flashrom.org\n", i, + "Please report a bug at " SUPPORT_EMAIL "\n", i, chip->name == NULL ? "unnamed" : chip->name); } if (selfcheck_eraseblocks(chip)) { diff --git a/jedec.c b/jedec.c index 358b850..fa02c7b 100644 --- a/jedec.c +++ b/jedec.c @@ -669,7 +669,7 @@ static int changelock_regspace2_block(const struct flashctx *flash, chipaddr off if (new_bits & ~REG2_MASK) { msg_cerr("Invalid locking change 0x%02x requested at 0x%0*" PRIxPTR "! " - "Please report a bug at flashrom(a)flashrom.org\n", + "Please report a bug at " SUPPORT_EMAIL "\n", new_bits, PRIxPTR_WIDTH, offset); return -1; } diff --git a/opaque.c b/opaque.c index a5a829c..5bf6737 100644 --- a/opaque.c +++ b/opaque.c @@ -56,7 +56,7 @@ int register_opaque_master(const struct opaque_master *mst) if (!mst->probe || !mst->read || !mst->write || !mst->erase) { msg_perr("%s called with incomplete master definition. " - "Please report a bug at flashrom(a)flashrom.org\n", + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return ERROR_FLASHROM_BUG; } diff --git a/pcidev.c b/pcidev.c index 2c78063..6023438 100644 --- a/pcidev.c +++ b/pcidev.c @@ -146,7 +146,7 @@ uintptr_t pcidev_readbar(struct pci_dev *dev, int bar) addr &= PCI_ROM_ADDRESS_MASK; break; case TYPE_UNKNOWN: - msg_perr("BAR type unknown, please report a bug at flashrom(a)flashrom.org\n"); + msg_perr("BAR type unknown, please report a bug at " SUPPORT_EMAIL "\n"); } return (uintptr_t)addr; @@ -156,7 +156,7 @@ static int pcidev_shutdown(void *data) { if (pacc == NULL) { msg_perr("%s: Tried to cleanup an invalid PCI context!\n" - "Please report a bug at flashrom(a)flashrom.org\n", __func__); + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return 1; } pci_cleanup(pacc); @@ -167,7 +167,7 @@ int pci_init_common(void) { if (pacc != NULL) { msg_perr("%s: Tried to allocate a new PCI context, but there is still an old one!\n" - "Please report a bug at flashrom(a)flashrom.org\n", __func__); + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return 1; } pacc = pci_alloc(); /* Get the pci_access structure */ @@ -225,7 +225,7 @@ struct pci_dev *pcidev_init(const struct dev_entry *devs, int bar) if (devs[i].status == NT) msg_pinfo("===\nThis PCI device is UNTESTED. Please report the 'flashrom -p " "xxxx' output \n" - "to flashrom(a)flashrom.org if it works for you. Please add the name " + "to " SUPPORT_EMAIL " if it works for you. Please add the name " "of your\n" "PCI device to the subject. Thank you for your help!\n===\n"); @@ -274,7 +274,7 @@ int undo_pci_write(void *p) struct undo_pci_write_data *data = p; if (pacc == NULL) { msg_perr("%s: Tried to undo PCI writes without a valid PCI context!\n" - "Please report a bug at flashrom(a)flashrom.org\n", __func__); + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return 1; } msg_pdbg("Restoring PCI config space for %02x:%02x:%01x reg 0x%02x\n", diff --git a/physmap.c b/physmap.c index ca5afd0..6fb5e3b 100644 --- a/physmap.c +++ b/physmap.c @@ -308,8 +308,7 @@ void physunmap_unaligned(void *virt_addr, size_t len) { /* No need to check for zero size, such mappings would have yielded ERROR_PTR. */ if (virt_addr == ERROR_PTR) { - msg_perr("Trying to unmap a nonexisting mapping!\n" - "Please report a bug at flashrom(a)flashrom.org\n"); + msg_perr("Trying to unmap a nonexisting mapping!\nPlease report a bug at " SUPPORT_EMAIL "\n"); return; } @@ -322,8 +321,7 @@ void physunmap(void *virt_addr, size_t len) /* No need to check for zero size, such mappings would have yielded ERROR_PTR. */ if (virt_addr == ERROR_PTR) { - msg_perr("Trying to unmap a nonexisting mapping!\n" - "Please report a bug at flashrom(a)flashrom.org\n"); + msg_perr("Trying to unmap a nonexisting mapping!\nPlease report a bug at " SUPPORT_EMAIL "\n"); return; } tmp = (uintptr_t)virt_addr; diff --git a/print.c b/print.c index 243aa49..de9f85f 100644 --- a/print.c +++ b/print.c @@ -522,7 +522,7 @@ int print_supported(void) break; default: msg_gerr("\n%s: %s: Uninitialized programmer type! Please report a bug at " - "flashrom(a)flashrom.org\n", __func__, prog.name); + SUPPORT_EMAIL "\n", __func__, prog.name); break; } } diff --git a/programmer.c b/programmer.c index fd634b2..c88e9b5 100644 --- a/programmer.c +++ b/programmer.c @@ -103,7 +103,7 @@ int register_par_master(const struct par_master *mst, !mst->chip_writen || !mst->chip_readb || !mst->chip_readw || !mst->chip_readl || !mst->chip_readn) { msg_perr("%s called with incomplete master definition. " - "Please report a bug at flashrom(a)flashrom.org\n", + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return ERROR_FLASHROM_BUG; } diff --git a/sb600spi.c b/sb600spi.c index 69fad65..381254e 100644 --- a/sb600spi.c +++ b/sb600spi.c @@ -106,8 +106,8 @@ static void determine_generation(struct pci_dev *dev) msg_pdbg("SB8xx/SB9xx/Hudson-1 detected.\n"); } else { msg_pwarn("SB device found but SMBus revision 0x%02x does not match known values.\n" - "Assuming SB8xx/SB9xx/Hudson-1. Please send a log to flashrom(a)flashrom.org\n", - rev); + "Assuming SB8xx/SB9xx/Hudson-1. Please send a log to " SUPPORT_EMAIL "\n", + rev); amd_gen = CHIPSET_SB89XX; } } else if (dev->device_id == 0x780e) { @@ -146,13 +146,13 @@ static void determine_generation(struct pci_dev *dev) msg_pdbg("Yangtze detected.\n"); } else { msg_pwarn("FCH device found but SMBus revision 0x%02x does not match known values.\n" - "Please report this to flashrom(a)flashrom.org and include this log and\n" + "Please report this to " SUPPORT_EMAIL " and include this log and\n" "the output of lspci -nnvx, thanks!.\n", rev); } #endif } else msg_pwarn("%s: Unknown LPC device %" PRIx16 ":%" PRIx16 ".\n" - "Please report this to flashrom(a)flashrom.org and include this log and\n" + "Please report this to " SUPPORT_EMAIL " and include this log and\n" "the output of lspci -nnvx, thanks!\n", __func__, dev->vendor_id, dev->device_id); } diff --git a/sfdp.c b/sfdp.c index bc69dd0..ed22392 100644 --- a/sfdp.c +++ b/sfdp.c @@ -89,8 +89,7 @@ static int sfdp_add_uniform_eraser(struct flashchip *chip, uint8_t opcode, uint3 if (erasefn == NULL || total_size == 0 || block_size == 0 || total_size % block_size != 0) { - msg_cdbg("%s: invalid input, please report to " - "flashrom(a)flashrom.org\n", __func__); + msg_cdbg("%s: invalid input, please report to " SUPPORT_EMAIL "\n", __func__); return 1; } @@ -120,7 +119,7 @@ static int sfdp_add_uniform_eraser(struct flashchip *chip, uint8_t opcode, uint3 return 0; } msg_cinfo("%s: Not enough space to store another eraser (i=%d)." - " Please report this at flashrom(a)flashrom.org\n", + " Please report this at " SUPPORT_EMAIL "\n", __func__, i); return 1; } diff --git a/spi.c b/spi.c index 894f73f..102e9b5 100644 --- a/spi.c +++ b/spi.c @@ -82,7 +82,7 @@ int default_spi_read(struct flashctx *flash, uint8_t *buf, unsigned int start, if (max_data == MAX_DATA_UNSPECIFIED) { msg_perr("%s called, but SPI read chunk size not defined " "on this hardware. Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + SUPPORT_EMAIL "\n", __func__); return 1; } return spi_read_chunked(flash, buf, start, len, max_data); @@ -94,7 +94,7 @@ int default_spi_write_256(struct flashctx *flash, const uint8_t *buf, unsigned i if (max_data == MAX_DATA_UNSPECIFIED) { msg_perr("%s called, but SPI write chunk size not defined " "on this hardware. Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + "" SUPPORT_EMAIL "\n", __func__); return 1; } return spi_write_chunked(flash, buf, start, len, max_data); @@ -174,7 +174,7 @@ int register_spi_master(const struct spi_master *mst) ((mst->command == default_spi_send_command) && (mst->multicommand == default_spi_send_multicommand))) { msg_perr("%s called with incomplete master definition. " - "Please report a bug at flashrom(a)flashrom.org\n", + "Please report a bug at " SUPPORT_EMAIL "\n", __func__); return ERROR_FLASHROM_BUG; } diff --git a/spi25.c b/spi25.c index 673bdf5..5cb9632 100644 --- a/spi25.c +++ b/spi25.c @@ -835,7 +835,7 @@ erasefunc_t *spi_get_erasefn_from_opcode(uint8_t opcode) return &spi_block_erase_db; default: msg_cinfo("%s: unknown erase opcode (0x%02x). Please report " - "this at flashrom(a)flashrom.org\n", __func__, opcode); + "this at " SUPPORT_EMAIL "\n", __func__, opcode); return NULL; } } @@ -1101,7 +1101,7 @@ int default_spi_write_aai(struct flashctx *flash, const uint8_t *buf, unsigned i /* The data sheet requires a start address with the low bit cleared. */ if (start % 2) { msg_cerr("%s: start address not even! Please report a bug at " - "flashrom(a)flashrom.org\n", __func__); + "" SUPPORT_EMAIL "\n", __func__); if (spi_chip_write_1(flash, buf, start, start % 2)) return SPI_GENERIC_ERROR; pos += start % 2; @@ -1119,7 +1119,7 @@ int default_spi_write_aai(struct flashctx *flash, const uint8_t *buf, unsigned i /* The data sheet requires total AAI write length to be even. */ if (len % 2) { msg_cerr("%s: total write length not even! Please report a " - "bug at flashrom(a)flashrom.org\n", __func__); + "bug at " SUPPORT_EMAIL "\n", __func__); /* Do not return an error for now. */ //return SPI_GENERIC_ERROR; } -- Kind regards, Stefan Tauner

1 0

Added support for Macronix MX23L1654, MX23L6454, and MX23L12854
by Michael Coppola 20 Aug '14

20 Aug '14

Hi, I've added support for the Macronix MX23L1654, MX23L6454, and MX23L12854 SPI mask ROM chips. I do not physically have these chips and have not tested, but I used the template for the MX23L3254 which I have previously verified to work with a real chip. Datasheets: http://www.mxic.com.tw/Lists/DataSheet/Attachments/1216/MX23L1654,%203.0-3.… http://www.mxic.com.tw/Lists/DataSheet/Attachments/1234/MX23L6454,%203.0-3.… http://www.mxic.com.tw/Lists/DataSheet/Attachments/1232/MX23L12854,%203.0-3… Signed-off-by: Michael Coppola <michael.n.coppola(a)gmail.com>

2 1

[commit] r1846 - trunk
by repository service 20 Aug '14

20 Aug '14

Author: stefanct Date: Wed Aug 20 20:56:35 2014 New Revision: 1846 URL: http://flashrom.org/trac/flashrom/changeset/1846 Log: Add support for Macronix MX23L1654, MX23L6454 and MX23L12854 mask ROMs. Signed-off-by: Michael Coppola <michael.n.coppola(a)gmail.com> Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Modified: trunk/flashchips.c trunk/flashchips.h Modified: trunk/flashchips.c ============================================================================== --- trunk/flashchips.c Wed Aug 20 17:39:38 2014 (r1845) +++ trunk/flashchips.c Wed Aug 20 20:56:35 2014 (r1846) @@ -6322,6 +6322,22 @@ { .vendor = "Macronix", + .name = "MX23L1654", + .bustype = BUS_SPI, + .manufacture_id = MACRONIX_ID, + .model_id = MACRONIX_MX23L1654, + .total_size = 2048, + .page_size = 256, + .tested = {.probe = NT, .read = NT, .erase = NA, .write = NA}, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .write = NULL, /* MX23L1654 is a mask ROM, so it is read-only */ + .read = spi_chip_read, /* Fast read (0x0B) supported */ + .voltage = {3000, 3600}, + }, + + { + .vendor = "Macronix", .name = "MX23L3254", .bustype = BUS_SPI, .manufacture_id = MACRONIX_ID, @@ -6335,6 +6351,38 @@ .read = spi_chip_read, /* Fast read (0x0B) supported */ .voltage = {3000, 3600}, }, + + { + .vendor = "Macronix", + .name = "MX23L6454", + .bustype = BUS_SPI, + .manufacture_id = MACRONIX_ID, + .model_id = MACRONIX_MX23L6454, + .total_size = 8192, + .page_size = 256, + .tested = {.probe = NT, .read = NT, .erase = NA, .write = NA}, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .write = NULL, /* MX23L6454 is a mask ROM, so it is read-only */ + .read = spi_chip_read, /* Fast read (0x0B) supported */ + .voltage = {3000, 3600}, + }, + + { + .vendor = "Macronix", + .name = "MX23L12854", + .bustype = BUS_SPI, + .manufacture_id = MACRONIX_ID, + .model_id = MACRONIX_MX23L12854, + .total_size = 16384, + .page_size = 256, + .tested = {.probe = NT, .read = NT, .erase = NA, .write = NA}, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .write = NULL, /* MX23L12854 is a mask ROM, so it is read-only */ + .read = spi_chip_read, /* Fast read (0x0B) supported */ + .voltage = {3000, 3600}, + }, { .vendor = "Macronix", Modified: trunk/flashchips.h ============================================================================== --- trunk/flashchips.h Wed Aug 20 17:39:38 2014 (r1845) +++ trunk/flashchips.h Wed Aug 20 20:56:35 2014 (r1846) @@ -448,7 +448,10 @@ #define MACRONIX_ID 0xC2 /* Macronix (MX) */ /* Mask ROMs */ +#define MACRONIX_MX23L1654 0x0515 #define MACRONIX_MX23L3254 0x0516 +#define MACRONIX_MX23L6454 0x0517 +#define MACRONIX_MX23L12854 0x0518 /* MX25 chips are SPI, first byte of device ID is memory type, * second byte of device ID is log(bitsize)-9. * Generalplus SPI chips seem to be compatible with Macronix

1 0

Re: [flashrom] [PATCH 3/3] Add support for Intel Wildcat Point PCH.
by Stefan Tauner 20 Aug '14

20 Aug '14

Committed in r1845. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

Re: [flashrom] [PATCH 2/3] Add support for Intel Silvermont: Bay Trail, Rangeley and Avoton.
by Stefan Tauner 20 Aug '14

20 Aug '14

Committed in r1844 -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom