flashrom

flashrom@flashrom.org

5 participants
9161 discussions

[commit] r1407 - trunk
by repository service 06 Aug '11

06 Aug '11

Author: stefanct Date: Sat Aug 6 18:16:45 2011 New Revision: 1407 URL: http://flashrom.org/trac/flashrom/changeset/1407 Log: Fix the A8N-SLI Deluxe board enable Long story short: - the A8N in print.c should be A8N-SLI Deluxe as documented - the A8N-SLI Deluxe board enable needs a DMI pattern, else it will get executed on the A8N-SLI Premium unnecessarily. http://www.flashrom.org/pipermail/flashrom/2010-August/004310.html the DMI string is taken from an ubuntu bug report (a real dmidecode or flashrom output was not found). https://bugs.launchpad.net/ubuntu/+source/linux/+bug/807980 - the other A8N-SLI variants seem to work without the board enable A8N-SLI Premium: http://www.flashrom.org/pipermail/flashrom/2010-August/004310.html A8N-SLI: http://www.coreboot.org/pipermail/coreboot/2009-June/049107.html http://www.coreboot.org/pipermail/coreboot/2009-May/049002.html Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/board_enable.c trunk/print.c Modified: trunk/board_enable.c ============================================================================== --- trunk/board_enable.c Sat Aug 6 18:16:33 2011 (r1406) +++ trunk/board_enable.c Sat Aug 6 18:16:45 2011 (r1407) @@ -2001,7 +2001,7 @@ {0x1002, 0x4372, 0x103c, 0x2a26, 0x1002, 0x4377, 0x103c, 0x2a26, NULL, NULL, NULL, P3, "ASUS", "A8AE-LE", 0, OK, amd_sbxxx_gpio9_raise}, {0x8086, 0x27A0, 0x1043, 0x1287, 0x8086, 0x27DF, 0x1043, 0x1287, "^A8J", NULL, NULL, P3, "ASUS", "A8Jm", 0, NT, intel_ich_gpio34_raise}, {0x10DE, 0x0260, 0x103c, 0x2a3e, 0x10DE, 0x0264, 0x103c, 0x2a3e, "NAGAMI2L", NULL, NULL, P3, "ASUS", "A8N-LA (Nagami-GL8E)", 0, OK, nvidia_mcp_gpio0_raise}, - {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, "^A8N-SLI", NULL, NULL, P3, "ASUS", "A8N-SLI Deluxe", 0, NT, board_shuttle_fn25}, + {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, "^A8N-SLI DELUXE", NULL, NULL, P3, "ASUS", "A8N-SLI Deluxe", 0, NT, board_shuttle_fn25}, {0x10de, 0x0264, 0x1043, 0x81bc, 0x10de, 0x02f0, 0x1043, 0x81cd, NULL, NULL, NULL, P3, "ASUS", "A8N-VM CSM", 0, OK, w83627ehf_gpio22_raise_2e}, {0x10DE, 0x0264, 0x1043, 0x81C0, 0x10DE, 0x0260, 0x1043, 0x81C0, NULL, NULL, NULL, P3, "ASUS", "M2NBP-VM CSM", 0, OK, nvidia_mcp_gpio0_raise}, {0x1106, 0x1336, 0x1043, 0x80ed, 0x1106, 0x3288, 0x1043, 0x8249, NULL, NULL, NULL, P3, "ASUS", "M2V-MX", 0, OK, via_vt823x_gpio5_raise}, Modified: trunk/print.c ============================================================================== --- trunk/print.c Sat Aug 6 18:16:33 2011 (r1406) +++ trunk/print.c Sat Aug 6 18:16:45 2011 (r1407) @@ -420,10 +420,10 @@ B("ASUS", "A7V8X-MX SE", 1, "http://www.asus.com/product.aspx?P_ID=1guVBT1qV5oqhHyZ", NULL), B("ASUS", "A7V8X-X", 1, "http://www.asus.com/product.aspx?P_ID=YcXfRrWHZ9RKoVmw", NULL), B("ASUS", "A8Jm", 1, "http://www.asus.com/product.aspx?P_ID=VztICtOgiU6drx4m", NULL), - B("ASUS", "A8N", 1, NULL, NULL), /* TODO: This should probably be A8N-SLI Deluxe, see http://www.coreboot.org/pipermail/flashrom/2009-November/000878.html */ B("ASUS", "A8N-E", 1, "http://www.asus.com/product.aspx?P_ID=DzbA8hgqchMBOVRz", NULL), B("ASUS", "A8N-LA (Nagami-GL8E)", 1, "http://h10025.www1.hp.com/ewfrf/wc/document?lc=en&cc=us&docname=c00647121&d…", "This is an OEM board from HP, the HP name is Nagami-GL8E."), B("ASUS", "A8N-SLI", 1, "http://www.asus.com/product.aspx?P_ID=J9FKa8z2xVId3pDK", NULL), + B("ASUS", "A8N-SLI Deluxe", 0, NULL, "Untested board enable."), B("ASUS", "A8N-SLI Premium", 1, "http://www.asus.com/product.aspx?P_ID=nbulqxniNmzf0mH1", NULL), B("ASUS", "A8N-VM", 1, "http://www.asus.com/Motherboards/AMD_Socket_939/A8NVM/", NULL), B("ASUS", "A8N-VM CSM", 1, "http://www.asus.com/product.aspx?P_ID=JBqqlpj4cspbSa3s", NULL),

1 0

[commit] r1406 - trunk
by repository service 06 Aug '11

06 Aug '11

Author: stefanct Date: Sat Aug 6 18:16:33 2011 New Revision: 1406 URL: http://flashrom.org/trac/flashrom/changeset/1406 Log: Fix the EP-8NPA7I (for the last time hopefully) - retag it as OK (tested by Jonathan Kollasch when he wrote it) http://patchwork.coreboot.org/patch/2106/ - add a line with identical pci ids but a different dmi pattern, so that EP-9NPA7I is also matched. combining multiple boards in one line is problematic due to print.c's detection of board enables - so dont bother for now. http://www.flashrom.org/pipermail/flashrom/2011-June/006878.html See previous commit for additional information. Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/board_enable.c trunk/print.c Modified: trunk/board_enable.c ============================================================================== --- trunk/board_enable.c Sat Aug 6 14:45:21 2011 (r1405) +++ trunk/board_enable.c Sat Aug 6 18:16:33 2011 (r1406) @@ -2027,7 +2027,8 @@ {0x10de, 0x03ea, 0x1019, 0x2602, 0x10de, 0x03e0, 0x1019, 0x2602, NULL, NULL, NULL, P3, "Elitegroup", "GeForce6100SM-M", 0, OK, board_ecs_geforce6100sm_m}, {0x1106, 0x3038, 0x1019, 0x0996, 0x1106, 0x3177, 0x1019, 0x0996, NULL, NULL, NULL, P3, "Elitegroup", "K7VTA3", 256, OK, NULL}, {0x1106, 0x3177, 0x1106, 0x3177, 0x1106, 0x3059, 0x1695, 0x3005, NULL, NULL, NULL, P3, "EPoX", "EP-8K5A2", 0, OK, w836xx_memw_enable_2e}, - {0x10DE, 0x005E, 0x1695, 0x1010, 0x10DE, 0x0050, 0x1695, 0x1010, "8NPA7I", NULL, NULL, P3, "EPoX", "EP-8NPA7I", 0, NT, nvidia_mcp_gpio4_raise}, /* needs http://patchwork.coreboot.org/patch/2125/ */ + {0x10DE, 0x005E, 0x1695, 0x1010, 0x10DE, 0x0050, 0x1695, 0x1010, "8NPA7I", NULL, NULL, P3, "EPoX", "EP-8NPA7I", 0, OK, nvidia_mcp_gpio4_raise}, + {0x10DE, 0x005E, 0x1695, 0x1010, 0x10DE, 0x0050, 0x1695, 0x1010, "9NPA7I", NULL, NULL, P3, "EPoX", "EP-9NPA7I", 0, OK, nvidia_mcp_gpio4_raise}, {0x10EC, 0x8139, 0x1695, 0x9001, 0x11C1, 0x5811, 0x1695, 0x9015, NULL, NULL, NULL, P3, "EPoX", "EP-8RDA3+", 0, OK, nvidia_mcp_gpio31_raise}, {0x8086, 0x7110, 0, 0, 0x8086, 0x7190, 0, 0, NULL, "epox", "ep-bx3", P3, "EPoX", "EP-BX3", 0, NT, intel_piix4_gpo22_raise}, {0x10de, 0x02f0, 0x105b, 0x0d01, 0x10de, 0x0264, 0x105b, 0x0d01, NULL, NULL, NULL, P3, "Foxconn", "6150K8MD-8EKRSH", 0, NT, nvidia_mcp_gpio2_raise}, Modified: trunk/print.c ============================================================================== --- trunk/print.c Sat Aug 6 14:45:21 2011 (r1405) +++ trunk/print.c Sat Aug 6 18:16:33 2011 (r1406) @@ -482,7 +482,7 @@ B("ASUS", "P5KC", 1, "http://www.asus.com/product.aspx?P_ID=fFZ8oUIGmLpwNMjj", NULL), B("ASUS", "P5L-MX", 1, "http://www.asus.com/product.aspx?P_ID=X70d3NCzH2DE9vWH", NULL), B("ASUS", "P5GD1 Pro", 1, "http://www.asus.com/product.aspx?P_ID=50M49xQh71EZOeM1", NULL), - B("ASUS", "P5N-E SLI", 0, "http://www.asus.com/product.aspx?P_ID=KyHOsOKWujC2QguJ", "Needs a patch (http://patchwork.coreboot.org/patch/2125/) and a board enable (http://patchwork.coreboot.org/patch/3298/)."), + B("ASUS", "P5N-E SLI", 0, "http://www.asus.com/product.aspx?P_ID=KyHOsOKWujC2QguJ", "Needs a board enable (http://patchwork.coreboot.org/patch/3298/)."), B("ASUS", "P5N32-E SLI", 1, "http://www.asus.com/product.aspx?P_ID=vBZLIBtPzYB2bLcb", NULL), B("ASUS", "P5ND2-SLI Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=WY7XroDuUImVbgp5", NULL), B("ASUS", "P5PE-VM", 1, "http://www.asus.com/product.aspx?P_ID=k3h0ZFVu9Lo1dUvk", NULL), @@ -512,8 +512,8 @@ B("Elitegroup", "RS485M-M", 1, "http://www.ecs.com.tw/ECSWebSite_2007/Products/ProductsDetail.aspx?Category…", NULL), B("Emerson", "ATCA-7360", 1, "http://www.emerson.com/sites/Network_Power/en-US/Products/Product_Detail/Pr…", NULL), B("EPoX", "EP-8K5A2", 1, "http://www.epox.com/product.asp?ID=EP-8K5A2", NULL), - B("EPoX", "EP-8NPA7I", 0, "http://www.epox.com/product.asp?ID=EP-8NPA7I", "Needs a patch (http://patchwork.coreboot.org/patch/2125/)."), - B("EPoX", "EP-9NPA7I", 0, "http://www.epox.com/product.asp?ID=EP-9NPA7I", "Needs a patch (http://patchwork.coreboot.org/patch/2125/) and the same board enable as the EP-8NPA7I."), + B("EPoX", "EP-8NPA7I", 1, "http://www.epox.com/product.asp?ID=EP-8NPA7I", NULL), + B("EPoX", "EP-9NPA7I", 1, "http://www.epox.com/product.asp?ID=EP-9NPA7I", NULL), B("EPoX", "EP-8RDA3+", 1, "http://www.epox.com/product.asp?ID=EP-8RDA3plus", NULL), B("EPoX", "EP-BX3", 1, "http://www.epox.com/product.asp?ID=EP-BX3", NULL), B("EVGA", "132-CK-NF78", 1, "http://www.evga.com/articles/385.asp", NULL),

