flashrom

flashrom@flashrom.org

5 participants
9161 discussions

ASUS A8N-VM CSM results (write/erase failed)
by Ville Skyttä 04 Jun '10

04 Jun '10

Hello, I tried out flashrom on an ASUS A8N-VM CSM board: read seemed to go fine, but write failed. Information about the board: http://www.asus.com/product.aspx?P_ID=JBqqlpj4cspbSa3s flashrom -V, lspci -nnvvxxx, superiotool -deV, flashrom -rV and flashrom -wV output is attached. The BIOS I was trying to write was the rev 1001 one available from the above ASUS page. Per agaran's and roysjosh's friendly help on IRC I checked the BIOS I read with -r with -v and it says VERIFIED so I suppose I'll go ahead and reboot later today. One note about the board testing Wiki page you may want to clarify: "To check if you can read the existing BIOS image from the chip, run flashrom -r backup.bin. Make sure that backup.bin contains a useful BIOS image. (Some chipsets will return 0xff for large areas of flash without any error messages.)" For a BIOS newbie like me, the remark in parenthesis doesn't tell much - is it a good or a bad sign if a chipset returns 0xff for large areas of flash without any error messages? (Bad, I guess ;))

2 4

[PATCH] Disallow the internal programmer on non-x86 for now
by Carl-Daniel Hailfinger 04 Jun '10

04 Jun '10

The internal programmer needs correct information about flash_base and chip window top/bottom alignment on non-x86 before it can be used. Abort any internal programmer action for now until the code is fixed. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: flashrom-disable_internal_nonx86/internal.c =================================================================== --- flashrom-disable_internal_nonx86/internal.c (Revision 1013) +++ flashrom-disable_internal_nonx86/internal.c (Arbeitskopie) @@ -165,16 +165,22 @@ pci_init(pacc); /* Initialize the PCI library */ pci_scan_bus(pacc); /* We want to get the list of devices */ - /* We look at the lbtable first to see if we need a +#if defined(__i386__) || defined(__x86_64__) + /* We look at the cbtable first to see if we need a * mainboard specific flash enable sequence. */ coreboot_init(); -#if defined(__i386__) || defined(__x86_64__) dmi_init(); /* Probe for the Super I/O chip and fill global struct superio. */ probe_superio(); +#else + /* FIXME: Enable cbtable searching on all non-x86 platforms supported + * by coreboot. + * FIXME: Find a replacement for DMI on non-x86. + * FIXME: Enable SuperI/O probing once port I/O is possible. + */ #endif /* Warn if a laptop is detected. */ @@ -200,6 +206,7 @@ } } +#if __FLASHROM_LITTLE_ENDIAN__ /* try to enable it. Failure IS an option, since not all motherboards * really need this to be done, etc., etc. */ @@ -220,7 +227,25 @@ * The error code might have been a warning only. * Besides that, we don't check the board enable return code either. */ +#if defined(__i386__) || defined(__x86_64__) return 0; +#else + msg_perr("Your platform is not supported yet for the internal " + "programmer due to missing flash_base and top/bottom " + "alignment information.\n" + "Aborting.\n"); + return 1; +#endif +#else + /* FIXME: Remove this unconditional abort once all PCI drivers are + * converted to use little-endian accesses for memory BARs. + */ + msg_perr("Your platform is not supported yet for the internal " + "programmer because it has not been converted from native " + "endian to little endian access yet.\n" + "Aborting.\n"); + return 1; +#endif } int internal_shutdown(void) -- http://www.hailfinger.org/

3 7

[commit] r1031 - trunk
by repository service 04 Jun '10

04 Jun '10

Author: hailfinger Date: Fri Jun 4 21:05:39 2010 New Revision: 1031 URL: http://flashrom.org/trac/coreboot/changeset/1031 Log: The internal programmer needs correct information about flash_base and chip window top/bottom alignment on non-x86 before it can be used. Abort any internal programmer action for now until the code is fixed. Add the concept of a processor enable for systems where flashing is impacted by processor settings or processor model. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Added: trunk/processor_enable.c Modified: trunk/Makefile trunk/flash.h trunk/internal.c Modified: trunk/Makefile ============================================================================== --- trunk/Makefile Fri Jun 4 19:07:39 2010 (r1030) +++ trunk/Makefile Fri Jun 4 21:05:39 2010 (r1031) @@ -124,7 +124,7 @@ ifeq ($(CONFIG_INTERNAL), yes) FEATURE_CFLAGS += -D'CONFIG_INTERNAL=1' -PROGRAMMER_OBJS += chipset_enable.o board_enable.o cbtable.o dmi.o internal.o +PROGRAMMER_OBJS += processor_enable.o chipset_enable.o board_enable.o cbtable.o dmi.o internal.o # FIXME: The PROGRAMMER_OBJS below should only be included on x86. PROGRAMMER_OBJS += it87spi.o ichspi.o sb600spi.o wbsio_spi.o NEED_PCI := yes Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Fri Jun 4 19:07:39 2010 (r1030) +++ trunk/flash.h Fri Jun 4 21:05:39 2010 (r1031) @@ -361,6 +361,9 @@ /* chipset_enable.c */ int chipset_flash_enable(void); +/* processor_enable.c */ +int processor_flash_enable(void); + /* physmap.c */ void *physmap(const char *descr, unsigned long phys_addr, size_t len); void *physmap_try_ro(const char *descr, unsigned long phys_addr, size_t len); Modified: trunk/internal.c ============================================================================== --- trunk/internal.c Fri Jun 4 19:07:39 2010 (r1030) +++ trunk/internal.c Fri Jun 4 21:05:39 2010 (r1031) @@ -165,16 +165,28 @@ pci_init(pacc); /* Initialize the PCI library */ pci_scan_bus(pacc); /* We want to get the list of devices */ - /* We look at the lbtable first to see if we need a + if (processor_flash_enable()) { + msg_perr("Processor detection/init failed.\n" + "Aborting.\n"); + return 1; + } + +#if defined(__i386__) || defined(__x86_64__) + /* We look at the cbtable first to see if we need a * mainboard specific flash enable sequence. */ coreboot_init(); -#if defined(__i386__) || defined(__x86_64__) dmi_init(); /* Probe for the Super I/O chip and fill global struct superio. */ probe_superio(); +#else + /* FIXME: Enable cbtable searching on all non-x86 platforms supported + * by coreboot. + * FIXME: Find a replacement for DMI on non-x86. + * FIXME: Enable Super I/O probing once port I/O is possible. + */ #endif /* Warn if a laptop is detected. */ @@ -200,6 +212,7 @@ } } +#if __FLASHROM_LITTLE_ENDIAN__ /* try to enable it. Failure IS an option, since not all motherboards * really need this to be done, etc., etc. */ @@ -220,7 +233,26 @@ * The error code might have been a warning only. * Besides that, we don't check the board enable return code either. */ +#if defined(__i386__) || defined(__x86_64__) return 0; +#else + msg_perr("Your platform is not supported yet for the internal " + "programmer due to missing\n" + "flash_base and top/bottom alignment information.\n" + "Aborting.\n"); + return 1; +#endif +#else + /* FIXME: Remove this unconditional abort once all PCI drivers are + * converted to use little-endian accesses for memory BARs. + */ + msg_perr("Your platform is not supported yet for the internal " + "programmer because it has\n" + "not been converted from native endian to little endian " + "access yet.\n" + "Aborting.\n"); + return 1; +#endif } int internal_shutdown(void) Added: trunk/processor_enable.c ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ trunk/processor_enable.c Fri Jun 4 21:05:39 2010 (r1031) @@ -0,0 +1,45 @@ +/* + * This file is part of the flashrom project. + * + * Copyright (C) 2010 Carl-Daniel Hailfinger + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +/* + * Contains the processor specific flash enables and system settings. + */ + +#include "flash.h" + +#if defined(__i386__) || defined(__x86_64__) + +int processor_flash_enable(void) +{ + /* On x86, flash access is not processor specific except on + * AMD Elan SC520, AMD Geode and maybe other SoC-style CPUs. + * FIXME: Move enable_flash_cs5536 and get_flashbase_sc520 here. + */ + return 0; +} + +#else + +int processor_flash_enable(void) +{ + /* Not implemented yet. Oh well. */ + return 1; +} + +#endif

1 0

[PATCH] Make the 'flashrom --help' output look a bit nicer
by Uwe Hermann 04 Jun '10

04 Jun '10

See patch. Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org

2 2

[commit] r1030 - trunk
by repository service 04 Jun '10

04 Jun '10

Author: uwe Date: Fri Jun 4 19:07:39 2010 New Revision: 1030 URL: http://flashrom.org/trac/coreboot/changeset/1030 Log: Make the 'flashrom --help' output look a bit nicer. Only print "flashrom" as the program name unconditionally, not the full path and program name (e.g. "/home/foo/bar/baz/flashrom" or on Windows "C:\Foo\Bar\Whatever\flashrom.exe"). The path or exact executable name is not really useful to print here, if you managed to run --help you already know it, and it just makes the output look ugly. Also, add a missing newline to make the output look nicer. Finally, revert the "CONFIG_PRINT_WIKI ?= yes" change which accidentally slipped into r1029. Signed-off-by: Uwe Hermann <uwe(a)hermann-uwe.de> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Modified: trunk/Makefile trunk/cli_classic.c Modified: trunk/Makefile ============================================================================== --- trunk/Makefile Fri Jun 4 18:39:35 2010 (r1029) +++ trunk/Makefile Fri Jun 4 19:07:39 2010 (r1030) @@ -120,7 +120,7 @@ CONFIG_DEDIPROG ?= no # Disable wiki printing by default. It is only useful if you have wiki access. -CONFIG_PRINT_WIKI ?= yes +CONFIG_PRINT_WIKI ?= no ifeq ($(CONFIG_INTERNAL), yes) FEATURE_CFLAGS += -D'CONFIG_INTERNAL=1' Modified: trunk/cli_classic.c ============================================================================== --- trunk/cli_classic.c Fri Jun 4 18:39:35 2010 (r1029) +++ trunk/cli_classic.c Fri Jun 4 19:07:39 2010 (r1030) @@ -38,14 +38,13 @@ int remaining = 0; enum programmer p; - printf("Usage: %s [-n] [-V] [-f] [-h|-R|-L|" + printf("Usage: flashrom [-n] [-V] [-f] [-h|-R|-L|" #if CONFIG_PRINT_WIKI == 1 "-z|" #endif "-E|-r <file>|-w <file>|-v <file>]\n" " [-c <chipname>] [-m [<vendor>:]<part>] [-l <file>]\n" - " [-i <image>] [-p <programmername>[:<parameters>]]\n", - name); + " [-i <image>] [-p <programmername>[:<parameters>]]\n\n"); printf("Please note that the command line interface for flashrom has " "changed between\n" @@ -119,9 +118,9 @@ "flash chips.\n\n"); } -void cli_classic_abort_usage(const char *name) +void cli_classic_abort_usage(void) { - printf("Please run \"%s --help\" for usage info.\n", name); + printf("Please run \"flashrom --help\" for usage info.\n"); exit(1); } @@ -185,7 +184,7 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } filename = strdup(optarg); read_it = 1; @@ -194,7 +193,7 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } filename = strdup(optarg); write_it = 1; @@ -204,12 +203,12 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } if (dont_verify_it) { fprintf(stderr, "--verify and --noverify are" "mutually exclusive. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } filename = strdup(optarg); verify_it = 1; @@ -218,7 +217,7 @@ if (verify_it) { fprintf(stderr, "--verify and --noverify are" "mutually exclusive. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } dont_verify_it = 1; break; @@ -232,7 +231,7 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } erase_it = 1; break; @@ -244,7 +243,7 @@ fprintf(stderr, "Error: Internal programmer support " "was not compiled in and --mainboard only\n" "applies to the internal programmer. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); #endif break; case 'f': @@ -253,7 +252,7 @@ case 'l': tempstr = strdup(optarg); if (read_romlayout(tempstr)) - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); break; case 'i': tempstr = strdup(optarg); @@ -263,7 +262,7 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } list_supported = 1; break; @@ -272,13 +271,13 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } list_supported_wiki = 1; #else fprintf(stderr, "Error: Wiki output was not compiled " "in. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); #endif break; case 'p': @@ -310,7 +309,7 @@ if (programmer == PROGRAMMER_INVALID) { fprintf(stderr, "Error: Unknown programmer " "%s.\n", optarg); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } break; case 'R': @@ -318,7 +317,7 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } exit(0); break; @@ -326,13 +325,13 @@ if (++operation_specified > 1) { fprintf(stderr, "More than one operation " "specified. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } cli_classic_usage(argv[0]); exit(0); break; default: - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); break; } } @@ -353,14 +352,14 @@ if (optind < argc) { fprintf(stderr, "Error: Extra parameter found.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } #if CONFIG_INTERNAL == 1 if ((programmer != PROGRAMMER_INTERNAL) && (lb_part || lb_vendor)) { fprintf(stderr, "Error: --mainboard requires the internal " "programmer. Aborting.\n"); - cli_classic_abort_usage(argv[0]); + cli_classic_abort_usage(); } #endif

1 0

[PATCH 1/1] Fix links for Gigabyte motherboards (2nd try)
by Peter Lemenkov 04 Jun '10

04 Jun '10

2 1

[commit] r1029 - trunk
by repository service 04 Jun '10

04 Jun '10

Author: uwe Date: Fri Jun 4 18:39:35 2010 New Revision: 1029 URL: http://flashrom.org/trac/coreboot/changeset/1029 Log: Fix URLs for GIGABYTE motherboards. Signed-off-by: Peter Lemenkov <lemenkov(a)gmail.com> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Modified: trunk/Makefile trunk/print.c Modified: trunk/Makefile ============================================================================== --- trunk/Makefile Thu Jun 3 23:48:13 2010 (r1028) +++ trunk/Makefile Fri Jun 4 18:39:35 2010 (r1029) @@ -120,7 +120,7 @@ CONFIG_DEDIPROG ?= no # Disable wiki printing by default. It is only useful if you have wiki access. -CONFIG_PRINT_WIKI ?= no +CONFIG_PRINT_WIKI ?= yes ifeq ($(CONFIG_INTERNAL), yes) FEATURE_CFLAGS += -D'CONFIG_INTERNAL=1' Modified: trunk/print.c ============================================================================== --- trunk/print.c Thu Jun 3 23:48:13 2010 (r1028) +++ trunk/print.c Fri Jun 4 18:39:35 2010 (r1029) @@ -332,24 +332,24 @@ B("EPoX", "EP-BX3", 1, "http://www.epox.com/product.asp?ID=EP-BX3", NULL), B("FIC", "VA-502", 0, "ftp://ftp.fic.com.tw/motherboard/manual/socket7/va-502/", NULL), B("GIGABYTE", "GA-2761GXDK", 1, "http://www.computerbase.de/news/hardware/mainboards/amd-systeme/2007/mai/gi…", NULL), - B("GIGABYTE", "GA-6BXC", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-6BXDU", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-6ZMA", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-7VT600", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-7ZM", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", "Works fine if you remove jumper JP9 on the board and disable the flash protection BIOS option."), - B("GIGABYTE", "GA-965P-DS4", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-EP35-DS3L", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-EX58-UD4P", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-K8N-SLI", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-M57SLI-S4", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-M61P-S3", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-MA69VM-S2", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-MA770T-UD3P", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-MA78G-DS3H", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), /* TODO: Rev 1.x or 2.x? */ - B("GIGABYTE", "GA-MA78GM-S2H", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), /* TODO: Rev. 1.0, 1.1, or 2.x? */ - B("GIGABYTE", "GA-MA78GPM-DS2H", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-MA790FX-DQ6", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), - B("GIGABYTE", "GA-MA790GP-DS4H", 1, "http://www.gigabyte.com.tw/Products/Motherboard/Products_Spec.aspx?ProductI…", NULL), + B("GIGABYTE", "GA-6BXC", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1445", NULL), + B("GIGABYTE", "GA-6BXDU", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1429", NULL), + B("GIGABYTE", "GA-6ZMA", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1541", 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-965P-DS4", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2288", NULL), + B("GIGABYTE", "GA-EP35-DS3L", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2778", NULL), + B("GIGABYTE", "GA-EX58-UD4P", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2986", NULL), + B("GIGABYTE", "GA-K8N-SLI", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=1928", NULL), + B("GIGABYTE", "GA-M57SLI-S4", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2287", NULL), + B("GIGABYTE", "GA-M61P-S3", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2434", NULL), + B("GIGABYTE", "GA-MA69VM-S2", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2500", NULL), + B("GIGABYTE", "GA-MA770T-UD3P", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=3096", NULL), + B("GIGABYTE", "GA-MA78G-DS3H", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2800", NULL), /* TODO: Rev 1.x or 2.x? */ + B("GIGABYTE", "GA-MA78GM-S2H", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2758", NULL), /* TODO: Rev. 1.0, 1.1, or 2.x? */ + B("GIGABYTE", "GA-MA78GPM-DS2H", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2859", NULL), + B("GIGABYTE", "GA-MA790FX-DQ6", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2690", NULL), + B("GIGABYTE", "GA-MA790GP-DS4H", 1, "http://www.gigabyte.com/products/product-page.aspx?pid=2887", NULL), B("HP", "DL145 G3", 1, "http://h20000.www2.hp.com/bizsupport/TechSupport/Document.jsp?objectID=c008…", NULL), B("HP", "Vectra VL400 PC", 1, "http://h20000.www2.hp.com/bizsupport/TechSupport/Document.jsp?objectID=c000…", NULL), B("HP", "Vectra VL420 SFF PC", 1, "http://h20000.www2.hp.com/bizsupport/TechSupport/Document.jsp?objectID=c000…", NULL),

1 0

"""help us finish support for your chipset. Thanks."""
by chris yagami 04 Jun '10

04 Jun '10

flashrom v0.9.1-r946 No coreboot table found. DMI string system-manufacturer: "To Be Filled By O.E.M." DMI string system-product-name: "To Be Filled By O.E.M." DMI string system-version: "To Be Filled By O.E.M." DMI string baseboard-manufacturer: "ASRock" DMI string baseboard-product-name: "N68-S".... Attached file txt

1 0

[PATCH] new windows port
by Stefan Reinauer 04 Jun '10

04 Jun '10

Hi, I separated the direct io access code from the flashrom changes, to make this code easily available for the other coreboot utilities, too. The remaining windows patch is rather small. I'll clean up the directio code some more and make a release asap. Best regards, Stefan -- coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br. Tel.: +49 761 7668825 • Fax: +49 761 7664613 Email: info(a)coresystems.de • http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg • HRB 7656 Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

3 9

[RFC] #ifdef or #if defined()
by Carl-Daniel Hailfinger 04 Jun '10

04 Jun '10

Right now flashrom uses a mix of #ifdef and #if defined() and due to that, readability suffers. Should we use #if defined() everywhere and kill #ifdef? Should we evaluate the preprocessor defines for their value instead of their presence to avoid strange problems once someone defines anything to 0? Regards, Carl-Daniel -- http://www.hailfinger.org/

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom