flashrom July 2011

flashrom@flashrom.org

50 participants
275 discussions

[commit] r1380 - trunk
by repository service 21 Jul '11

21 Jul '11

Author: hailfinger Date: Thu Jul 21 23:21:04 2011 New Revision: 1380 URL: http://flashrom.org/trac/flashrom/changeset/1380 Log: Fix out-of-bounds access if all erase functions fail. Fix detection of unchanged chip contents on erase failure. Return error if no usable erase functions exist. Thanks to Stefan Tauner for spotting the last problem. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/flashrom.c Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Thu Jul 21 21:59:34 2011 (r1379) +++ trunk/flashrom.c Thu Jul 21 23:21:04 2011 (r1380) @@ -1507,7 +1507,7 @@ int erase_and_write_flash(struct flashchip *flash, uint8_t *oldcontents, uint8_t *newcontents) { - int k, ret = 0; + int k, ret = 1; uint8_t *curcontents; unsigned long size = flash->total_size * 1024; unsigned int usable_erasefunctions = count_usable_erasers(flash); @@ -1522,8 +1522,12 @@ memcpy(curcontents, oldcontents, size); for (k = 0; k < NUM_ERASEFUNCTIONS; k++) { + if (!usable_erasefunctions) { + msg_cdbg("No usable erase functions left.\n"); + break; + } msg_cdbg("Looking at blockwise erase function %i... ", k); - if (check_block_eraser(flash, k, 1) && usable_erasefunctions) { + if (check_block_eraser(flash, k, 1)) { msg_cdbg("Looking for another erase function.\n"); continue; } @@ -1535,10 +1539,8 @@ if (!ret) break; /* Write/erase failed, so try to find out what the current chip - * contents are. If no usable erase functions remain, we could - * abort the loop instead of continuing, the effect is the same. - * The only difference is whether the reason for other unusable - * functions is printed or not. If in doubt, verbosity wins. + * contents are. If no usable erase functions remain, we can + * skip this: the next iteration will break immediately anyway. */ if (!usable_erasefunctions) continue;

1 0

Re: [flashrom] [PATCH] chipset_enable.c: add a message in the case the chipset enable is untested
by Paul Menzel 21 Jul '11

21 Jul '11

Am Donnerstag, den 21.07.2011, 14:23 +0200 schrieb Stefan Tauner: > old output: > Calibrating delay loop... OK. > Found chipset "Intel QS57", enabling flash write... OK. > This chipset supports the following protocols: FWH, SPI. > > new non-verbose output for tested chipsets: > Calibrating delay loop... OK. > Found chipset "Intel QS57". Enabling flash write... OK. > This chipset supports the following protocols: FWH, SPI. > > new non-verbose output for untested chipsets: > Found chipset "Intel QS57". > 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! A similar message should be shown somewhere else. I would put more information in it. This chipset is marked as untested. If you are using an up-to-date version of flashrom <http://flashrom.org/Downloads> please email a report to flashrom(a)flashrom.org with the subject (variable with the name (here Intel QS57) including a verbose (-V) log. Thank you! Do we have preferences if the log should be pasted or attached? > Enabling flash write... OK. > This chipset supports the following protocols: FWH, SPI. > > Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> > --- > chipset_enable.c | 19 ++++++++++++++----- > 1 files changed, 14 insertions(+), 5 deletions(-) […] Thanks, Paul

2 2

Re: [flashrom] [PATCH] chipset_enable.c: add a message in the case the chipset enable is untested
by Uwe Hermann 21 Jul '11

21 Jul '11

On Thu, Jul 21, 2011 at 02:23:05PM +0200, Stefan Tauner wrote: > Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> > + msg_pinfo("Found chipset \"%s %s\"", > + chipset_enables[i].vendor_name, > + chipset_enables[i].device_name); > + msg_pdbg("with PCI ID %04x:%04x", > chipset_enables[i].vendor_id, > chipset_enables[i].device_id); I'd print the IDs unconditionally, not only with -V, but that's for another patch. Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org

2 1

[commit] r1379 - trunk
by repository service 21 Jul '11

21 Jul '11

Author: stefanct Date: Thu Jul 21 21:59:34 2011 New Revision: 1379 URL: http://flashrom.org/trac/flashrom/changeset/1379 Log: chipset_enable.c: add a message for untested chipset enables old output: Calibrating delay loop... OK. Found chipset "Intel QS57", enabling flash write... OK. This chipset supports the following protocols: FWH, SPI. new non-verbose output for tested chipsets: Calibrating delay loop... OK. Found chipset "Intel QS57". Enabling flash write... OK. This chipset supports the following protocols: FWH, SPI. new non-verbose output for untested chipsets: Found chipset "Intel QS57". 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: FWH, SPI. Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Modified: trunk/chipset_enable.c Modified: trunk/chipset_enable.c ============================================================================== --- trunk/chipset_enable.c Thu Jul 21 21:52:00 2011 (r1378) +++ trunk/chipset_enable.c Thu Jul 21 21:59:34 2011 (r1379) @@ -1226,13 +1226,22 @@ chipset_enables[i].device_name); continue; } - msg_pinfo("Found chipset \"%s %s\", enabling flash write... ", - chipset_enables[i].vendor_name, - chipset_enables[i].device_name); - msg_pdbg("chipset PCI ID is %04x:%04x, ", + msg_pinfo("Found chipset \"%s %s\"", + chipset_enables[i].vendor_name, + chipset_enables[i].device_name); + msg_pdbg("with PCI ID %04x:%04x", chipset_enables[i].vendor_id, chipset_enables[i].device_id); + msg_pinfo(". "); + if (chipset_enables[i].status == NT) { + msg_pinfo("\nThis chipset is marked as untested. If " + "you are using an up-to-date version\nof " + "flashrom please email a report to " + "flashrom(a)flashrom.org including a\nverbose " + "(-V) log. Thank you!\n"); + } + msg_pinfo("Enabling flash write... "); ret = chipset_enables[i].doit(dev, chipset_enables[i].device_name); if (ret == NOT_DONE_YET) {

1 0

Re: [flashrom] [PATCH] flashrom.8: explain read accesses as part of the write operation
by Uwe Hermann 21 Jul '11

21 Jul '11

On Thu, Jul 21, 2011 at 01:06:29PM +0200, Stefan Tauner wrote: > Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Lookѕ good, also did a quick check against the code. Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org

2 1

[commit] r1378 - trunk
by repository service 21 Jul '11

21 Jul '11

Author: stefanct Date: Thu Jul 21 21:52:00 2011 New Revision: 1378 URL: http://flashrom.org/trac/flashrom/changeset/1378 Log: flashrom.8: explain read accesses as part of the write operation Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Acked-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Modified: trunk/flashrom.8 Modified: trunk/flashrom.8 ============================================================================== --- trunk/flashrom.8 Thu Jul 21 11:18:18 2011 (r1377) +++ trunk/flashrom.8 Thu Jul 21 21:52:00 2011 (r1378) @@ -45,6 +45,13 @@ into flash ROM. This will first automatically .B erase the chip, then write to it. +.sp +In the process the chip is also read several times. First an in-memory backup +is made for disaster recovery and to be able to skip regions that are +already equal to the image file. This copy is updated along with the write +operation. In case of erase errors it is even re-read completely. After +writing has finished and if verification is enabled, the whole flash chip is +read out and compared with the input image. .TP .B "\-n, \-\-noverify" Skip the automatic verification of flash ROM contents after writing. Using this @@ -541,7 +548,7 @@ backup. This is caused by the embedded controller (EC) present in many laptops, which interacts badly with any flash attempts. This is a hardware limitation and flashrom will attempt to detect it and abort immediately for safety reasons. -.SH LICENCE +.SH LICENSE .B flashrom is covered by the GNU General Public License (GPL), version 2. Some files are additionally available under the GPL (version 2, or any later version). @@ -623,6 +630,6 @@ .br All authors can be reached via email at <flashrom(a)flashrom.org>. .PP -This manual page was written by Uwe Hermann <uwe(a)hermann-uwe.de> -and Carl-Daniel Hailfinger. +This manual page was written by Uwe Hermann <uwe(a)hermann-uwe.de>, +Carl-Daniel Hailfinger and others. It is licensed under the terms of the GNU GPL (version 2 or later).

1 0

[commit] r1369 - trunk
by repository service 21 Jul '11

21 Jul '11

Author: stefanct Date: Wed Jul 13 00:01:44 2011 New Revision: 1369 URL: http://flashrom.org/trac/flashrom/changeset/1369 Log: update motherboard URLs in print.c Signed-off-by: Benjamin Bellec <b.bellec(a)gmail.com> with small changes: 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/print.c Modified: trunk/print.c ============================================================================== --- trunk/print.c Sat Jul 9 21:46:53 2011 (r1368) +++ trunk/print.c Wed Jul 13 00:01:44 2011 (r1369) @@ -376,15 +376,15 @@ const struct board_info boards_known[] = { #if defined(__i386__) || defined(__x86_64__) B("A-Trend", "ATC-6220", 1, "http://www.motherboard.cz/mb/atrend/atc6220.htm", NULL), - B("abit", "AN-M2", 1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?DEFTITLE=…", NULL), + B("abit", "AN-M2", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=…", NULL), B("abit", "AX8", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=…", NULL), B("abit", "BM6", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NA…", NULL), - B("abit", "Fatal1ty F-I90HD", 1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?pMODEL_NA…", NULL), + B("abit", "Fatal1ty F-I90HD", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NA…", NULL), B("abit", "IC7", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NA…", NULL), B("abit", "IP35", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=LG…", NULL), - B("abit", "IP35 Pro", 1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?fMTYPE=LG…", NULL), + B("abit", "IP35 Pro", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=LG…", NULL), B("abit", "IS-10", 0, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NA…", "Reported by deejkuba(a)aol.com to flashrom(a)coreboot.org, no public archive. Missing board enable and/or M50FW040 unlocking. May work now."), - B("abit", "KN8 Ultra", 1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?DEFTITLE=…", NULL), + B("abit", "KN8 Ultra", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=…", NULL), B("abit", "NF-M2 nView", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=So…", NULL), B("abit", "NF7-S", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=So…", NULL), B("abit", "VA6", 1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=Sl…", NULL), @@ -399,16 +399,16 @@ B("ASI", "MB-5BLMP", 1, "http://www.hojerteknik.com/winnet.htm", "Used in the IGEL WinNET III thin client."), B("ASRock", "775i65G", 1, "http://www.asrock.com/mb/overview.asp?Model=775i65G", NULL), B("ASRock", "890GX Extreme3", 1, "http://www.asrock.com/mb/overview.asp?Model=890GX%20Extreme3", NULL), - B("ASRock", "939A785GMH/128M", 1, "http://www.asrock.com/mb/overview.asp?Model=939A785GMH/128M&s=939", NULL), + B("ASRock", "939A785GMH/128M", 1, "http://www.asrock.com/mb/overview.asp?Model=939A785GMH/128M", NULL), B("ASRock", "A330GC", 1, "http://www.asrock.com/mb/overview.asp?Model=A330GC", NULL), - B("ASRock", "A770CrossFire", 1, "http://www.asrock.com/mb/overview.asp?Model=A770CrossFire&s=AM2%%2b", NULL), + B("ASRock", "A770CrossFire", 1, "http://www.asrock.com/mb/overview.asp?Model=A770CrossFire", NULL), B("ASRock", "ALiveNF6G-DVI", 1, "http://www.asrock.com/mb/overview.asp?Model=ALiveNF6G-DVI", NULL), B("ASRock", "K7S41", 1, "http://www.asrock.com/mb/overview.asp?Model=K7S41", NULL), B("ASRock", "K7S41GX", 1, "http://www.asrock.com/mb/overview.asp?Model=K7S41GX", NULL), - B("ASRock", "K7VT4A+", 0, "http://www.asrock.com/mb/overview.asp?Model=K7VT4A%%2b&s=", "No chip found, probably due to flash translation. http://www.flashrom.org/pipermail/flashrom/2009-August/000393.html"), + B("ASRock", "K7VT4A+", 0, "http://www.asrock.com/mb/overview.asp?Model=K7VT4A%2b", "No chip found, probably due to flash translation. http://www.flashrom.org/pipermail/flashrom/2009-August/000393.html"), B("ASRock", "K8S8X", 1, "http://www.asrock.com/mb/overview.asp?Model=K8S8X", NULL), - B("ASRock", "M3A790GXH/128M", 1, "http://www.asrock.com/MB/overview.asp?Model=M3A790GXH/128M", NULL), - B("ASRock", "P4i65GV", 1, NULL, NULL), + B("ASRock", "M3A790GXH/128M", 1, "http://www.asrock.com/mb/overview.asp?Model=M3A790GXH/128M", NULL), + B("ASRock", "P4i65GV", 1, "http://www.asrock.com/mb/overview.asp?Model=P4i65GV", NULL), B("ASUS", "A7N8X Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=wAsRYm41KTp78MFC", NULL), B("ASUS", "A7N8X-E Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=TmQtPJv4jIxmL9C2", NULL), B("ASUS", "A7V133", 1, "ftp://ftp.asus.com.tw/pub/ASUS/mb/socka/kt133a/a7v133/", NULL), @@ -525,9 +525,9 @@ B("GIGABYTE", "GA-6IEM", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1379", NULL), B("GIGABYTE", "GA-6VXE7+", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2410", NULL), B("GIGABYTE", "GA-6ZMA", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1541", NULL), - B("GIGABYTE", "GA-MA785GMT-UD2H", 1, "http://www.gigabyte.de/Products/Motherboard/Products_Overview.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-770TA-UD3", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=3272#ov", NULL), - B("GIGABYTE", "GA-7DXR", 1, "http://www.gigabyte.de/Products/Motherboard/Products_Spec.aspx?ProductID=13…", NULL), + B("GIGABYTE", "GA-MA785GMT-UD2H (rev. 1.0)", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=3156", NULL), + B("GIGABYTE", "GA-770TA-UD3", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=3272", NULL), + B("GIGABYTE", "GA-7DXR", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1302", NULL), B("GIGABYTE", "GA-7VT600", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1666", NULL), B("GIGABYTE", "GA-7ZM", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1366", "Works fine if you remove jumper JP9 on the board and disable the flash protection BIOS option."), B("GIGABYTE", "GA-8IP775", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1830", NULL), @@ -579,43 +579,43 @@ B("Lanner", "EM-8510C", 1, NULL, NULL), B("Lex", "CV700A", 1, "http://www.lex.com.tw/product/CV700A-spec.htm", NULL), B("Mitac", "6513WU", 1, "http://web.archive.org/web/20050313054828/http://www.mitac.com/micweb/produ…", NULL), - B("MSI", "MS-6153", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=336", NULL), + B("MSI", "MS-6153", 1, "http://www.msi.com/product/mb/MS-6153.html", NULL), B("MSI", "MS-6156", 1, "http://uk.ts.fujitsu.com/rl/servicesupport/techsupport/boards/Motherboards/…", NULL), - B("MSI", "MS-6163 (MS-6163 Pro)",1, "http://www.msi.com/index.php?func=proddesc&prod_no=340", NULL), - B("MSI", "MS-6178", 0, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=343", "Immediately powers off if you try to hot-plug the chip. However, this does '''not''' happen if you use coreboot. Owned by Uwe Hermann <uwe(a)hermann-uwe.de>."), - B("MSI", "MS-6330 (K7T Turbo)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=327", NULL), - B("MSI", "MS-6391 (845 Pro4)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=293", NULL), - B("MSI", "MS-6561 (745 Ultra)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=274", NULL), + B("MSI", "MS-6163 (MS-6163 Pro)",1, "http://www.msi.com/product/mb/MS-6163-Pro.html", NULL), + B("MSI", "MS-6178", 0, "http://www.msi.com/product/mb/MS-6178.html", "Immediately powers off if you try to hot-plug the chip. However, this does '''not''' happen if you use coreboot. Owned by Uwe Hermann <uwe(a)hermann-uwe.de>."), + B("MSI", "MS-6330 (K7T Turbo)", 1, "http://www.msi.com/product/mb/K7T-Turbo.html", NULL), + B("MSI", "MS-6391 (845 Pro4)", 1, "http://www.msi.com/product/mb/845-Pro4.html", NULL), + B("MSI", "MS-6561 (745 Ultra)", 1, "http://www.msi.com/product/mb/745-Ultra.html", NULL), B("MSI", "MS-6566 (845 Ultra-C)",1, "http://www.msi.com/product/mb/845-Ultra-C.html", NULL), - B("MSI", "MS-6570 (K7N2)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=519", NULL), + B("MSI", "MS-6570 (K7N2)", 1, "http://www.msi.com/product/mb/K7N2.html", NULL), B("MSI", "MS-6577 (Xenon)", 1, "http://h10025.www1.hp.com/ewfrf/wc/document?product=90390&lc=en&cc=us&dlc=e…", "This is an OEM board from HP, the HP name is Xenon."), - B("MSI", "MS-6590 (KT4 Ultra)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=502", NULL), - B("MSI", "MS-6702E (K8T Neo2-F)",1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=588", NULL), - B("MSI", "MS-6712 (KT4V)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=505", NULL), - B("MSI", "MS-6787 (P4MAM-V/P4MAM-L)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=577", NULL), - B("MSI", "MS-7005 (651M-L)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=559", NULL), - B("MSI", "MS-7025 (K8N Neo2 Platinum)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=587", NULL), + B("MSI", "MS-6590 (KT4 Ultra)", 1, "http://www.msi.com/product/mb/KT4-Ultra.html", NULL), + B("MSI", "MS-6702E (K8T Neo2-F)",1, "http://www.msi.com/product/mb/K8T-Neo2-F--FIR.html", NULL), + B("MSI", "MS-6712 (KT4V)", 1, "http://www.msi.com/product/mb/KT4V---KT4V-L--v1-0-.html", NULL), + B("MSI", "MS-6787 (P4MAM-V/P4MAM-L)", 1, "http://www.msi.com/service/search/?kw=6787&type=product", NULL), + B("MSI", "MS-7005 (651M-L)", 1, "http://www.msi.com/product/mb/651M-L.html", NULL), + B("MSI", "MS-7025 (K8N Neo2 Platinum)", 1, "http://www.msi.com/product/mb/K8N-Neo2-Platinum.html", NULL), B("MSI", "MS-7046", 1, "http://www.heimir.de/ms7046/", NULL), - B("MSI", "MS-7061 (KM4M-V/KM4AM-V)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=594", NULL), + B("MSI", "MS-7061 (KM3M-V/KM4M-V/KM4AM-V)", 1, "http://www.msi.com/service/search/?kw=7061&type=product", NULL), B("MSI", "MS-7065", 1, "http://browse.geekbench.ca/geekbench2/view/53114", NULL), - B("MSI", "MS-7135 (K8N Neo3)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=170", NULL), + B("MSI", "MS-7135 (K8N Neo3)", 1, "http://www.msi.com/product/mb/K8N-Neo3.html", NULL), B("MSI", "MS-7168 (Orion)", 1, "http://support.packardbell.co.uk/uk/item/index.php?i=spec_orion&pi=platform…", NULL), - B("MSI", "MS-7207 (K8NGM2-L)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=224", NULL), - B("MSI", "MS-7211 (PM8M3-V)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=234", NULL), - B("MSI", "MS-7236 (945PL Neo3)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1173", NULL), - B("MSI", "MS-7253 (K9VGM-V)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=260", NULL), - B("MSI", "MS-7255 (P4M890M)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1082", NULL), - B("MSI", "MS-7260 (K9N Neo)", 0, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=255", "Interestingly flashrom does not work when the vendor BIOS is booted, but it ''does'' work flawlessly when the machine is booted with coreboot. Owned by Uwe Hermann <uwe(a)hermann-uwe.de>."), - B("MSI", "MS-7312 (K9MM-V)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1104", NULL), - B("MSI", "MS-7345 (P35 Neo2-FIR)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1261", NULL), - B("MSI", "MS-7368 (K9AG Neo2-Digital)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1241", NULL), - B("MSI", "MS-7376 (K9A2 Platinum)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1332", NULL), + B("MSI", "MS-7207 (K8NGM2-L)", 1, "http://www.msi.com/product/mb/K8NGM2-FID--IL--L.html", NULL), + B("MSI", "MS-7211 (PM8M3-V)", 1, "http://www.msi.com/product/mb/PM8M3-V.html", NULL), + B("MSI", "MS-7236 (945PL Neo3)", 1, "http://www.msi.com/product/mb/945PL-Neo3.html", NULL), + B("MSI", "MS-7253 (K9VGM-V)", 1, "http://www.msi.com/product/mb/K9VGM-V.html", NULL), + B("MSI", "MS-7255 (P4M890M)", 1, "http://www.msi.com/product/mb/P4M890M-L-IL.html", NULL), + B("MSI", "MS-7260 (K9N Neo PCB 1.0)", 0, "http://www.msi.com/product/mb/K9N-Neo--PCB-1-0-.html", "Interestingly flashrom does not work when the vendor BIOS is booted, but it ''does'' work flawlessly when the machine is booted with coreboot. Owned by Uwe Hermann <uwe(a)hermann-uwe.de>."), + B("MSI", "MS-7312 (K9MM-V)", 1, "http://www.msi.com/product/mb/K9MM-V.html", NULL), + B("MSI", "MS-7345 (P35 Neo2-FIR)", 1, "http://www.msi.com/product/mb/P35-Neo2-FR---FIR.html", NULL), + B("MSI", "MS-7368 (K9AG Neo2-Digital)", 1, "http://www.msi.com/product/mb/K9AG-Neo2-Digital.html", NULL), + B("MSI", "MS-7376 (K9A2 Platinum V1)", 1, "http://www.msi.com/product/mb/K9A2-Platinum.html", NULL), B("MSI", "MS-7529 (G31M3-L(S) V2)", 1, "http://www.msi.com/product/mb/G31M3-L-V2---G31M3-LS-V2.html", NULL), - B("MSI", "MS-7529 (G31TM-P21)", 1, "http://msi.com/product/mb/G31TM-P21.html", NULL), - B("MSI", "MS-7596 (785GM-E51)", 1, "http://eu.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=1866", NULL), + B("MSI", "MS-7529 (G31TM-P21)", 1, "http://www.msi.com/product/mb/G31TM-P21.html", NULL), + B("MSI", "MS-7596 (785GM-E51)", 1, "http://www.msi.com/product/mb/785GM-E51.html", NULL), B("MSI", "MS-7599 (870-C45)", 1, "http://www.msi.com/product/mb/870-C45.html", NULL), - B("MSI", "MS-7640 (890FXA-GD70)",1, "http://www.msi.com/product/mb/890FXA-GD70.html", "Reported by \"Linux User #330250\" http://flashrom.org/pipermail/flashrom/2011-March/006072.html"), - B("MSI", "MS-7642 (890GXM-G65)", 1, "http://www.msi.com/index.php?func=proddesc&maincat_no=1&prod_no=2012", NULL), + B("MSI", "MS-7640 (890FXA-GD70)",1, "http://www.msi.com/product/mb/890FXA-GD70.html", NULL), + B("MSI", "MS-7642 (890GXM-G65)", 1, "http://www.msi.com/product/mb/890GXM-G65.html", NULL), B("MSI", "MS-7698 (E350IA-E45)", 1, "http://www.msi.com/product/mb/E350IA-E45.html", NULL), B("NEC", "PowerMate 2000", 1, "http://support.necam.com/mobilesolutions/hardware/Desktops/pm2000/celeron/", NULL), B("Nokia", "IP530", 1, NULL, NULL),

3 2

[commit] r1375 - trunk
by repository service 21 Jul '11

21 Jul '11

Author: stefanct Date: Tue Jul 19 10:50:18 2011 New Revision: 1375 URL: http://flashrom.org/trac/flashrom/changeset/1375 Log: annotate additional flashchips with voltage ranges also fix a few others and remove the one for Intel 28F002BC/BL/BV/BX-T because we need to investigate it further. Signed-off-by: Steven Zakulec <spzakulec(a)gmail.com> Acked-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> Modified: trunk/flashchips.c Modified: trunk/flashchips.c ============================================================================== --- trunk/flashchips.c Tue Jul 19 09:58:06 2011 (r1374) +++ trunk/flashchips.c Tue Jul 19 10:50:18 2011 (r1375) @@ -113,7 +113,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {4750, 5250}, /* -55 speed is +-5%, all others +-10% */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -55, others 4.5-5.5V */ }, { @@ -145,6 +145,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {4750, 5250}, /* 4.75-5.25V for type -55, others 4.5-5.5V */ }, { @@ -256,6 +257,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -45R, others 2.7-3.6V */ }, { @@ -286,6 +288,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -45R, others 2.7-3.6V */ }, { @@ -317,6 +320,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -55, others 2.7-3.6V */ }, { @@ -348,6 +352,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -55, others 2.7-3.6V */ }, { @@ -379,6 +384,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {2700, 3600}, }, { @@ -410,6 +416,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {2700, 3600}, }, { @@ -441,6 +448,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600} /* 3.0-3.6V for type -70R, others 2.7-3.6V */ }, { @@ -472,6 +480,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600} /* 3.0-3.6V for type -70R, others 2.7-3.6V */ }, { @@ -498,7 +507,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {2700, 3600}, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -60R, others 2.7-3.6V*/ }, { @@ -525,7 +534,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {2700, 3600}, + .voltage = {3000, 3600}, /* 3.0-3.6V for type -70R, others 2.7-3.6V */ }, { @@ -591,6 +600,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -1332,7 +1342,7 @@ .unlock = spi_disable_blockprotect_at25df, .write = spi_chip_write_256, .read = spi_chip_read, - .voltage = {2300, 3600}, /* Datasheet says 2.3-3.6V or 2.7-3.6V */ + .voltage = {2700, 3600}, /* 2.3-3.6V & 2.7-3.6V models available */ }, { @@ -1370,7 +1380,7 @@ .unlock = spi_disable_blockprotect_at25df, .write = spi_chip_write_256, .read = spi_chip_read, - .voltage = {2300, 3600}, /* Datasheet says 2.3-3.6V or 2.7-3.6V */ + .voltage = {2700, 3600}, /* 2.3-3.6V & 2.7-3.6V models available */ }, { @@ -1446,6 +1456,7 @@ .unlock = spi_disable_blockprotect_at25df_sec, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -1773,7 +1784,7 @@ }, .write = NULL /* Incompatible Page write */, .read = spi_chip_read, - .voltage = {2700, 3600}, /* Datasheet says 3.0-3.6 V or 2.7-3.6 V */ + .voltage = {2700, 3600}, /* 3.0-3.6V for higher speed, 2.7-3.6V normal */ }, { @@ -2098,7 +2109,7 @@ .probe_timing = TIMING_ZERO, .write = NULL, .read = NULL, - .voltage = {2500, 3600}, /* Datasheet says 2.5-3.6 V or 2.7-3.6 V */ + .voltage = {2500, 3600}, /* 2.5-3.6V & 2.7-3.6V models available */ }, { @@ -2114,7 +2125,7 @@ .probe_timing = TIMING_ZERO, .write = NULL, .read = NULL, - .voltage = {2500, 3600}, /* Datasheet says 2.5-3.6 V or 2.7-3.6 V */ + .voltage = {2700, 3600}, /* 2.5-3.6V & 2.7-3.6V models available */ }, { @@ -2130,7 +2141,7 @@ .probe_timing = TIMING_ZERO, .write = NULL, .read = NULL, - .voltage = {2500, 3600}, /* Datasheet says 2.5-3.6 V or 2.7-3.6 V */ + .voltage = {2700, 3600}, /* 2.5-3.6V & 2.7-3.6V models available */ }, { @@ -2317,6 +2328,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {4500, 5500}, }, { @@ -3378,7 +3390,7 @@ }, .write = write_m29f400bt, .read = read_memmapped, - .voltage = {4750, 5250}, /* 5.0V +-5% for -55 model, +-10% for rest */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -55, others 4.5-5.5V */ }, { @@ -3410,7 +3422,7 @@ }, .write = write_m29f400bt, .read = read_memmapped, - .voltage = {4750, 5250}, /* 5.0V +-5% for -55 model, +-10% for rest */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -55, others 4.5-5.5V */ }, { @@ -3442,7 +3454,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {4750, 5250}, /* 5.0V +-5% for -45 model, +-10% for rest */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -45, others 4.5-5.5V */ }, { @@ -3474,7 +3486,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {4750, 5250}, /* 5.0V +-5% for -45 model, +-10% for rest */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -45, others 4.5-5.5V */ }, { @@ -3501,6 +3513,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {4500, 5500}, }, { @@ -3582,7 +3595,6 @@ }, .write = write_82802ab, .read = read_memmapped, - .voltage = {4500, 5500}, /* 5.0V +-10% read, 12V fast program & erase- +-5% standard, +-10% option */ }, { @@ -4052,6 +4064,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -4344,6 +4357,7 @@ }, .write = write_jedec_1, .read = read_memmapped, + .voltage = {4500, 5500}, }, { @@ -4775,6 +4789,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -4900,6 +4915,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5201,6 +5217,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5281,6 +5298,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5307,6 +5325,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5336,6 +5355,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_1, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5473,6 +5493,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_1, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -5538,6 +5559,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_1, .read = spi_chip_read, + .voltage = {3000, 3600}, }, { @@ -5573,6 +5595,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_1, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -6780,6 +6803,7 @@ .unlock = spi_disable_blockprotect, .write = spi_chip_write_256, .read = spi_chip_read, + .voltage = {2700, 3600}, }, { @@ -6840,7 +6864,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {4750, 5250}, /* Datasheet says some are only 4.75-5.25 V */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -X, others 4.5-5.5V */ }, { @@ -6872,7 +6896,7 @@ }, .write = write_jedec_1, .read = read_memmapped, - .voltage = {4750, 5250}, /* Datasheet says some are only 4.75-5.25 V */ + .voltage = {4750, 5250}, /* 4.75-5.25V for type -X, others 4.5-5.5V */ }, { @@ -8131,6 +8155,7 @@ .printlock = printlock_w39l040, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, }, { @@ -8275,6 +8300,7 @@ .unlock = unlock_w39v040fb, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* Also has 12V fast program */ }, { @@ -8302,6 +8328,7 @@ .printlock = printlock_w39v040fc, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* Also has 12V fast program */ }, { @@ -8478,7 +8505,7 @@ .unlock = unlock_w39v080fa, .write = write_jedec_1, .read = read_memmapped, - .voltage = {3000, 3600}, /* 12 V fast program mode */ + .voltage = {3000, 3600}, /* Also has 12V fast program */ }, { @@ -8506,6 +8533,7 @@ .printlock = printlock_w39v080fa_dual, .write = write_jedec_1, .read = read_memmapped, + .voltage = {3000, 3600}, /* Also has 12V fast program */ }, {

2 1

[PATCH] ft2232_spi: Improve error handling, remove exit() calls
by Uwe Hermann 21 Jul '11

21 Jul '11

See patch. This is just a first step, there is certainly room for further improving the error handling (error #defines such as FL_ERR_MALLOC, consistent and documented return codes, and more). Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org

2 2

[PATCH] dummyflasher.c: add an "empty" parameter and initialize with random bytes by default
by Stefan Tauner 21 Jul '11

21 Jul '11

previously the dummies were initialized to be empty (all ones), which makes writes skip erasing altogether. with this patch the default is to fill it with random bytes instead and the old behavior can be enforced with stating "empty=yes" on the command line. Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> --- dummyflasher.c | 22 ++++++++++++++++++++-- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/dummyflasher.c b/dummyflasher.c index fca228c..b463ea4 100644 --- a/dummyflasher.c +++ b/dummyflasher.c @@ -94,6 +94,8 @@ int dummy_init(void) { char *bustext = NULL; char *tmp = NULL; + char *empty; + int e = 0; #if EMULATE_CHIP struct stat image_stat; #endif @@ -128,6 +130,13 @@ int dummy_init(void) msg_pdbg("Support for all flash bus types disabled.\n"); free(bustext); + empty = extract_programmer_param("empty"); + if (empty != NULL) { + if (strstr(empty, "yes")) + e = 1; + free(empty); + } + tmp = extract_programmer_param("spi_write_256_chunksize"); if (tmp) { spi_write_256_chunksize = atoi(tmp); @@ -198,8 +207,17 @@ int dummy_init(void) return 1; } - msg_pdbg("Filling fake flash chip with 0xff, size %i\n", emu_chip_size); - memset(flashchip_contents, 0xff, emu_chip_size); + msg_pdbg("Filling fake flash chip containing %i bytes with ", + emu_chip_size); + if (e) { + msg_pdbg("0xff.\n"); + memset(flashchip_contents, 0xff, emu_chip_size); + } else { + msg_pdbg("random data.\n"); + unsigned int i; + for (i = 0; i < emu_chip_size; i++) + flashchip_contents[i] = rand(); + } emu_persistent_image = extract_programmer_param("image"); if (!emu_persistent_image) { -- 1.7.1

1 0

← Newer
1
...
13
14
15
16
17
18
19
...
28
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom July 2011