1 0

[commit] r1405 - trunk
by repository service 06 Aug '11

06 Aug '11

Author: stefanct Date: Sat Aug 6 14:45:21 2011 New Revision: 1405 URL: http://flashrom.org/trac/flashrom/changeset/1405 Log: Clear byte 0x92 of the LPC bridge for all CK804 (and MCP51) chipsets The OEM BIOS on the EPoX EP-8PA7I and a number of other boards clear byte 0x92 in the LPC bridge configuration space. Do the same for all CK804 chips, assuming this to be some sort of chipset-generic write-enable. Currently the same chipset enable is used for MCP51 (nForce 430). There have been reports of successful writes with its variations (e.g. A8N-LA (Nagami-GL8E)), but they were not tagged as OK. Due to the new "unsupported chipset"-message we will get success reports in the case this patch does not break anything on the MCP51-based boards. See also: http://www.flashrom.org/pipermail/flashrom/2011-July/007252.html http://patchwork.coreboot.org/patch/3176/ Signed-off-by: Jonathan Kollasch <jakllsch(a)kollasch.net> Acked-by: Joshua Roys <roysjosh(a)gmail.com> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/chipset_enable.c Modified: trunk/chipset_enable.c ============================================================================== --- trunk/chipset_enable.c Thu Aug 4 19:40:25 2011 (r1404) +++ trunk/chipset_enable.c Sat Aug 6 14:45:21 2011 (r1405) @@ -801,6 +801,12 @@ { uint8_t old, new; + pci_write_byte(dev, 0x92, 0x00); + if (pci_read_byte(dev, 0x92) != 0x00) { + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x92, 0x00, name); + } + old = pci_read_byte(dev, 0x88); new = old | 0xc0; if (new != old) {

1 0

[commit] r1404 - trunk
by repository service 04 Aug '11

04 Aug '11

Author: stefanct Date: Thu Aug 4 19:40:25 2011 New Revision: 1404 URL: http://flashrom.org/trac/flashrom/changeset/1404 Log: Introduce msg_*dbg2 Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Modified: trunk/cli_output.c trunk/flash.h trunk/flashrom.8 Modified: trunk/cli_output.c ============================================================================== --- trunk/cli_output.c Thu Aug 4 19:37:58 2011 (r1403) +++ trunk/cli_output.c Thu Aug 4 19:40:25 2011 (r1404) @@ -33,6 +33,9 @@ output_type = stderr; break; case MSG_BARF: + if (verbose < 3) + return 0; + case MSG_DEBUG2: if (verbose < 2) return 0; case MSG_DEBUG: Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Thu Aug 4 19:37:58 2011 (r1403) +++ trunk/flash.h Thu Aug 4 19:40:25 2011 (r1404) @@ -233,7 +233,8 @@ #define MSG_ERROR 0 #define MSG_INFO 1 #define MSG_DEBUG 2 -#define MSG_BARF 3 +#define MSG_DEBUG2 3 +#define MSG_BARF 4 #define msg_gerr(...) print(MSG_ERROR, __VA_ARGS__) /* general errors */ #define msg_perr(...) print(MSG_ERROR, __VA_ARGS__) /* programmer errors */ #define msg_cerr(...) print(MSG_ERROR, __VA_ARGS__) /* chip errors */ @@ -243,6 +244,9 @@ #define msg_gdbg(...) print(MSG_DEBUG, __VA_ARGS__) /* general debug */ #define msg_pdbg(...) print(MSG_DEBUG, __VA_ARGS__) /* programmer debug */ #define msg_cdbg(...) print(MSG_DEBUG, __VA_ARGS__) /* chip debug */ +#define msg_gdbg2(...) print(MSG_DEBUG2, __VA_ARGS__) /* general debug2 */ +#define msg_pdbg2(...) print(MSG_DEBUG2, __VA_ARGS__) /* programmer debug2 */ +#define msg_cdbg2(...) print(MSG_DEBUG2, __VA_ARGS__) /* chip debug2 */ #define msg_gspew(...) print(MSG_BARF, __VA_ARGS__) /* general debug barf */ #define msg_pspew(...) print(MSG_BARF, __VA_ARGS__) /* programmer debug barf */ #define msg_cspew(...) print(MSG_BARF, __VA_ARGS__) /* chip debug barf */ Modified: trunk/flashrom.8 ============================================================================== --- trunk/flashrom.8 Thu Aug 4 19:37:58 2011 (r1403) +++ trunk/flashrom.8 Thu Aug 4 19:40:25 2011 (r1404) @@ -77,8 +77,8 @@ .TP .B "\-V, \-\-verbose" More verbose output. This option can be supplied multiple times -(max. 2 times, i.e. -.BR \-VV ) +(max. 3 times, i.e. +.BR \-VVV ) for even more debug output. .TP .B "\-c, \-\-chip" <chipname>

1 0

[commit] r1403 - trunk
by repository service 04 Aug '11

04 Aug '11

Author: stefanct Date: Thu Aug 4 19:37:58 2011 New Revision: 1403 URL: http://flashrom.org/trac/flashrom/changeset/1403 Log: Rephrase warnings in chipset_enable.c Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/chipset_enable.c Modified: trunk/chipset_enable.c ============================================================================== --- trunk/chipset_enable.c Mon Aug 1 20:39:28 2011 (r1402) +++ trunk/chipset_enable.c Thu Aug 4 19:37:58 2011 (r1403) @@ -77,8 +77,8 @@ rpci_write_byte(dev, 0x40, new); newer = pci_read_byte(dev, 0x40); if (newer != new) { - msg_pinfo("tried to set register 0x%x to 0x%x on %s " - "failed (WARNING ONLY)\n", 0x40, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x40, new, name); msg_pinfo("Stuck at 0x%x\n", newer); return -1; } @@ -166,8 +166,8 @@ rpci_write_byte(sbdev, 0x45, new); newer = pci_read_byte(sbdev, 0x45); if (newer != new) { - msg_pinfo("tried to set register 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x45, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x45, new, name); msg_pinfo("Stuck at 0x%x\n", newer); ret = -1; } @@ -193,8 +193,8 @@ rpci_write_byte(sbdev, 0x45, new); newer = pci_read_byte(sbdev, 0x45); if (newer != new) { - msg_pinfo("tried to set register 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x45, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x45, new, name); msg_pinfo("Stuck at 0x%x\n", newer); ret = -1; } @@ -241,8 +241,8 @@ rpci_write_word(dev, xbcs, new); if (pci_read_word(dev, xbcs) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", xbcs, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", xbcs, new, name); return -1; } @@ -287,8 +287,8 @@ rpci_write_byte(dev, bios_cntl, new); if (pci_read_byte(dev, bios_cntl) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", bios_cntl, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", bios_cntl, new, name); return -1; } @@ -674,8 +674,8 @@ new = pci_read_byte(dev, 0x52); if (new != 0xee) { - msg_pinfo("tried to set register 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x52, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x52, new, name); return -1; } @@ -693,8 +693,8 @@ if (new != old) { rpci_write_byte(dev, 0x43, new); if (pci_read_byte(dev, 0x43) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x43, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x43, new, name); } } @@ -706,8 +706,8 @@ rpci_write_byte(dev, 0x40, new); if (pci_read_byte(dev, 0x40) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x40, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x40, new, name); return -1; } @@ -806,8 +806,8 @@ if (new != old) { rpci_write_byte(dev, 0x88, new); if (pci_read_byte(dev, 0x88) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x88, new, name); + msg_pinfo("Setting register to set 0x%x to 0x%x on %s " + "failed (WARNING ONLY).\n", 0x88, new, name); } } @@ -818,8 +818,8 @@ rpci_write_byte(dev, 0x6d, new); if (pci_read_byte(dev, 0x6d) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x6d, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x6d, new, name); return -1; } @@ -902,8 +902,8 @@ rpci_write_byte(dev, 0x6d, new); if (pci_read_byte(dev, 0x6d) != new) { - msg_pinfo("tried to set 0x%x to 0x%x on %s failed " - "(WARNING ONLY)\n", 0x6d, new, name); + msg_pinfo("Setting register 0x%x to 0x%x on %s failed " + "(WARNING ONLY).\n", 0x6d, new, name); return -1; }

1 0

A8M2N-LA flash fail fail
by Márton Miklós 03 Aug '11

03 Aug '11

Hello all, I have just tried to flash my BIOS using the flashrom, and I have ran into the following situation: lucifer@lucifer-desktop:~/Letöltések/flashrom-0.9.4$ sudo ./flashrom -w /tmp/sp35111/3.10 flashrom v0.9.4-r1395 on Linux 2.6.35-28-generic (i686), built with libpci 3.1.7, GCC 4.4.5, little endian flashrom is free software, get the source code at http://www.flashrom.org Calibrating delay loop... OK. Found chipset "NVIDIA MCP51". This chipset is marked as untested. If you are using an up-to-date version of flashrom please email a report to flashrom(a)flashrom.org including a verbose (-V) log. Thank you! Enabling flash write... OK. This chipset supports the following protocols: Non-SPI. Found PMC flash chip "Pm49FL004" (512 kB, LPC, FWH) at physical address 0xfff80000. Flash image seems to be a legacy BIOS. Disabling coreboot-related checks. Reading old flash chip contents... done. Erasing and writing flash chip... ERASE FAILED at 0x00000000! Expected=0xff, Read=0x23, failed byte count from 0x00000000-0x00000fff: 0xfec ERASE FAILED! Reading current flash chip contents... done. ERASE FAILED at 0x00000000! Expected=0xff, Read=0x23, failed byte count from 0x00000000-0x0000ffff: 0xfef9 ERASE FAILED! Reading current flash chip contents... done. ERASE FAILED at 0x00000000! Expected=0xff, Read=0x23, failed byte count from 0x00000000-0x0007ffff: 0x6494b ERASE FAILED! FAILED! Uh oh. Erase/write failed. Checking if anything changed. Good. It seems nothing was changed. Writing to the flash chip apparently didn't do anything. This means we have to add special support for your board, programmer or flash chip. Please report this on IRC at irc.freenode.net (channel #flashrom) or mail flashrom(a)flashrom.org! ------------------------------------------------------------------------------- You may now reboot or simply leave the machine running. Flashrom info: lucifer@lucifer-desktop:~/Letöltések/flashrom-0.9.4$ sudo ./flashrom flashrom v0.9.4-r1395 on Linux 2.6.35-28-generic (i686), built with libpci 3.1.7, GCC 4.4.5, little endian flashrom is free software, get the source code at http://www.flashrom.org Calibrating delay loop... OK. Found chipset "NVIDIA MCP51". This chipset is marked as untested. If you are using an up-to-date version of flashrom please email a report to flashrom(a)flashrom.org including a verbose (-V) log. Thank you! Enabling flash write... OK. This chipset supports the following protocols: Non-SPI. Found PMC flash chip "Pm49FL004" (512 kB, LPC, FWH) at physical address 0xfff80000. No operations were specified. I have tried the latest revision (1402) but it did not works too. Let me know if I should provide any additional information. Regards, Miklós Márton

1 0

Re: [flashrom] [PATCH] Add deferred -i processing
by Stefan Tauner 03 Aug '11

03 Aug '11

On Tue, 2 Aug 2011 14:48:15 -0700 David Hendricks <dhendrix(a)google.com> wrote: > The patch is mostly ack-able, save for a few nits. > > On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < > stefan.tauner(a)student.tuwien.ac.at> wrote: > > > My implementation does not defer the processing until doit(), but after the > > argument parsing loop only (doit() should not contain argument checks). > > > > Just FYI -- The reason we deferred processing until doit() was because our > usage case calls for reading the ROM content to find pre-existing mappings > (e.g. fmap) which exist in the ROM image. It might be useful to eventually > do the same thing for cbfs and Intel's flash descriptor. jup i am aware of that. but i dont like to inflate doit() with even more stuff, that does not belong there. we need to think about refactoring the whole processing... we need something like pre-doit() before doit() but after init and unlocking. a bit like the shutdown process with the registration of some callbacks... i hope work on libflashrom will enforce us to move forward with that. > On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < > stefan.tauner(a)student.tuwien.ac.at> wrote: > > > This allows to specify -i and -l parameters in any order. > > > > I don't like the output in error cases much, any ideas? > > example of a good run: > > flashrom -p dummy:emulate=SST25VF032B -i normal -w > > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback > > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > > libpci 3.1.7, GCC 4.4.5, little endian > > flashrom is free software, get the source code at http://www.flashrom.org > > > > Using region(s): "normal", "gfxrom", "fallback". > > Calibrating delay loop... OK. > > Found SST flash chip "SST25VF032B" (4096 kB, SPI) on dummy. > > Reading old flash chip contents... done. > > Erasing and writing flash chip... Erase/write done. > > Verifying flash... VERIFIED. > > > > Looks okay to me. And IIRC -V will list all ROM layout entries found even if > they are not included, which is nice for debugging. that was printed at default verbosity before, and is now spew-verbose (which is now at -VVV level). but yes i wanted to keep something for debugging. > > On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < > stefan.tauner(a)student.tuwien.ac.at> wrote: > > > example of a semi-good run: > > flashrom -p dummy:emulate=SST25VF032B -i normal -w > > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback > > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > > libpci 3.1.7, GCC 4.4.5, little endian > > flashrom is free software, get the source code at http://www.flashrom.org > > > > Maximum number of ROM images (4) in layout file reached before end of > > layout file. > > Ignoring the rest of the layout file. > > > > I think that if the user accidentally specifies an invalid layout or number > of included regions, then Flashrom should quit. Otherwise Flashrom might not > flash everything the user intends, leaving the content in an inconsistent > (probably bricked) state. This has become problematic due to complicated ROM > layouts in systems where multiple masters use a shared SPI flash (e.g. an EC > or Intel's ME insanity). yes, that's certainly a well argued proposal imo. carl-daniel? > On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < > stefan.tauner(a)student.tuwien.ac.at> wrote: > > > example of a faulty run: > > flashrom -p dummy:emulate=SST25VF032B -i normal -w > > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback -i bios > > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > > libpci 3.1.7, GCC 4.4.5, little endian > > flashrom is free software, get the source code at http://www.flashrom.org > > > > Maximum number of ROM images (4) in layout file reached before end of > > layout file. > > Ignoring the rest of the layout file. > > Using region(s): "normal", "gfxrom", "fallback"Invalid region specified: > > "bios" > > > > If you split that last line so that the "Invalid region" part shows up on a > newline, it looks okay. that is one option yes. but i tend to think we may defer the output of the used regions and push it into its own loop at the end of process_include_args. this was there would be no "using region..." output in case of errors at all. and it may even be more readable(?) > > Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> > > --- > > cli_classic.c | 11 ++++----- > > flash.h | 2 + > > layout.c | 68 > > ++++++++++++++++++++++++++++++++++++++++++++++++++------ > > 3 files changed, 67 insertions(+), 14 deletions(-) > > > > diff --git a/cli_classic.c b/cli_classic.c > > index 36fe9ad..e44ea86 100644 > > --- a/cli_classic.c > > +++ b/cli_classic.c > > @@ -233,14 +233,9 @@ int cli_classic(int argc, char *argv[]) > > cli_classic_abort_usage(); > > break; > > case 'i': > > - /* FIXME: -l has to be specified before -i. */ > > tempstr = strdup(optarg); > > - if (find_romentry(tempstr) < 0) { > > - fprintf(stderr, "Error: image %s not found > > in " > > - "layout file or -i specified before > > " > > - "-l\n", tempstr); > > + if (register_include_arg(tempstr) < 0) > > cli_classic_abort_usage(); > > - } > > break; > > case 'L': > > if (++operation_specified > 1) { > > @@ -325,6 +320,10 @@ int cli_classic(int argc, char *argv[]) > > cli_classic_abort_usage(); > > } > > > > + if (process_include_args() < 0) { > > + cli_classic_abort_usage(); > > + } > > + > > /* FIXME: Print the actions flashrom will take. */ > > > > if (list_supported) { > > diff --git a/flash.h b/flash.h > > index 5b49e9d..0848255 100644 > > --- a/flash.h > > +++ b/flash.h > > @@ -251,6 +251,8 @@ int print(int type, const char *fmt, ...) > > __attribute__((format(printf, 2, 3))); > > int cli_classic(int argc, char *argv[]); > > > > /* layout.c */ > > +int register_include_arg(char *name); > > +int process_include_args(void); > > int read_romlayout(char *name); > > int find_romentry(char *name); > > int handle_romentries(struct flashchip *flash, uint8_t *oldcontents, > > uint8_t *newcontents); > > diff --git a/layout.c b/layout.c > > index d719a05..936e316 100644 > > --- a/layout.c > > +++ b/layout.c > > @@ -41,6 +41,12 @@ typedef struct { > > char name[256]; > > } romlayout_t; > > > > +/* include_args lists arguments specified at the command line with -i. > > They > > + * must be processed at some point so that desired regions are marked as > > + * "included" in the rom_entries list. > > + */ > > +static char *include_args[MAX_ROMLAYOUT]; > > +static int num_include_args = 0; /* the number of valid entries. */ > > static romlayout_t rom_entries[MAX_ROMLAYOUT]; > > > > #if CONFIG_INTERNAL == 1 /* FIXME: Move the whole block to cbtable.c? */ > > @@ -194,6 +200,20 @@ int read_romlayout(char *name) > > } > > #endif > > > > +/* register an include argument (-i) for later processing */ > > +int register_include_arg(char *name) > > +{ > > + if (num_include_args >= MAX_ROMLAYOUT) { > > + msg_gerr("Too many regions included (%i).\n", > > num_include_args); > > + return -1; > > + } > > + > > + include_args[num_include_args] = name; > > + num_include_args++; > > + return num_include_args; > > +} > > + > > +/* returns the index of the entry (or a negative value if it is not found) > > */ > > int find_romentry(char *name) > > { > > int i; > > @@ -201,20 +221,49 @@ int find_romentry(char *name) > > if (!romimages) > > return -1; > > > > - msg_ginfo("Looking for \"%s\"... ", name); > > - > > + msg_gspew("Looking for region \"%s\"... ", name); > > for (i = 0; i < romimages; i++) { > > if (!strcmp(rom_entries[i].name, name)) { > > rom_entries[i].included = 1; > > - msg_ginfo("found.\n"); > > + msg_gspew("found.\n"); > > return i; > > } > > } > > - msg_ginfo("not found.\n"); // Not found. Error. > > - > > + msg_gspew("not found.\n"); > > return -1; > > } > > > > +/* process -i arguments > > + * returns 0 to indicate success, <0 to indicate failure > > > > I think the usual Flashrom convention is to return 1 to indicate failure. I > used <0 out of habit. Let's go ahead and fix that coding convention for > upstream if desired, and update the call sites to do error checking > correctly. well -1 is also used very often, but that's no argument to introduce more inconsistencies :) i'll fix this, thanks. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

Re: [flashrom] [PATCH] Add deferred -i processing
by David Hendricks 02 Aug '11

02 Aug '11

The patch is mostly ack-able, save for a few nits. On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < stefan.tauner(a)student.tuwien.ac.at> wrote: > My implementation does not defer the processing until doit(), but after the > argument parsing loop only (doit() should not contain argument checks). > Just FYI -- The reason we deferred processing until doit() was because our usage case calls for reading the ROM content to find pre-existing mappings (e.g. fmap) which exist in the ROM image. It might be useful to eventually do the same thing for cbfs and Intel's flash descriptor. On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < stefan.tauner(a)student.tuwien.ac.at> wrote: > This allows to specify -i and -l parameters in any order. > > I don't like the output in error cases much, any ideas? > example of a good run: > flashrom -p dummy:emulate=SST25VF032B -i normal -w > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > libpci 3.1.7, GCC 4.4.5, little endian > flashrom is free software, get the source code at http://www.flashrom.org > > Using region(s): "normal", "gfxrom", "fallback". > Calibrating delay loop... OK. > Found SST flash chip "SST25VF032B" (4096 kB, SPI) on dummy. > Reading old flash chip contents... done. > Erasing and writing flash chip... Erase/write done. > Verifying flash... VERIFIED. > Looks okay to me. And IIRC -V will list all ROM layout entries found even if they are not included, which is nice for debugging. On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < stefan.tauner(a)student.tuwien.ac.at> wrote: > example of a semi-good run: > flashrom -p dummy:emulate=SST25VF032B -i normal -w > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > libpci 3.1.7, GCC 4.4.5, little endian > flashrom is free software, get the source code at http://www.flashrom.org > > Maximum number of ROM images (4) in layout file reached before end of > layout file. > Ignoring the rest of the layout file. > I think that if the user accidentally specifies an invalid layout or number of included regions, then Flashrom should quit. Otherwise Flashrom might not flash everything the user intends, leaving the content in an inconsistent (probably bricked) state. This has become problematic due to complicated ROM layouts in systems where multiple masters use a shared SPI flash (e.g. an EC or Intel's ME insanity). We can apply this fix in a follow-up patch if you'd prefer: Index: layout.c =================================================================== --- layout.c (revision 1402) +++ layout.c (working copy) @@ -159,7 +159,7 @@ "file reached before end of layout file.\n", MAX_ROMLAYOUT); msg_gerr("Ignoring the rest of the layout file.\n"); - break; + return 1; } if (2 != fscanf(romlayout, "%s %s\n", tempstr, rom_entries[romimages].name)) continue; On Mon, Aug 1, 2011 at 6:05 PM, Stefan Tauner < stefan.tauner(a)student.tuwien.ac.at> wrote: > example of a faulty run: > flashrom -p dummy:emulate=SST25VF032B -i normal -w > ../testimages/4096kB.rand.img -l test.layout -i gfxrom -i fallback -i bios > flashrom v0.9.4-r1396 on Linux 2.6.35-30-generic (x86_64), built with > libpci 3.1.7, GCC 4.4.5, little endian > flashrom is free software, get the source code at http://www.flashrom.org > > Maximum number of ROM images (4) in layout file reached before end of > layout file. > Ignoring the rest of the layout file. > Using region(s): "normal", "gfxrom", "fallback"Invalid region specified: > "bios" > If you split that last line so that the "Invalid region" part shows up on a newline, it looks okay. Please run "flashrom --help" for usage info. > Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> > --- > cli_classic.c | 11 ++++----- > flash.h | 2 + > layout.c | 68 > ++++++++++++++++++++++++++++++++++++++++++++++++++------ > 3 files changed, 67 insertions(+), 14 deletions(-) > > diff --git a/cli_classic.c b/cli_classic.c > index 36fe9ad..e44ea86 100644 > --- a/cli_classic.c > +++ b/cli_classic.c > @@ -233,14 +233,9 @@ int cli_classic(int argc, char *argv[]) > cli_classic_abort_usage(); > break; > case 'i': > - /* FIXME: -l has to be specified before -i. */ > tempstr = strdup(optarg); > - if (find_romentry(tempstr) < 0) { > - fprintf(stderr, "Error: image %s not found > in " > - "layout file or -i specified before > " > - "-l\n", tempstr); > + if (register_include_arg(tempstr) < 0) > cli_classic_abort_usage(); > - } > break; > case 'L': > if (++operation_specified > 1) { > @@ -325,6 +320,10 @@ int cli_classic(int argc, char *argv[]) > cli_classic_abort_usage(); > } > > + if (process_include_args() < 0) { > + cli_classic_abort_usage(); > + } > + > /* FIXME: Print the actions flashrom will take. */ > > if (list_supported) { > diff --git a/flash.h b/flash.h > index 5b49e9d..0848255 100644 > --- a/flash.h > +++ b/flash.h > @@ -251,6 +251,8 @@ int print(int type, const char *fmt, ...) > __attribute__((format(printf, 2, 3))); > int cli_classic(int argc, char *argv[]); > > /* layout.c */ > +int register_include_arg(char *name); > +int process_include_args(void); > int read_romlayout(char *name); > int find_romentry(char *name); > int handle_romentries(struct flashchip *flash, uint8_t *oldcontents, > uint8_t *newcontents); > diff --git a/layout.c b/layout.c > index d719a05..936e316 100644 > --- a/layout.c > +++ b/layout.c > @@ -41,6 +41,12 @@ typedef struct { > char name[256]; > } romlayout_t; > > +/* include_args lists arguments specified at the command line with -i. > They > + * must be processed at some point so that desired regions are marked as > + * "included" in the rom_entries list. > + */ > +static char *include_args[MAX_ROMLAYOUT]; > +static int num_include_args = 0; /* the number of valid entries. */ > static romlayout_t rom_entries[MAX_ROMLAYOUT]; > > #if CONFIG_INTERNAL == 1 /* FIXME: Move the whole block to cbtable.c? */ > @@ -194,6 +200,20 @@ int read_romlayout(char *name) > } > #endif > > +/* register an include argument (-i) for later processing */ > +int register_include_arg(char *name) > +{ > + if (num_include_args >= MAX_ROMLAYOUT) { > + msg_gerr("Too many regions included (%i).\n", > num_include_args); > + return -1; > + } > + > + include_args[num_include_args] = name; > + num_include_args++; > + return num_include_args; > +} > + > +/* returns the index of the entry (or a negative value if it is not found) > */ > int find_romentry(char *name) > { > int i; > @@ -201,20 +221,49 @@ int find_romentry(char *name) > if (!romimages) > return -1; > > - msg_ginfo("Looking for \"%s\"... ", name); > - > + msg_gspew("Looking for region \"%s\"... ", name); > for (i = 0; i < romimages; i++) { > if (!strcmp(rom_entries[i].name, name)) { > rom_entries[i].included = 1; > - msg_ginfo("found.\n"); > + msg_gspew("found.\n"); > return i; > } > } > - msg_ginfo("not found.\n"); // Not found. Error. > - > + msg_gspew("not found.\n"); > return -1; > } > > +/* process -i arguments > + * returns 0 to indicate success, <0 to indicate failure > I think the usual Flashrom convention is to return 1 to indicate failure. I used <0 out of habit. Let's go ahead and fix that coding convention for upstream if desired, and update the call sites to do error checking correctly. > + */ > +int process_include_args(void) > +{ > + int i; > + unsigned int found = 0; > + for (i = 0; i < num_include_args && include_args[i] != NULL; i++) { > + /* User has specified an area, but no layout file is > loaded. */ > + if (!romimages) { > + msg_gerr("Region requested (with -i \"%s\"), " > + "but no layout data is available.\n", > + include_args[i]); > + return -1; > return 1 + } > + > + if (find_romentry(include_args[i]) < 0) { > + msg_gerr("Invalid region specified: \"%s\"\n", > + include_args[i]); > + return -1; > return 1 > + } > + if (found == 0) > + msg_ginfo("Using region(s): \"%s\"", > include_args[i]); > + else > + msg_ginfo(", \"%s\"", include_args[i]); > + found++; > + } > + msg_ginfo(".\n"); > + return 0; > +} > + > int find_next_included_romentry(unsigned int start) > { > int i; > @@ -246,11 +295,12 @@ int handle_romentries(struct flashchip *flash, > uint8_t *oldcontents, uint8_t *ne > int entry; > unsigned int size = flash->total_size * 1024; > > - /* If no layout file was specified or the layout file was empty, > assume > - * that the user wants to flash the complete new image. > + /* If no regions were specified for inclusion, assume > + * that the user wants to write the complete new image. > */ > - if (!romimages) > + if (num_include_args == 0) > return 0; > + > /* Non-included romentries are ignored. > * The union of all included romentries is used from the new image. > */ > @@ -262,6 +312,8 @@ int handle_romentries(struct flashchip *flash, uint8_t > *oldcontents, uint8_t *ne > size - start); > break; > } > + > + /* For non-included region, copy from old content. */ > if (rom_entries[entry].start > start) > memcpy(newcontents + start, oldcontents + start, > rom_entries[entry].start - start); > -- > 1.7.1 > > -- David Hendricks (dhendrix) Systems Software Engineer, Google Inc.

1 0

[PATCH] Confirm EN29LV800 bus, timing, and page size
by TJ 02 Aug '11

02 Aug '11

Uncertain values now confirmed. If you're happy with the new EN29LV800 code I'll submit a combined patch with a Signed-off-by tag. --- flashchips.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/flashchips.c b/flashchips.c index b5be180..10b8dea 100644 --- a/flashchips.c +++ b/flashchips.c @@ -2166,15 +2166,15 @@ struct flashchip flashchips[] = { { .vendor = "EON", .name = "EN29LV800(B)(C)B", - .bustype = CHIP_BUSTYPE_UNKNOWN, + .bustype = CHIP_BUSTYPE_PARALLEL, .manufacture_id = EON_ID, .model_id = EN_29LV800B, .total_size = 1024, - .page_size = 1024, // TODO: check this + .page_size = 1024, .feature_bits = FEATURE_ADDR_AAA | FEATURE_EITHER_RESET, .tested = TEST_UNTESTED, .probe = probe_jedec, - .probe_timing = TIMING_ZERO, /* TODO: check Datasheet has no timing info specified */ + .probe_timing = TIMING_ZERO, .erase = NULL, .block_erasers = { @@ -2198,15 +2198,15 @@ struct flashchip flashchips[] = { { .vendor = "EON", .name = "EN29LV800(B)(C)T", - .bustype = CHIP_BUSTYPE_UNKNOWN, + .bustype = CHIP_BUSTYPE_PARALLEL, .manufacture_id = EON_ID, .model_id = EN_29LV800T, .total_size = 1024, - .page_size = 1024, // TODO: check this + .page_size = 1024, .feature_bits = FEATURE_ADDR_AAA | FEATURE_EITHER_RESET, .tested = TEST_UNTESTED, .probe = probe_jedec, - .probe_timing = TIMING_ZERO, /* TODO: check Datasheet has no timing info specified */ + .probe_timing = TIMING_ZERO, .erase = NULL, .block_erasers = { -- 1.6.3.3

4 5

Report back
by Pulp and Paper Automation 02 Aug '11

02 Aug '11

Hi First thanks for a great service. I successfully tested a Intel NIC card Model Intel PRO/100 S desktop adapter Chipset Intel 82550EY Atmel flash chip "AT49BV512" There is also an onboard Atmel 93C46 that could not be read with –p nicintel_spi? Here is the edited results pupa@pupa-IMBA-9454ISA:~$ sudo flashrom -R test.bin -p nicintel Mapping Intel NIC control/status reg at 0xff8ff000, unaligned size 0x10. Found Atmel flash chip "AT49BV512" (64 kB, Parallel) on nicintel. Reading old flash chip contents... done. Verifying flash... VERIFIED. pupa@pupa-IMBA-9454ISA:~$ sudo flashrom -E -p nicintel Calibrating delay loop... OK. Found "Intel 82557/8/9/0/1 Ethernet Pro 100" (8086:1229, BDF 03:0d.0). Mapping Intel NIC control/status reg at 0xff8ff000, unaligned size 0x10. Found Atmel flash chip "AT49BV512" (64 kB, Parallel) on nicintel. Erasing and writing flash chip... Erase/write done. pupa@pupa-IMBA-9454ISA:~$ sudo flashrom -v test.bin -p nicintel Calibrating delay loop... OK. Mapping Intel NIC control/status reg at 0xff8ff000, unaligned size 0x10. Found Atmel flash chip "AT49BV512" (64 kB, Parallel) on nicintel. Reading old flash chip contents... done. Verifying flash... VERIFY FAILED at 0x00000000! Expected=0x55, Read=0xff, failed byte count from 0x00000000-0x0000ffff: 0xd0cf pupa@pupa-IMBA-9454ISA:~$ sudo flashrom -w test.bin -p nicintel Calibrating delay loop... OK. Mapping Intel NIC control/status reg at 0xff8ff000, unaligned size 0x10. Found Atmel flash chip "AT49BV512" (64 kB, Parallel) on nicintel. Reading old flash chip contents... done. Erasing and writing flash chip... Erase/write done. Verifying flash... VERIFIED. pupa@pupa-IMBA-9454ISA:~$ sudo flashrom -v test.bin -p nicintel Calibrating delay loop... OK. Found "Intel 82557/8/9/0/1 Ethernet Pro 100" (8086:1229, BDF 03:0d.0). Mapping Intel NIC control/status reg at 0xff8ff000, unaligned size 0x10. Found Atmel flash chip "AT49BV512" (64 kB, Parallel) on nicintel. Reading old flash chip contents... done. Verifying flash... VERIFIED.

2 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom