flashrom

flashrom@flashrom.org

5 participants
9161 discussions

[PATCH] Make programmer_param static
by Carl-Daniel Hailfinger 08 Jul '10

08 Jul '10

Make programmer_param static by converting all users to extract_programmer_param. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: flashrom-programmer_param_static/flash.h =================================================================== --- flashrom-programmer_param_static/flash.h (Revision 1070) +++ flashrom-programmer_param_static/flash.h (Arbeitskopie) @@ -570,7 +570,6 @@ }; extern struct decode_sizes max_rom_decode; extern int programmer_may_write; -extern char *programmer_param; extern unsigned long flashbase; extern int verbose; extern const char * const flashrom_version; @@ -585,6 +584,7 @@ int min(int a, int b); int max(int a, int b); char *extract_param(char **haystack, char *needle, char *delim); +char *extract_programmer_param(char *needle, char *delim); int check_erased_range(struct flashchip *flash, int start, int len); int verify_range(struct flashchip *flash, uint8_t *cmpbuf, int start, int len, char *message); int need_erase(uint8_t *have, uint8_t *want, int len, enum write_granularity gran); Index: flashrom-programmer_param_static/it87spi.c =================================================================== --- flashrom-programmer_param_static/it87spi.c (Revision 1070) +++ flashrom-programmer_param_static/it87spi.c (Arbeitskopie) @@ -143,7 +143,7 @@ flashport |= sio_read(port, 0x65); msg_pdbg("Serial flash port 0x%04x\n", flashport); /* Non-default port requested? */ - portpos = extract_param(&programmer_param, "it87spiport", ",:"); + portpos = extract_programmer_param("it87spiport", ",:"); if (portpos && strlen(portpos)) { flashport = strtol(portpos, (char **)NULL, 0); msg_pinfo("Forcing serial flash port 0x%04x\n", Index: flashrom-programmer_param_static/buspirate_spi.c =================================================================== --- flashrom-programmer_param_static/buspirate_spi.c (Revision 1070) +++ flashrom-programmer_param_static/buspirate_spi.c (Arbeitskopie) @@ -101,14 +101,14 @@ char *speed = NULL; int spispeed = 0x7; - dev = extract_param(&programmer_param, "dev", ",:"); + dev = extract_programmer_param("dev", ",:"); if (!dev || !strlen(dev)) { msg_perr("No serial device given. Use flashrom -p " "buspirate_spi:dev=/dev/ttyUSB0\n"); return 1; } - speed = extract_param(&programmer_param, "spispeed", ",:"); + speed = extract_programmer_param("spispeed", ",:"); if (speed) { for (i = 0; spispeeds[i].name; i++) if (!strncasecmp(spispeeds[i].name, speed, Index: flashrom-programmer_param_static/pcidev.c =================================================================== --- flashrom-programmer_param_static/pcidev.c (Revision 1070) +++ flashrom-programmer_param_static/pcidev.c (Arbeitskopie) @@ -94,7 +94,7 @@ /* Filter by vendor and also bb:dd.f (if supplied by the user). */ filter.vendor = vendor_id; - pcidev_bdf = extract_param(&programmer_param, "pci", ","); + pcidev_bdf = extract_programmer_param("pci", ","); if (pcidev_bdf != NULL) { if ((msg = pci_filter_parse_slot(&filter, pcidev_bdf))) { msg_perr("Error: %s\n", msg); Index: flashrom-programmer_param_static/serprog.c =================================================================== --- flashrom-programmer_param_static/serprog.c (Revision 1070) +++ flashrom-programmer_param_static/serprog.c (Arbeitskopie) @@ -306,7 +306,7 @@ int have_device = 0; /* the parameter is either of format "dev=/dev/device:baud" or "ip=ip:port" */ - device = extract_param(&programmer_param, "dev", ","); + device = extract_programmer_param("dev", ","); if (device && strlen(device)) { baudport = strstr(device, ":"); if (baudport) { @@ -333,7 +333,7 @@ } free(device); - device = extract_param(&programmer_param, "ip", ","); + device = extract_programmer_param("ip", ","); if (have_device && device) { msg_perr("Error: Both host and device specified.\n" "Please use either dev= or ip= but not both.\n"); Index: flashrom-programmer_param_static/ft2232_spi.c =================================================================== --- flashrom-programmer_param_static/ft2232_spi.c (Revision 1070) +++ flashrom-programmer_param_static/ft2232_spi.c (Arbeitskopie) @@ -80,7 +80,7 @@ enum ftdi_interface ft2232_interface = INTERFACE_B; char *arg; - arg = extract_param(&programmer_param, "type", ",:"); + arg = extract_programmer_param("type", ",:"); if (arg) { if (!strcasecmp(arg, "2232H")) ft2232_type = FTDI_FT2232H; @@ -93,7 +93,7 @@ } } free(arg); - arg = extract_param(&programmer_param, "port", ",:"); + arg = extract_programmer_param("port", ",:"); if (arg) { switch (toupper(*arg)) { case 'A': Index: flashrom-programmer_param_static/dummyflasher.c =================================================================== --- flashrom-programmer_param_static/dummyflasher.c (Revision 1070) +++ flashrom-programmer_param_static/dummyflasher.c (Arbeitskopie) @@ -37,7 +37,7 @@ msg_pspew("%s\n", __func__); - bustext = extract_param(&programmer_param, "bus", ",:"); + bustext = extract_programmer_param("bus", ",:"); msg_pdbg("Requested buses are: %s\n", bustext ? bustext : "default"); if (!bustext) bustext = strdup("parallel+lpc+fwh+spi"); Index: flashrom-programmer_param_static/chipset_enable.c =================================================================== --- flashrom-programmer_param_static/chipset_enable.c (Revision 1070) +++ flashrom-programmer_param_static/chipset_enable.c (Arbeitskopie) @@ -300,7 +300,7 @@ int max_decode_fwh_decode = 0; int contiguous = 1; - idsel = extract_param(&programmer_param, "fwh_idsel", ",:"); + idsel = extract_programmer_param("fwh_idsel", ",:"); if (idsel && strlen(idsel)) { fwh_conf = (uint32_t)strtoul(idsel, NULL, 0); Index: flashrom-programmer_param_static/flashrom.c =================================================================== --- flashrom-programmer_param_static/flashrom.c (Revision 1070) +++ flashrom-programmer_param_static/flashrom.c (Arbeitskopie) @@ -89,7 +89,7 @@ ; #endif -char *programmer_param = NULL; +static char *programmer_param = NULL; /* Supported buses for the current programmer. */ enum chipbustype buses_supported; @@ -641,6 +641,11 @@ return opt; } +char *extract_programmer_param(char *needle, char *delim) +{ + return extract_param(&programmer_param, needle, delim); +} + /* start is an offset to the base address of the flash chip */ int check_erased_range(struct flashchip *flash, int start, int len) { Index: flashrom-programmer_param_static/internal.c =================================================================== --- flashrom-programmer_param_static/internal.c (Revision 1070) +++ flashrom-programmer_param_static/internal.c (Arbeitskopie) @@ -121,7 +121,7 @@ int force_laptop = 0; char *arg; - arg = extract_param(&programmer_param, "boardenable", ",:"); + arg = extract_programmer_param("boardenable", ",:"); if (arg && !strcmp(arg,"force")) { force_boardenable = 1; } else if (arg && !strlen(arg)) { @@ -135,7 +135,7 @@ } free(arg); - arg = extract_param(&programmer_param, "boardmismatch", ",:"); + arg = extract_programmer_param("boardmismatch", ",:"); if (arg && !strcmp(arg,"force")) { force_boardmismatch = 1; } else if (arg && !strlen(arg)) { @@ -149,7 +149,7 @@ } free(arg); - arg = extract_param(&programmer_param, "laptop", ",:"); + arg = extract_programmer_param("laptop", ",:"); if (arg && !strcmp(arg,"force_I_want_a_brick")) { force_laptop = 1; } else if (arg && !strlen(arg)) { -- http://www.hailfinger.org/

2 4

[commit] r1072 - trunk
by repository service 08 Jul '10

08 Jul '10

Author: hailfinger Date: Thu Jul 8 12:13:37 2010 New Revision: 1072 URL: http://flashrom.org/trac/coreboot/changeset/1072 Log: Make programmer_param static by converting all users to extract_programmer_param. Programmer parameters can no longer be separated with a colon, they have to be separated with a comma. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Acked-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> Modified: trunk/buspirate_spi.c trunk/chipset_enable.c trunk/dummyflasher.c trunk/flash.h trunk/flashrom.c trunk/ft2232_spi.c trunk/internal.c trunk/it87spi.c trunk/pcidev.c trunk/serprog.c Modified: trunk/buspirate_spi.c ============================================================================== --- trunk/buspirate_spi.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/buspirate_spi.c Thu Jul 8 12:13:37 2010 (r1072) @@ -101,14 +101,14 @@ char *speed = NULL; int spispeed = 0x7; - dev = extract_param(&programmer_param, "dev", ",:"); + dev = extract_programmer_param("dev"); if (!dev || !strlen(dev)) { msg_perr("No serial device given. Use flashrom -p " "buspirate_spi:dev=/dev/ttyUSB0\n"); return 1; } - speed = extract_param(&programmer_param, "spispeed", ",:"); + speed = extract_programmer_param("spispeed"); if (speed) { for (i = 0; spispeeds[i].name; i++) if (!strncasecmp(spispeeds[i].name, speed, Modified: trunk/chipset_enable.c ============================================================================== --- trunk/chipset_enable.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/chipset_enable.c Thu Jul 8 12:13:37 2010 (r1072) @@ -300,7 +300,7 @@ int max_decode_fwh_decode = 0; int contiguous = 1; - idsel = extract_param(&programmer_param, "fwh_idsel", ",:"); + idsel = extract_programmer_param("fwh_idsel"); if (idsel && strlen(idsel)) { fwh_conf = (uint32_t)strtoul(idsel, NULL, 0); Modified: trunk/dummyflasher.c ============================================================================== --- trunk/dummyflasher.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/dummyflasher.c Thu Jul 8 12:13:37 2010 (r1072) @@ -37,7 +37,7 @@ msg_pspew("%s\n", __func__); - bustext = extract_param(&programmer_param, "bus", ",:"); + bustext = extract_programmer_param("bus"); msg_pdbg("Requested buses are: %s\n", bustext ? bustext : "default"); if (!bustext) bustext = strdup("parallel+lpc+fwh+spi"); Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/flash.h Thu Jul 8 12:13:37 2010 (r1072) @@ -570,7 +570,6 @@ }; extern struct decode_sizes max_rom_decode; extern int programmer_may_write; -extern char *programmer_param; extern unsigned long flashbase; extern int verbose; extern const char * const flashrom_version; @@ -585,6 +584,7 @@ int min(int a, int b); int max(int a, int b); char *extract_param(char **haystack, char *needle, char *delim); +char *extract_programmer_param(char *param_name); int check_erased_range(struct flashchip *flash, int start, int len); int verify_range(struct flashchip *flash, uint8_t *cmpbuf, int start, int len, char *message); int need_erase(uint8_t *have, uint8_t *want, int len, enum write_granularity gran); Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/flashrom.c Thu Jul 8 12:13:37 2010 (r1072) @@ -89,7 +89,7 @@ ; #endif -char *programmer_param = NULL; +static char *programmer_param = NULL; /* Supported buses for the current programmer. */ enum chipbustype buses_supported; @@ -641,6 +641,11 @@ return opt; } +char *extract_programmer_param(char *param_name) +{ + return extract_param(&programmer_param, param_name, ","); +} + /* start is an offset to the base address of the flash chip */ int check_erased_range(struct flashchip *flash, int start, int len) { Modified: trunk/ft2232_spi.c ============================================================================== --- trunk/ft2232_spi.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/ft2232_spi.c Thu Jul 8 12:13:37 2010 (r1072) @@ -80,7 +80,7 @@ enum ftdi_interface ft2232_interface = INTERFACE_B; char *arg; - arg = extract_param(&programmer_param, "type", ",:"); + arg = extract_programmer_param("type"); if (arg) { if (!strcasecmp(arg, "2232H")) ft2232_type = FTDI_FT2232H; @@ -93,7 +93,7 @@ } } free(arg); - arg = extract_param(&programmer_param, "port", ",:"); + arg = extract_programmer_param("port"); if (arg) { switch (toupper(*arg)) { case 'A': Modified: trunk/internal.c ============================================================================== --- trunk/internal.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/internal.c Thu Jul 8 12:13:37 2010 (r1072) @@ -121,7 +121,7 @@ int force_laptop = 0; char *arg; - arg = extract_param(&programmer_param, "boardenable", ",:"); + arg = extract_programmer_param("boardenable"); if (arg && !strcmp(arg,"force")) { force_boardenable = 1; } else if (arg && !strlen(arg)) { @@ -135,7 +135,7 @@ } free(arg); - arg = extract_param(&programmer_param, "boardmismatch", ",:"); + arg = extract_programmer_param("boardmismatch"); if (arg && !strcmp(arg,"force")) { force_boardmismatch = 1; } else if (arg && !strlen(arg)) { @@ -149,7 +149,7 @@ } free(arg); - arg = extract_param(&programmer_param, "laptop", ",:"); + arg = extract_programmer_param("laptop"); if (arg && !strcmp(arg,"force_I_want_a_brick")) { force_laptop = 1; } else if (arg && !strlen(arg)) { Modified: trunk/it87spi.c ============================================================================== --- trunk/it87spi.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/it87spi.c Thu Jul 8 12:13:37 2010 (r1072) @@ -143,7 +143,7 @@ flashport |= sio_read(port, 0x65); msg_pdbg("Serial flash port 0x%04x\n", flashport); /* Non-default port requested? */ - portpos = extract_param(&programmer_param, "it87spiport", ",:"); + portpos = extract_programmer_param("it87spiport"); if (portpos && strlen(portpos)) { flashport = strtol(portpos, (char **)NULL, 0); msg_pinfo("Forcing serial flash port 0x%04x\n", Modified: trunk/pcidev.c ============================================================================== --- trunk/pcidev.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/pcidev.c Thu Jul 8 12:13:37 2010 (r1072) @@ -94,7 +94,7 @@ /* Filter by vendor and also bb:dd.f (if supplied by the user). */ filter.vendor = vendor_id; - pcidev_bdf = extract_param(&programmer_param, "pci", ","); + pcidev_bdf = extract_programmer_param("pci"); if (pcidev_bdf != NULL) { if ((msg = pci_filter_parse_slot(&filter, pcidev_bdf))) { msg_perr("Error: %s\n", msg); Modified: trunk/serprog.c ============================================================================== --- trunk/serprog.c Thu Jul 8 11:32:18 2010 (r1071) +++ trunk/serprog.c Thu Jul 8 12:13:37 2010 (r1072) @@ -306,7 +306,7 @@ int have_device = 0; /* the parameter is either of format "dev=/dev/device:baud" or "ip=ip:port" */ - device = extract_param(&programmer_param, "dev", ","); + device = extract_programmer_param("dev"); if (device && strlen(device)) { baudport = strstr(device, ":"); if (baudport) { @@ -333,7 +333,7 @@ } free(device); - device = extract_param(&programmer_param, "ip", ","); + device = extract_programmer_param("ip"); if (have_device && device) { msg_perr("Error: Both host and device specified.\n" "Please use either dev= or ip= but not both.\n");

1 0

Re: [flashrom] [PATCH] Fix ASUS A8N-VM CSM board enable entry
by Carl-Daniel Hailfinger 08 Jul '10

08 Jul '10

On 08.07.2010 00:43, Michael Karcher wrote: > r1063 had several issues: The PCI IDs for this board are copy/pasted from > the A8N and plain wrong for this board and the board enable is marked as > tested although it isn't. Finally the board description was slightly > wrong and the URL missing. > > Signed-off-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> > Your explanation looks good, but I don't own the hardware. Still... Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Regards, Carl-Daniel -- http://www.hailfinger.org/

2 1

[commit] r1071 - trunk
by repository service 08 Jul '10

08 Jul '10

Author: mkarcher Date: Thu Jul 8 11:32:18 2010 New Revision: 1071 URL: http://flashrom.org/trac/coreboot/changeset/1071 Log: Fix ASUS A8N-VM CSM board enable entry r1063 had several issues: The PCI IDs for this board are copy/pasted from the A8N and plain wrong for this board and the board enable is marked as tested although it isn't. Finally the board description was slightly wrong and the URL missing. Signed-off-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Modified: trunk/board_enable.c trunk/print.c Modified: trunk/board_enable.c ============================================================================== --- trunk/board_enable.c Tue Jul 6 11:55:48 2010 (r1070) +++ trunk/board_enable.c Thu Jul 8 11:32:18 2010 (r1071) @@ -321,7 +321,7 @@ * Winbond W83627EHF: Raise GPIO24. * * Suited for: - * - Asus A8N VM CSM + * - Asus A8N-VM CSM: AMD Socket 939 + GeForce 6150 (C51) + MCP51. */ static int w83627ehf_gpio24_raise_2e() { @@ -1601,7 +1601,7 @@ {0x8086, 0x27A0, 0x1043, 0x1287, 0x8086, 0x27DF, 0x1043, 0x1287, "^A8J", NULL, NULL, "ASUS", "A8JM", 0, NT, intel_ich_gpio34_raise}, {0x10DE, 0x0260, 0x103c, 0x2a3e, 0x10DE, 0x0264, 0x103c, 0x2a3e, "NAGAMI", NULL, NULL, "ASUS", "A8N-LA", 0, NT, nvidia_mcp_gpio0_raise}, {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, NULL, NULL, NULL, "ASUS", "A8N", 0, NT, board_shuttle_fn25}, - {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, NULL, NULL, NULL, "ASUS", "A8N-VM CSM", 0, OK, w83627ehf_gpio24_raise_2e}, + {0x10de, 0x0264, 0x1043, 0x81bc, 0x10de, 0x02f0, 0x1043, 0x81cd, NULL, NULL, NULL, "ASUS", "A8N-VM CSM", 0, NT, w83627ehf_gpio24_raise_2e}, {0x10DE, 0x0264, 0x1043, 0x81C0, 0x10DE, 0x0260, 0x1043, 0x81C0, NULL, NULL, NULL, "ASUS", "M2NBP-VM CSM", 0, OK, nvidia_mcp_gpio0_raise}, {0x1106, 0x1336, 0x1043, 0x80ed, 0x1106, 0x3288, 0x1043, 0x8249, NULL, NULL, NULL, "ASUS", "M2V-MX", 0, OK, via_vt823x_gpio5_raise}, {0x8086, 0x1A30, 0x1043, 0x8025, 0x8086, 0x244B, 0x104D, 0x80F0, NULL, NULL, NULL, "ASUS", "P4B266-LM", 0, OK, intel_ich_gpio21_raise}, Modified: trunk/print.c ============================================================================== --- trunk/print.c Tue Jul 6 11:55:48 2010 (r1070) +++ trunk/print.c Thu Jul 8 11:32:18 2010 (r1071) @@ -288,6 +288,7 @@ B("ASUS", "A8N-E", 1, "http://www.asus.com/product.aspx?P_ID=DzbA8hgqchMBOVRz", NULL), B("ASUS", "A8N-SLI", 1, "http://www.asus.com/product.aspx?P_ID=J9FKa8z2xVId3pDK", NULL), B("ASUS", "A8N-SLI Premium", 1, "http://www.asus.com/product.aspx?P_ID=nbulqxniNmzf0mH1", NULL), + B("ASUS", "A8N-VM CSM", 1, "http://www.asus.com/product.aspx?P_ID=JBqqlpj4cspbSa3s", NULL), B("ASUS", "A8NE-FM/S", 1, "http://www.hardwareschotte.de/hardware/preise/proid_1266090/preis_ASUS+A8NE…", NULL), B("ASUS", "A8V Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=tvpdgPNCPaABZRVU", NULL), B("ASUS", "A8V-E Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=hQBPIJWEZnnGAZEh", NULL),

1 0

Re: [flashrom] [PATCH] Board enable for HP Vectra VL420SFF
by Mattias Mattsson 08 Jul '10

08 Jul '10

On Fri, Feb 12, 2010 at 05:09:59PM +0100, Michael Karcher wrote: > --- > Hello Mattias, could you please try whether current svn flashrom (at least > r896) with this patch applied outputs > Disabling write protection for HP VL420 SFF > on invocation with "flashrom -V"? If yes, please reply to this mail with a > mail containing the line "Acked-by: Mattias Mattsson <vitplister(a)gmail.com>". > You don't have to try anything dangerous like erasing/writing the flash > chip, as only the board recognition has changed. Tested with latest flashrom (r1070). Read, write, erase and verify works as expected. Logs attached. Many thanks for your work and sorry for taking so long to test. -mattias

1 0

[PATCH] Fix ASUS A8N-VM CSM board enable entry
by Michael Karcher 08 Jul '10

08 Jul '10

r1063 had several issues: The PCI IDs for this board are copy/pasted from the A8N and plain wrong for this board and the board enable is marked as tested although it isn't. Finally the board description was slightly wrong and the URL missing. Signed-off-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> --- board_enable.c | 4 ++-- print.c | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/board_enable.c b/board_enable.c index 44267b0..1c12b51 100644 --- a/board_enable.c +++ b/board_enable.c @@ -321,7 +321,7 @@ static int w83627hf_gpio24_raise_2e() * Winbond W83627EHF: Raise GPIO24. * * Suited for: - * - Asus A8N VM CSM + * - Asus A8N-VM CSM: AMD Socket 939 + GeForce 6150 (C51) + MCP51. */ static int w83627ehf_gpio24_raise_2e() { @@ -1601,7 +1601,7 @@ const struct board_pciid_enable board_pciid_enables[] = { {0x8086, 0x27A0, 0x1043, 0x1287, 0x8086, 0x27DF, 0x1043, 0x1287, "^A8J", NULL, NULL, "ASUS", "A8JM", 0, NT, intel_ich_gpio34_raise}, {0x10DE, 0x0260, 0x103c, 0x2a3e, 0x10DE, 0x0264, 0x103c, 0x2a3e, "NAGAMI", NULL, NULL, "ASUS", "A8N-LA", 0, NT, nvidia_mcp_gpio0_raise}, {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, NULL, NULL, NULL, "ASUS", "A8N", 0, NT, board_shuttle_fn25}, - {0x10DE, 0x005E, 0x1043, 0x815A, 0x10DE, 0x0054, 0x1043, 0x815A, NULL, NULL, NULL, "ASUS", "A8N-VM CSM", 0, OK, w83627ehf_gpio24_raise_2e}, + {0x10de, 0x0264, 0x1043, 0x81bc, 0x10de, 0x02f0, 0x1043, 0x81cd, NULL, NULL, NULL, "ASUS", "A8N-VM CSM", 0, NT, w83627ehf_gpio24_raise_2e}, {0x10DE, 0x0264, 0x1043, 0x81C0, 0x10DE, 0x0260, 0x1043, 0x81C0, NULL, NULL, NULL, "ASUS", "M2NBP-VM CSM", 0, OK, nvidia_mcp_gpio0_raise}, {0x1106, 0x1336, 0x1043, 0x80ed, 0x1106, 0x3288, 0x1043, 0x8249, NULL, NULL, NULL, "ASUS", "M2V-MX", 0, OK, via_vt823x_gpio5_raise}, {0x8086, 0x1A30, 0x1043, 0x8025, 0x8086, 0x244B, 0x104D, 0x80F0, NULL, NULL, NULL, "ASUS", "P4B266-LM", 0, OK, intel_ich_gpio21_raise}, diff --git a/print.c b/print.c index 3bb14ed..65ff1e3 100644 --- a/print.c +++ b/print.c @@ -288,6 +288,7 @@ const struct board_info boards_known[] = { B("ASUS", "A8N-E", 1, "http://www.asus.com/product.aspx?P_ID=DzbA8hgqchMBOVRz", NULL), B("ASUS", "A8N-SLI", 1, "http://www.asus.com/product.aspx?P_ID=J9FKa8z2xVId3pDK", NULL), B("ASUS", "A8N-SLI Premium", 1, "http://www.asus.com/product.aspx?P_ID=nbulqxniNmzf0mH1", NULL), + B("ASUS", "A8N-VM CSM", 1, "http://www.asus.com/product.aspx?P_ID=JBqqlpj4cspbSa3s", NULL), B("ASUS", "A8NE-FM/S", 1, "http://www.hardwareschotte.de/hardware/preise/proid_1266090/preis_ASUS+A8NE…", NULL), B("ASUS", "A8V Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=tvpdgPNCPaABZRVU", NULL), B("ASUS", "A8V-E Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=hQBPIJWEZnnGAZEh", NULL), -- 1.7.1

1 0

[PATCH] Board enable for ASUS P5PE-VM
by Michael Karcher 08 Jul '10

08 Jul '10

Signed-off-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> --- board_enable.c | 2 ++ print.c | 1 + 2 files changed, 3 insertions(+), 0 deletions(-) diff --git a/board_enable.c b/board_enable.c index 44267b0..a32a114 100644 --- a/board_enable.c +++ b/board_enable.c @@ -1189,6 +1189,7 @@ static int intel_ich_gpio19_raise(void) * - ASUS P4B266LM (Sony Vaio PCV-RX650): socket478 + 845D + ICH2. * - ASUS P4C800-E Deluxe: socket478 + 875P + ICH5. * - ASUS P4P800-E Deluxe: Intel socket478 + 865PE + ICH5R. + * - ASUS P5PE-VM: Intel LGA775 + 865G + ICH5. */ static int intel_ich_gpio21_raise(void) { @@ -1611,6 +1612,7 @@ const struct board_pciid_enable board_pciid_enables[] = { {0x8086, 0x2570, 0x1043, 0x80F2, 0x105A, 0x3373, 0x1043, 0x80F5, NULL, NULL, NULL, "ASUS", "P4P800-E Deluxe", 0, OK, intel_ich_gpio21_raise}, {0x10B9, 0x1541, 0, 0, 0x10B9, 0x1533, 0, 0, "^P5A$", "asus", "p5a", "ASUS", "P5A", 0, OK, board_asus_p5a}, {0x10DE, 0x0030, 0x1043, 0x818a, 0x8086, 0x100E, 0x1043, 0x80EE, NULL, NULL, NULL, "ASUS", "P5ND2-SLI Deluxe", 0, OK, nvidia_mcp_gpio10_raise}, + {0x8086, 0x24dd, 0x1043, 0x80a6, 0x8086, 0x2570, 0x1043, 0x8157, NULL, NULL, NULL, "ASUS", "P5PE-VM", 0, OK, intel_ich_gpio21_raise}, {0x1106, 0x3149, 0x1565, 0x3206, 0x1106, 0x3344, 0x1565, 0x1202, NULL, NULL, NULL, "Biostar", "P4M80-M4", 0, OK, it8705f_write_enable_2e}, {0x8086, 0x3590, 0x1028, 0x016c, 0x1000, 0x0030, 0x1028, 0x016c, NULL, NULL, NULL, "Dell", "PowerEdge 1850", 0, OK, intel_ich_gpio23_raise}, {0x1039, 0x5513, 0x1019, 0x0A41, 0x1039, 0x0018, 0, 0, NULL, NULL, NULL, "Elitegroup", "K7S6A", 0, OK, it8705f_write_enable_2e}, diff --git a/print.c b/print.c index 3bb14ed..a996bcb 100644 --- a/print.c +++ b/print.c @@ -321,6 +321,7 @@ const struct board_info boards_known[] = { 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", "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), B("ASUS", "P6T Deluxe", 1, "http://www.asus.com/product.aspx?P_ID=vXixf82co6Q5v0BZ", NULL), B("ASUS", "P6T Deluxe V2", 1, "http://www.asus.com/product.aspx?P_ID=iRlP8RG9han6saZx", NULL), B("BCOM", "WinNET100", 1, "http://www.coreboot.org/BCOM_WINNET100", "Used in the IGEL-316 thin client."), -- 1.7.1

1 0

gigabyte dual bios programming
by Qing Pei Wang 06 Jul '10

06 Jul '10

hi all, Since i am trying to do my 780 mass porting, i faced a problem about Gigabyte dual bios mainboard. my SF100 programmer can not detect the spi chip unless i removed it from the mainboard, but i can not use this method to test my coreboot code for i would program the chip lots of time. Is there any idea about this? it's pretty strange about these dual bios things. Any suggestion will be welcome for me. -- Wang Qing Pei MSN:wangqingpei@hotmail.com <MSN%3Awangqingpei(a)hotmail.com> Gmail:wangqingpei@gmail.com <Gmail%3Awangqingpei(a)gmail.com> Phone:86+13426369984

2 1

[PATCH] Use extract_param everywhere
by Carl-Daniel Hailfinger 06 Jul '10

06 Jul '10

Various places in the flashrom source feature custom parameter extraction from programmer_param. This led to wildly differing syntax for programmer parameters, and it also voids pretty much every assumption you could make about programmer_param. The latter is a problem for libflashrom. Use extract_param everywhere, clean up related code and make it more foolproof. Add two instances of exit(1) where we have no option to return an error. Remove six instances of exit(1) where returning an error was possible. Once this patch is in, it is very easy to make programmer_param static. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: flashrom-extract_param_everywhere/flashrom.8 =================================================================== --- flashrom-extract_param_everywhere/flashrom.8 (Revision 1067) +++ flashrom-extract_param_everywhere/flashrom.8 (Arbeitskopie) @@ -183,7 +183,7 @@ .BR "* ft2232_spi" " (for SPI flash ROMs attached to a FT2232H/FT4232H based\ USB SPI programmer)" .sp -.BR "* serprog" " (for flash ROMs attached to Urja's AVR programmer)" +.BR "* serprog" " (for flash ROMs attached to a programmer speaking serprog)" .sp .BR "* buspirate_spi" " (for SPI flash ROMs attached to a Bus Pirate)" .sp @@ -296,22 +296,23 @@ .BR "dummy " programmer An optional parameter specifies the bus types it should support. For that you have to use the -.B "flashrom \-p dummy:type" +.B "flashrom \-p dummy:bus=[type[+type[+type]]]" syntax where .B type -can be any comma-separated combination of -.BR parallel ", " lpc ", " fwh ", " spi ", or " all -(in any order). +can be any of +.BR parallel ", " lpc ", " fwh ", " spi +in any order. If you specify bus without type, all buses will be disabled. +If you do not specify bus, all buses will be enabled. .sp Example: -.B "flashrom \-p dummy:lpc,fwh" +.B "flashrom \-p dummy:bus=lpc+fwh" .TP .BR "nic3com" , " nicrealtek" , " nicsmc1211" , " gfxnvidia" , " satasii \ " and " atahpt " programmers These programmers have an option to specify the PCI address of the card your want to use, which must be specified if more than one card supported by the selected programmer is installed in your system. The syntax is -.BR "flashrom \-p xxxx:bb:dd.f" , +.BR "flashrom \-p xxxx:pci=bb:dd.f" , where .B xxxx is the name of the programmer @@ -323,7 +324,7 @@ is the PCI function number of the desired device. .sp Example: -.B "flashrom \-p nic3com:05:04.0" +.B "flashrom \-p nic3com:pci=05:04.0" .TP .BR "it87spi " programmer An optional @@ -342,7 +343,7 @@ An optional parameter specifies the controller type and interface/port it should support. For that you have to use the .sp -.B " flashrom \-p ft2232_spi:model,port=interface" +.B " flashrom \-p ft2232_spi:type=model,port=interface" .sp syntax where .B model @@ -363,11 +364,11 @@ programmer. In the device/baud combination, the device has to start with a slash. For serial, you have to use the .sp -.B " flashrom \-p serprog:/dev/device:baud" +.B " flashrom \-p serprog:dev=/dev/device:baud" .sp syntax and for IP, you have to use .sp -.B " flashrom \-p serprog:ip:port" +.B " flashrom \-p serprog:ip=ipaddr:port" .sp instead. More information about serprog is available in .B serprog-protocol.txt Index: flashrom-extract_param_everywhere/drkaiser.c =================================================================== --- flashrom-extract_param_everywhere/drkaiser.c (Revision 1067) +++ flashrom-extract_param_everywhere/drkaiser.c (Arbeitskopie) @@ -36,10 +36,14 @@ int drkaiser_init(void) { uint32_t addr; + char *devstring; get_io_perms(); + + devstring = extract_param(&programmer_param, "pci", ","); pcidev_init(PCI_VENDOR_ID_DRKAISER, PCI_BASE_ADDRESS_2, - drkaiser_pcidev, programmer_param); + drkaiser_pcidev, devstring); + free(devstring); /* Write magic register to enable flash write. */ pci_write_word(pcidev_dev, PCI_MAGIC_DRKAISER_ADDR, @@ -53,6 +57,7 @@ addr, 128 * 1024); buses_supported = CHIP_BUSTYPE_PARALLEL; + return 0; } @@ -60,7 +65,6 @@ { /* Write protect the flash again. */ pci_write_word(pcidev_dev, PCI_MAGIC_DRKAISER_ADDR, 0); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/it87spi.c =================================================================== --- flashrom-extract_param_everywhere/it87spi.c (Revision 1067) +++ flashrom-extract_param_everywhere/it87spi.c (Arbeitskopie) @@ -142,22 +142,24 @@ flashport = sio_read(port, 0x64) << 8; flashport |= sio_read(port, 0x65); msg_pdbg("Serial flash port 0x%04x\n", flashport); - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; + /* Non-default port requested? */ + portpos = extract_param(&programmer_param, "it87spiport", ",:"); + if (portpos && strlen(portpos)) { + flashport = strtol(portpos, (char **)NULL, 0); + msg_pinfo("Forcing serial flash port 0x%04x\n", + flashport); + sio_write(port, 0x64, (flashport >> 8)); + sio_write(port, 0x65, (flashport & 0xff)); + } else if (portpos) { + msg_perr("Error: it87spiport specified, but no port " + "given.\n"); + free(portpos); + /* FIXME: Return failure here once it87spi_common_init() + * can handle the return value sanely. + */ + exit(1); } - if (programmer_param) { - portpos = extract_param(&programmer_param, - "it87spiport=", ",:"); - if (portpos) { - flashport = strtol(portpos, (char **)NULL, 0); - msg_pinfo("Forcing serial flash port 0x%04x\n", - flashport); - sio_write(port, 0x64, (flashport >> 8)); - sio_write(port, 0x65, (flashport & 0xff)); - free(portpos); - } - } + free(portpos); exit_conf_mode_ite(port); break; /* TODO: Handle more IT87xx if they support flash translation */ Index: flashrom-extract_param_everywhere/buspirate_spi.c =================================================================== --- flashrom-extract_param_everywhere/buspirate_spi.c (Revision 1067) +++ flashrom-extract_param_everywhere/buspirate_spi.c (Arbeitskopie) @@ -101,24 +101,18 @@ char *speed = NULL; int spispeed = 0x7; - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; - } - if (programmer_param) { - dev = extract_param(&programmer_param, "dev=", ",:"); - speed = extract_param(&programmer_param, "spispeed=", ",:"); - if (strlen(programmer_param)) - msg_perr("Unhandled programmer parameters: %s\n", - programmer_param); - free(programmer_param); - programmer_param = NULL; - } - if (!dev) { + dev = extract_param(&programmer_param, "dev", ",:"); + if (!dev || !strlen(dev)) { msg_perr("No serial device given. Use flashrom -p " "buspirate_spi:dev=/dev/ttyUSB0\n"); return 1; } + ret = buspirate_serialport_setup(dev); + if (ret) + return ret; + free(dev); + + speed = extract_param(&programmer_param, "spispeed", ",:"); if (speed) { for (i = 0; spispeeds[i].name; i++) if (!strncasecmp(spispeeds[i].name, speed, @@ -129,13 +123,11 @@ if (!spispeeds[i].name) msg_perr("Invalid SPI speed, using default.\n"); } + free(speed); + /* This works because speeds numbering starts at 0 and is contiguous. */ msg_pdbg("SPI speed is %sHz\n", spispeeds[spispeed].name); - ret = buspirate_serialport_setup(dev); - if (ret) - return ret; - /* This is the brute force version, but it should work. */ for (i = 0; i < 19; i++) { /* Enter raw bitbang mode */ Index: flashrom-extract_param_everywhere/gfxnvidia.c =================================================================== --- flashrom-extract_param_everywhere/gfxnvidia.c (Revision 1067) +++ flashrom-extract_param_everywhere/gfxnvidia.c (Arbeitskopie) @@ -58,11 +58,15 @@ int gfxnvidia_init(void) { uint32_t reg32; + char *devstring; get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_NVIDIA, PCI_BASE_ADDRESS_0, - gfx_nvidia, programmer_param); + gfx_nvidia, devstring); + free(devstring); + io_base_addr += 0x300000; msg_pinfo("Detected NVIDIA I/O base address: 0x%x.\n", io_base_addr); @@ -87,7 +91,6 @@ reg32 |= (1 << 0); pci_write_long(pcidev_dev, 0x50, reg32); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/atahpt.c =================================================================== --- flashrom-extract_param_everywhere/atahpt.c (Revision 1067) +++ flashrom-extract_param_everywhere/atahpt.c (Arbeitskopie) @@ -41,11 +41,14 @@ int atahpt_init(void) { uint32_t reg32; + char *devstring; get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_HPT, PCI_BASE_ADDRESS_4, - ata_hpt, programmer_param); + ata_hpt, devstring); + free(devstring); /* Enable flash access. */ reg32 = pci_read_long(pcidev_dev, REG_FLASH_ACCESS); @@ -66,7 +69,6 @@ reg32 &= ~(1 << 24); pci_write_long(pcidev_dev, REG_FLASH_ACCESS, reg32); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/serprog.c =================================================================== --- flashrom-extract_param_everywhere/serprog.c (Revision 1067) +++ flashrom-extract_param_everywhere/serprog.c (Arbeitskopie) @@ -298,43 +298,80 @@ int serprog_init(void) { uint16_t iface; - int len; unsigned char pgmname[17]; unsigned char rbuf[3]; unsigned char c; - char *num; - char *dev; - msg_pspew("%s\n", __func__); - /* the parameter is either of format "/dev/device:baud" or "ip:port" */ - if ((!programmer_param) || (!strlen(programmer_param))) { - nodevice: - msg_perr("Error: No device/host given for the serial programmer driver.\n" - "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n"); - exit(1); + char *device; + char *baudport; + int have_device = 0; + + /* the parameter is either of format "dev=/dev/device:baud" or "ip=ip:port" */ + device = extract_param(&programmer_param, "dev", ","); + if (device && strlen(device)) { + baudport = strstr(device, ":"); + if (baudport) { + /* Split device from baudrate. */ + *baudport = '\0'; + baudport++; + } + if (!baudport || !strlen(baudport)) { + msg_perr("Error: No baudrate specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud\n"); + free(device); + return 1; + } + if (strlen(device)) { + sp_fd = sp_openserport(device, atoi(baudport)); + have_device++; + } } - num = strstr(programmer_param, ":"); - len = num - programmer_param; - if (!len) goto nodevice; - if (!num) { - msg_perr("Error: No port or baudrate specified to serial programmer driver.\n" - "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n"); - exit(1); + if (device && !strlen(device)) { + msg_perr("Error: No device specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud\n"); + free(device); + return 1; } - len = num - programmer_param; - dev = malloc(len + 1); - if (!dev) sp_die("Error: memory allocation failure"); - memcpy(dev, programmer_param, len); - dev[len] = 0; - num = strdup(num + 1); - if (!num) sp_die("Error: memory allocation failure"); - free(programmer_param); - programmer_param = NULL; + free(device); - if (dev[0] == '/') sp_fd = sp_openserport(dev, atoi(num)); - else sp_fd = sp_opensocket(dev, atoi(num)); + device = extract_param(&programmer_param, "ip", ","); + if (have_device && device) { + msg_perr("Error: Both host and device specified.\n" + "Please use either dev= or ip= but not both.\n"); + free(device); + return 1; + } + if (device && strlen(device)) { + baudport = strstr(device, ":"); + if (baudport) { + /* Split host from port. */ + *baudport = '\0'; + baudport++; + } + if (!baudport || !strlen(baudport)) { + msg_perr("Error: No port specified.\n" + "Use flashrom -p serprog:ip=ipaddr:port\n"); + free(device); + return 1; + } + if (strlen(device)) { + sp_fd = sp_opensocket(device, atoi(baudport)); + have_device++; + } + } + if (device && !strlen(device)) { + msg_perr("Error: No host specified.\n" + "Use flashrom -p serprog:ip=ipaddr:port\n"); + free(device); + return 1; + } + free(device); - free(dev); dev = NULL; - free(num); num = NULL; + if (!have_device) { + msg_perr("Error: Neither host nor device specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud or " + "flashrom -p serprog:ip=ipaddr:port\n"); + return 1; + } msg_pdbg(MSGHEADER "connected - attempting to synchronize\n"); Index: flashrom-extract_param_everywhere/nicrealtek.c =================================================================== --- flashrom-extract_param_everywhere/nicrealtek.c (Revision 1067) +++ flashrom-extract_param_everywhere/nicrealtek.c (Arbeitskopie) @@ -41,10 +41,14 @@ int nicrealtek_init(void) { + char *devstring; + get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_REALTEK, PCI_BASE_ADDRESS_0, - nics_realtek, programmer_param); + nics_realtek, devstring); + free(devstring); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -53,10 +57,14 @@ int nicsmc1211_init(void) { + char *devstring; + get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_SMC1211, PCI_BASE_ADDRESS_0, - nics_realteksmc1211, programmer_param); + nics_realteksmc1211, devstring); + free(devstring); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -66,7 +74,6 @@ int nicrealtek_shutdown(void) { /* FIXME: We forgot to disable software access again. */ - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/nic3com.c =================================================================== --- flashrom-extract_param_everywhere/nic3com.c (Revision 1067) +++ flashrom-extract_param_everywhere/nic3com.c (Arbeitskopie) @@ -56,10 +56,15 @@ int nic3com_init(void) { + char *devstring; + get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_3COM, PCI_BASE_ADDRESS_0, - nics_3com, programmer_param); + nics_3com, devstring); + free(devstring); + id = pcidev_dev->device_id; /* 3COM 3C90xB cards need a special fixup. */ @@ -96,7 +101,6 @@ OUTL(internal_conf, io_base_addr + INTERNAL_CONFIG); } - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/ft2232_spi.c =================================================================== --- flashrom-extract_param_everywhere/ft2232_spi.c (Revision 1067) +++ flashrom-extract_param_everywhere/ft2232_spi.c (Arbeitskopie) @@ -76,46 +76,49 @@ int f; struct ftdi_context *ftdic = &ftdic_context; unsigned char buf[512]; - char *portpos = NULL; int ft2232_type = FTDI_FT4232H; enum ftdi_interface ft2232_interface = INTERFACE_B; + char *arg; - if (ftdi_init(ftdic) < 0) { - msg_perr("ftdi_init failed\n"); - return EXIT_FAILURE; // TODO - } - - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; - } - if (programmer_param) { - if (strstr(programmer_param, "2232")) + arg = extract_param(&programmer_param, "type", ",:"); + if (arg) { + if (!strcmp(arg, "2232H")) ft2232_type = FTDI_FT2232H; - if (strstr(programmer_param, "4232")) + else if (!strcmp(arg, "4232H")) ft2232_type = FTDI_FT4232H; - portpos = strstr(programmer_param, "port="); - if (portpos) { - portpos += 5; - switch (toupper(*portpos)) { - case 'A': - ft2232_interface = INTERFACE_A; - break; - case 'B': - ft2232_interface = INTERFACE_B; - break; - default: - msg_perr("Invalid interface specified, " - "using default.\n"); - } + else { + msg_perr("Error: Invalid device type specified.\n"); + free(arg); + return 1; } - free(programmer_param); } + free(arg); + arg = extract_param(&programmer_param, "port", ",:"); + if (arg) { + switch (toupper(*arg)) { + case 'A': + ft2232_interface = INTERFACE_A; + break; + case 'B': + ft2232_interface = INTERFACE_B; + break; + default: + msg_perr("Error: Invalid port/interface specified.\n"); + free(arg); + return 1; + } + } + free(arg); msg_pdbg("Using device type %s ", (ft2232_type == FTDI_FT2232H) ? "2232H" : "4232H"); msg_pdbg("interface %s\n", (ft2232_interface == INTERFACE_A) ? "A" : "B"); + if (ftdi_init(ftdic) < 0) { + msg_perr("ftdi_init failed\n"); + return EXIT_FAILURE; // TODO + } + f = ftdi_usb_open(ftdic, 0x0403, ft2232_type); if (f < 0 && f != -5) { Index: flashrom-extract_param_everywhere/satasii.c =================================================================== --- flashrom-extract_param_everywhere/satasii.c (Revision 1067) +++ flashrom-extract_param_everywhere/satasii.c (Arbeitskopie) @@ -43,11 +43,15 @@ { uint32_t addr; uint16_t reg_offset; + char *devstring; get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); pcidev_init(PCI_VENDOR_ID_SII, PCI_BASE_ADDRESS_0, satas_sii, - programmer_param); + devstring); + free(devstring); + id = pcidev_dev->device_id; if ((id == 0x3132) || (id == 0x3124)) { @@ -71,7 +75,6 @@ int satasii_shutdown(void) { - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Index: flashrom-extract_param_everywhere/dummyflasher.c =================================================================== --- flashrom-extract_param_everywhere/dummyflasher.c (Revision 1067) +++ flashrom-extract_param_everywhere/dummyflasher.c (Arbeitskopie) @@ -25,44 +25,46 @@ #include "flash.h" #include "chipdrivers.h" +static void tolower_string(char *str) +{ + for (; *str != '\0'; str++) + *str = (char)tolower((unsigned char)*str); +} + int dummy_init(void) { - int i; + char *bustext = NULL; + msg_pspew("%s\n", __func__); - /* "all" is equivalent to specifying no type. */ - if (programmer_param && (!strcmp(programmer_param, "all"))) { - free(programmer_param); - programmer_param = NULL; - } - if (!programmer_param) - programmer_param = strdup("parallel,lpc,fwh,spi"); + bustext = extract_param(&programmer_param, "bus", ",:"); + msg_pdbg("Requested buses are: %s\n", bustext ? bustext : "default"); + if (!bustext) + bustext = strdup("parallel+lpc+fwh+spi"); /* Convert the parameters to lowercase. */ - for (i = 0; programmer_param[i] != '\0'; i++) - programmer_param[i] = - (char)tolower((unsigned char)programmer_param[i]); + tolower_string(bustext); buses_supported = CHIP_BUSTYPE_NONE; - if (strstr(programmer_param, "parallel")) { + if (strstr(bustext, "parallel")) { buses_supported |= CHIP_BUSTYPE_PARALLEL; msg_pdbg("Enabling support for %s flash.\n", "parallel"); } - if (strstr(programmer_param, "lpc")) { + if (strstr(bustext, "lpc")) { buses_supported |= CHIP_BUSTYPE_LPC; msg_pdbg("Enabling support for %s flash.\n", "LPC"); } - if (strstr(programmer_param, "fwh")) { + if (strstr(bustext, "fwh")) { buses_supported |= CHIP_BUSTYPE_FWH; msg_pdbg("Enabling support for %s flash.\n", "FWH"); } - if (strstr(programmer_param, "spi")) { + if (strstr(bustext, "spi")) { buses_supported |= CHIP_BUSTYPE_SPI; spi_controller = SPI_CONTROLLER_DUMMY; msg_pdbg("Enabling support for %s flash.\n", "SPI"); } if (buses_supported == CHIP_BUSTYPE_NONE) msg_pdbg("Support for all flash bus types disabled.\n"); - free(programmer_param); + free(bustext); return 0; } Index: flashrom-extract_param_everywhere/chipset_enable.c =================================================================== --- flashrom-extract_param_everywhere/chipset_enable.c (Revision 1067) +++ flashrom-extract_param_everywhere/chipset_enable.c (Arbeitskopie) @@ -300,11 +300,8 @@ int max_decode_fwh_decode = 0; int contiguous = 1; - if (programmer_param) - idsel = strstr(programmer_param, "fwh_idsel="); - - if (idsel) { - idsel += strlen("fwh_idsel="); + idsel = extract_param(&programmer_param, "fwh_idsel", ",:"); + if (idsel && strlen(idsel)) { fwh_conf = (uint32_t)strtoul(idsel, NULL, 0); /* FIXME: Need to undo this on shutdown. */ @@ -312,7 +309,15 @@ pci_write_long(dev, 0xd0, fwh_conf); pci_write_word(dev, 0xd4, fwh_conf); /* FIXME: Decode settings are not changed. */ + } else if (idsel) { + msg_perr("Error: idsel= specified, but no number given.\n"); + free(idsel); + /* FIXME: Return failure here once internal_init() starts + * to care about the return value of the chipset enable. + */ + exit(1); } + free(idsel); /* Ignore all legacy ranges below 1 MB. * We currently only support flashing the chip which responds to Index: flashrom-extract_param_everywhere/flashrom.c =================================================================== --- flashrom-extract_param_everywhere/flashrom.c (Revision 1067) +++ flashrom-extract_param_everywhere/flashrom.c (Arbeitskopie) @@ -428,7 +428,15 @@ int programmer_init(void) { - return programmer_table[programmer].init(); + int ret; + + ret = programmer_table[programmer].init(); + if (programmer_param && strlen(programmer_param)) { + msg_perr("Unhandled programmer parameters: %s\n", + programmer_param); + ret = 1; + } + return ret; } int programmer_shutdown(void) @@ -536,22 +544,24 @@ char *strcat_realloc(char *dest, const char *src) { dest = realloc(dest, strlen(dest) + strlen(src) + 1); - if (!dest) + if (!dest) { + msg_gerr("Out of memory!\n"); return NULL; + } strcat(dest, src); return dest; } /* This is a somewhat hacked function similar in some ways to strtok(). - * It will look for needle in haystack, return a copy of needle and remove - * everything from the first occurrence of needle to the next delimiter - * from haystack. + * It will look for needle with a subsequent '=' in haystack, return a copy of + * needle and remove everything from the first occurrence of needle to the next + * delimiter from haystack. */ char *extract_param(char **haystack, char *needle, char *delim) { - char *param_pos, *rest, *tmp; - char *dev = NULL; - int devlen; + char *param_pos, *opt_pos, *rest; + char *opt = NULL; + int optlen; int needlelen; needlelen = strlen(needle); @@ -567,43 +577,41 @@ do { if (!param_pos) return NULL; - /* Beginning of the string? */ - if (param_pos == *haystack) - break; - /* After a delimiter? */ - if (strchr(delim, *(param_pos - 1))) - break; + /* Needle followed by '='? */ + if (param_pos[needlelen] == '=') { + + /* Beginning of the string? */ + if (param_pos == *haystack) + break; + /* After a delimiter? */ + if (strchr(delim, *(param_pos - 1))) + break; + } /* Continue searching. */ param_pos++; param_pos = strstr(param_pos, needle); } while (1); - + if (param_pos) { - param_pos += strlen(needle); - devlen = strcspn(param_pos, delim); - if (devlen) { - dev = malloc(devlen + 1); - if (!dev) { - msg_gerr("Out of memory!\n"); - exit(1); - } - strncpy(dev, param_pos, devlen); - dev[devlen] = '\0'; + /* Get the string after needle and '='. */ + opt_pos = param_pos + needlelen + 1; + optlen = strcspn(opt_pos, delim); + /* Return an empty string if the parameter was empty. */ + opt = malloc(optlen + 1); + if (!opt) { + msg_gerr("Out of memory!\n"); + exit(1); } - rest = param_pos + devlen; + strncpy(opt, opt_pos, optlen); + opt[optlen] = '\0'; + rest = opt_pos + optlen; + /* Skip all delimiters after the current parameter. */ rest += strspn(rest, delim); - param_pos -= strlen(needle); memmove(param_pos, rest, strlen(rest) + 1); - tmp = realloc(*haystack, strlen(*haystack) + 1); - if (!tmp) { - msg_gerr("Out of memory!\n"); - exit(1); - } - *haystack = tmp; + /* We could shrink haystack, but the effort is not worth it. */ } - - return dev; + return opt; } /* start is an offset to the base address of the flash chip */ Index: flashrom-extract_param_everywhere/internal.c =================================================================== --- flashrom-extract_param_everywhere/internal.c (Revision 1067) +++ flashrom-extract_param_everywhere/internal.c (Arbeitskopie) @@ -121,36 +121,45 @@ int force_laptop = 0; char *arg; - arg = extract_param(&programmer_param, "boardenable=", ",:"); + arg = extract_param(&programmer_param, "boardenable", ",:"); if (arg && !strcmp(arg,"force")) { force_boardenable = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for boardenable.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for boardenable: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); - arg = extract_param(&programmer_param, "boardmismatch=", ",:"); + arg = extract_param(&programmer_param, "boardmismatch", ",:"); if (arg && !strcmp(arg,"force")) { force_boardmismatch = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for boardmismatch.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for boardmismatch: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); - arg = extract_param(&programmer_param, "laptop=", ",:"); + arg = extract_param(&programmer_param, "laptop", ",:"); if (arg && !strcmp(arg,"force_I_want_a_brick")) { force_laptop = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for laptop.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for laptop: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); Index: flashrom-extract_param_everywhere/nicnatsemi.c =================================================================== --- flashrom-extract_param_everywhere/nicnatsemi.c (Revision 1067) +++ flashrom-extract_param_everywhere/nicnatsemi.c (Arbeitskopie) @@ -36,10 +36,14 @@ int nicnatsemi_init(void) { + char *devstring; + get_io_perms(); + devstring = extract_param(&programmer_param, "pci", ","); io_base_addr = pcidev_init(PCI_VENDOR_ID_NATSEMI, PCI_BASE_ADDRESS_0, - nics_natsemi, programmer_param); + nics_natsemi, devstring); + free(devstring); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -48,7 +52,6 @@ int nicnatsemi_shutdown(void) { - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; -- http://www.hailfinger.org/

2 6

[commit] r1070 - trunk
by repository service 06 Jul '10

06 Jul '10

Author: hailfinger Date: Tue Jul 6 11:55:48 2010 New Revision: 1070 URL: http://flashrom.org/trac/coreboot/changeset/1070 Log: Various places in the flashrom source feature custom parameter extraction from programmer_param. This led to wildly differing syntax for programmer parameters, and it also voids pretty much every assumption you could make about programmer_param. The latter is a problem for libflashrom. Use extract_param everywhere, clean up related code and make it more foolproof. Add two instances of exit(1) where we have no option to return an error. Remove six instances of exit(1) where returning an error was possible. WARNING: This changes programmer parameter syntax for a few programmers! Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Acked-by: Michael Karcher <flashrom(a)mkarcher.dialup.fu-berlin.de> Modified: trunk/atahpt.c trunk/buspirate_spi.c trunk/chipset_enable.c trunk/drkaiser.c trunk/dummyflasher.c trunk/flash.h trunk/flashrom.8 trunk/flashrom.c trunk/ft2232_spi.c trunk/gfxnvidia.c trunk/internal.c trunk/it87spi.c trunk/nic3com.c trunk/nicnatsemi.c trunk/nicrealtek.c trunk/pcidev.c trunk/satasii.c trunk/serprog.c Modified: trunk/atahpt.c ============================================================================== --- trunk/atahpt.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/atahpt.c Tue Jul 6 11:55:48 2010 (r1070) @@ -45,7 +45,7 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_HPT, PCI_BASE_ADDRESS_4, - ata_hpt, programmer_param); + ata_hpt); /* Enable flash access. */ reg32 = pci_read_long(pcidev_dev, REG_FLASH_ACCESS); @@ -66,7 +66,6 @@ reg32 &= ~(1 << 24); pci_write_long(pcidev_dev, REG_FLASH_ACCESS, reg32); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/buspirate_spi.c ============================================================================== --- trunk/buspirate_spi.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/buspirate_spi.c Tue Jul 6 11:55:48 2010 (r1070) @@ -101,24 +101,14 @@ char *speed = NULL; int spispeed = 0x7; - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; - } - if (programmer_param) { - dev = extract_param(&programmer_param, "dev=", ",:"); - speed = extract_param(&programmer_param, "spispeed=", ",:"); - if (strlen(programmer_param)) - msg_perr("Unhandled programmer parameters: %s\n", - programmer_param); - free(programmer_param); - programmer_param = NULL; - } - if (!dev) { + dev = extract_param(&programmer_param, "dev", ",:"); + if (!dev || !strlen(dev)) { msg_perr("No serial device given. Use flashrom -p " "buspirate_spi:dev=/dev/ttyUSB0\n"); return 1; } + + speed = extract_param(&programmer_param, "spispeed", ",:"); if (speed) { for (i = 0; spispeeds[i].name; i++) if (!strncasecmp(spispeeds[i].name, speed, @@ -129,12 +119,15 @@ if (!spispeeds[i].name) msg_perr("Invalid SPI speed, using default.\n"); } + free(speed); + /* This works because speeds numbering starts at 0 and is contiguous. */ msg_pdbg("SPI speed is %sHz\n", spispeeds[spispeed].name); ret = buspirate_serialport_setup(dev); if (ret) return ret; + free(dev); /* This is the brute force version, but it should work. */ for (i = 0; i < 19; i++) { Modified: trunk/chipset_enable.c ============================================================================== --- trunk/chipset_enable.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/chipset_enable.c Tue Jul 6 11:55:48 2010 (r1070) @@ -300,11 +300,8 @@ int max_decode_fwh_decode = 0; int contiguous = 1; - if (programmer_param) - idsel = strstr(programmer_param, "fwh_idsel="); - - if (idsel) { - idsel += strlen("fwh_idsel="); + idsel = extract_param(&programmer_param, "fwh_idsel", ",:"); + if (idsel && strlen(idsel)) { fwh_conf = (uint32_t)strtoul(idsel, NULL, 0); /* FIXME: Need to undo this on shutdown. */ @@ -312,7 +309,15 @@ pci_write_long(dev, 0xd0, fwh_conf); pci_write_word(dev, 0xd4, fwh_conf); /* FIXME: Decode settings are not changed. */ + } else if (idsel) { + msg_perr("Error: idsel= specified, but no number given.\n"); + free(idsel); + /* FIXME: Return failure here once internal_init() starts + * to care about the return value of the chipset enable. + */ + exit(1); } + free(idsel); /* Ignore all legacy ranges below 1 MB. * We currently only support flashing the chip which responds to Modified: trunk/drkaiser.c ============================================================================== --- trunk/drkaiser.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/drkaiser.c Tue Jul 6 11:55:48 2010 (r1070) @@ -38,8 +38,9 @@ uint32_t addr; get_io_perms(); + pcidev_init(PCI_VENDOR_ID_DRKAISER, PCI_BASE_ADDRESS_2, - drkaiser_pcidev, programmer_param); + drkaiser_pcidev); /* Write magic register to enable flash write. */ pci_write_word(pcidev_dev, PCI_MAGIC_DRKAISER_ADDR, @@ -53,6 +54,7 @@ addr, 128 * 1024); buses_supported = CHIP_BUSTYPE_PARALLEL; + return 0; } @@ -60,7 +62,6 @@ { /* Write protect the flash again. */ pci_write_word(pcidev_dev, PCI_MAGIC_DRKAISER_ADDR, 0); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/dummyflasher.c ============================================================================== --- trunk/dummyflasher.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/dummyflasher.c Tue Jul 6 11:55:48 2010 (r1070) @@ -25,44 +25,46 @@ #include "flash.h" #include "chipdrivers.h" +static void tolower_string(char *str) +{ + for (; *str != '\0'; str++) + *str = (char)tolower((unsigned char)*str); +} + int dummy_init(void) { - int i; + char *bustext = NULL; + msg_pspew("%s\n", __func__); - /* "all" is equivalent to specifying no type. */ - if (programmer_param && (!strcmp(programmer_param, "all"))) { - free(programmer_param); - programmer_param = NULL; - } - if (!programmer_param) - programmer_param = strdup("parallel,lpc,fwh,spi"); + bustext = extract_param(&programmer_param, "bus", ",:"); + msg_pdbg("Requested buses are: %s\n", bustext ? bustext : "default"); + if (!bustext) + bustext = strdup("parallel+lpc+fwh+spi"); /* Convert the parameters to lowercase. */ - for (i = 0; programmer_param[i] != '\0'; i++) - programmer_param[i] = - (char)tolower((unsigned char)programmer_param[i]); + tolower_string(bustext); buses_supported = CHIP_BUSTYPE_NONE; - if (strstr(programmer_param, "parallel")) { + if (strstr(bustext, "parallel")) { buses_supported |= CHIP_BUSTYPE_PARALLEL; msg_pdbg("Enabling support for %s flash.\n", "parallel"); } - if (strstr(programmer_param, "lpc")) { + if (strstr(bustext, "lpc")) { buses_supported |= CHIP_BUSTYPE_LPC; msg_pdbg("Enabling support for %s flash.\n", "LPC"); } - if (strstr(programmer_param, "fwh")) { + if (strstr(bustext, "fwh")) { buses_supported |= CHIP_BUSTYPE_FWH; msg_pdbg("Enabling support for %s flash.\n", "FWH"); } - if (strstr(programmer_param, "spi")) { + if (strstr(bustext, "spi")) { buses_supported |= CHIP_BUSTYPE_SPI; spi_controller = SPI_CONTROLLER_DUMMY; msg_pdbg("Enabling support for %s flash.\n", "SPI"); } if (buses_supported == CHIP_BUSTYPE_NONE) msg_pdbg("Support for all flash bus types disabled.\n"); - free(programmer_param); + free(bustext); return 0; } Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/flash.h Tue Jul 6 11:55:48 2010 (r1070) @@ -336,7 +336,7 @@ const char *device_name; }; uint32_t pcidev_validate(struct pci_dev *dev, uint32_t bar, const struct pcidev_status *devs); -uint32_t pcidev_init(uint16_t vendor_id, uint32_t bar, const struct pcidev_status *devs, char *pcidev_bdf); +uint32_t pcidev_init(uint16_t vendor_id, uint32_t bar, const struct pcidev_status *devs); #endif /* print.c */ Modified: trunk/flashrom.8 ============================================================================== --- trunk/flashrom.8 Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/flashrom.8 Tue Jul 6 11:55:48 2010 (r1070) @@ -183,7 +183,7 @@ .BR "* ft2232_spi" " (for SPI flash ROMs attached to a FT2232H/FT4232H based\ USB SPI programmer)" .sp -.BR "* serprog" " (for flash ROMs attached to Urja's AVR programmer)" +.BR "* serprog" " (for flash ROMs attached to a programmer speaking serprog)" .sp .BR "* buspirate_spi" " (for SPI flash ROMs attached to a Bus Pirate)" .sp @@ -296,22 +296,23 @@ .BR "dummy " programmer An optional parameter specifies the bus types it should support. For that you have to use the -.B "flashrom \-p dummy:type" +.B "flashrom \-p dummy:bus=[type[+type[+type]]]" syntax where .B type -can be any comma-separated combination of -.BR parallel ", " lpc ", " fwh ", " spi ", or " all -(in any order). +can be any of +.BR parallel ", " lpc ", " fwh ", " spi +in any order. If you specify bus without type, all buses will be disabled. +If you do not specify bus, all buses will be enabled. .sp Example: -.B "flashrom \-p dummy:lpc,fwh" +.B "flashrom \-p dummy:bus=lpc+fwh" .TP .BR "nic3com" , " nicrealtek" , " nicsmc1211" , " gfxnvidia" , " satasii \ " and " atahpt " programmers These programmers have an option to specify the PCI address of the card your want to use, which must be specified if more than one card supported by the selected programmer is installed in your system. The syntax is -.BR "flashrom \-p xxxx:bb:dd.f" , +.BR "flashrom \-p xxxx:pci=bb:dd.f" , where .B xxxx is the name of the programmer @@ -323,7 +324,7 @@ is the PCI function number of the desired device. .sp Example: -.B "flashrom \-p nic3com:05:04.0" +.B "flashrom \-p nic3com:pci=05:04.0" .TP .BR "it87spi " programmer An optional @@ -342,7 +343,7 @@ An optional parameter specifies the controller type and interface/port it should support. For that you have to use the .sp -.B " flashrom \-p ft2232_spi:model,port=interface" +.B " flashrom \-p ft2232_spi:type=model,port=interface" .sp syntax where .B model @@ -363,11 +364,11 @@ programmer. In the device/baud combination, the device has to start with a slash. For serial, you have to use the .sp -.B " flashrom \-p serprog:/dev/device:baud" +.B " flashrom \-p serprog:dev=/dev/device:baud" .sp syntax and for IP, you have to use .sp -.B " flashrom \-p serprog:ip:port" +.B " flashrom \-p serprog:ip=ipaddr:port" .sp instead. More information about serprog is available in .B serprog-protocol.txt Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/flashrom.c Tue Jul 6 11:55:48 2010 (r1070) @@ -434,6 +434,7 @@ int programmer_init(char *param) { + int ret; /* Initialize all programmer specific data. */ /* Default to unlimited decode sizes. */ max_rom_decode = (const struct decode_sizes) { @@ -456,7 +457,13 @@ programmer_param = param; msg_pdbg("Initializing %s programmer\n", programmer_table[programmer].name); - return programmer_table[programmer].init(); + ret = programmer_table[programmer].init(); + if (programmer_param && strlen(programmer_param)) { + msg_perr("Unhandled programmer parameters: %s\n", + programmer_param); + /* Do not error out here, the init itself was successful. */ + } + return ret; } int programmer_shutdown(void) @@ -564,22 +571,24 @@ char *strcat_realloc(char *dest, const char *src) { dest = realloc(dest, strlen(dest) + strlen(src) + 1); - if (!dest) + if (!dest) { + msg_gerr("Out of memory!\n"); return NULL; + } strcat(dest, src); return dest; } /* This is a somewhat hacked function similar in some ways to strtok(). - * It will look for needle in haystack, return a copy of needle and remove - * everything from the first occurrence of needle to the next delimiter - * from haystack. + * It will look for needle with a subsequent '=' in haystack, return a copy of + * needle and remove everything from the first occurrence of needle to the next + * delimiter from haystack. */ char *extract_param(char **haystack, char *needle, char *delim) { - char *param_pos, *rest, *tmp; - char *dev = NULL; - int devlen; + char *param_pos, *opt_pos, *rest; + char *opt = NULL; + int optlen; int needlelen; needlelen = strlen(needle); @@ -595,43 +604,41 @@ do { if (!param_pos) return NULL; - /* Beginning of the string? */ - if (param_pos == *haystack) - break; - /* After a delimiter? */ - if (strchr(delim, *(param_pos - 1))) - break; + /* Needle followed by '='? */ + if (param_pos[needlelen] == '=') { + + /* Beginning of the string? */ + if (param_pos == *haystack) + break; + /* After a delimiter? */ + if (strchr(delim, *(param_pos - 1))) + break; + } /* Continue searching. */ param_pos++; param_pos = strstr(param_pos, needle); } while (1); - + if (param_pos) { - param_pos += strlen(needle); - devlen = strcspn(param_pos, delim); - if (devlen) { - dev = malloc(devlen + 1); - if (!dev) { - msg_gerr("Out of memory!\n"); - exit(1); - } - strncpy(dev, param_pos, devlen); - dev[devlen] = '\0'; - } - rest = param_pos + devlen; - rest += strspn(rest, delim); - param_pos -= strlen(needle); - memmove(param_pos, rest, strlen(rest) + 1); - tmp = realloc(*haystack, strlen(*haystack) + 1); - if (!tmp) { + /* Get the string after needle and '='. */ + opt_pos = param_pos + needlelen + 1; + optlen = strcspn(opt_pos, delim); + /* Return an empty string if the parameter was empty. */ + opt = malloc(optlen + 1); + if (!opt) { msg_gerr("Out of memory!\n"); exit(1); } - *haystack = tmp; + strncpy(opt, opt_pos, optlen); + opt[optlen] = '\0'; + rest = opt_pos + optlen; + /* Skip all delimiters after the current parameter. */ + rest += strspn(rest, delim); + memmove(param_pos, rest, strlen(rest) + 1); + /* We could shrink haystack, but the effort is not worth it. */ } - - return dev; + return opt; } /* start is an offset to the base address of the flash chip */ Modified: trunk/ft2232_spi.c ============================================================================== --- trunk/ft2232_spi.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/ft2232_spi.c Tue Jul 6 11:55:48 2010 (r1070) @@ -76,46 +76,49 @@ int f; struct ftdi_context *ftdic = &ftdic_context; unsigned char buf[512]; - char *portpos = NULL; int ft2232_type = FTDI_FT4232H; enum ftdi_interface ft2232_interface = INTERFACE_B; + char *arg; - if (ftdi_init(ftdic) < 0) { - msg_perr("ftdi_init failed\n"); - return EXIT_FAILURE; // TODO - } - - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; - } - if (programmer_param) { - if (strstr(programmer_param, "2232")) + arg = extract_param(&programmer_param, "type", ",:"); + if (arg) { + if (!strcasecmp(arg, "2232H")) ft2232_type = FTDI_FT2232H; - if (strstr(programmer_param, "4232")) + else if (!strcasecmp(arg, "4232H")) ft2232_type = FTDI_FT4232H; - portpos = strstr(programmer_param, "port="); - if (portpos) { - portpos += 5; - switch (toupper(*portpos)) { - case 'A': - ft2232_interface = INTERFACE_A; - break; - case 'B': - ft2232_interface = INTERFACE_B; - break; - default: - msg_perr("Invalid interface specified, " - "using default.\n"); - } + else { + msg_perr("Error: Invalid device type specified.\n"); + free(arg); + return 1; } - free(programmer_param); } + free(arg); + arg = extract_param(&programmer_param, "port", ",:"); + if (arg) { + switch (toupper(*arg)) { + case 'A': + ft2232_interface = INTERFACE_A; + break; + case 'B': + ft2232_interface = INTERFACE_B; + break; + default: + msg_perr("Error: Invalid port/interface specified.\n"); + free(arg); + return 1; + } + } + free(arg); msg_pdbg("Using device type %s ", (ft2232_type == FTDI_FT2232H) ? "2232H" : "4232H"); msg_pdbg("interface %s\n", (ft2232_interface == INTERFACE_A) ? "A" : "B"); + if (ftdi_init(ftdic) < 0) { + msg_perr("ftdi_init failed\n"); + return EXIT_FAILURE; // TODO + } + f = ftdi_usb_open(ftdic, 0x0403, ft2232_type); if (f < 0 && f != -5) { Modified: trunk/gfxnvidia.c ============================================================================== --- trunk/gfxnvidia.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/gfxnvidia.c Tue Jul 6 11:55:48 2010 (r1070) @@ -62,7 +62,8 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_NVIDIA, PCI_BASE_ADDRESS_0, - gfx_nvidia, programmer_param); + gfx_nvidia); + io_base_addr += 0x300000; msg_pinfo("Detected NVIDIA I/O base address: 0x%x.\n", io_base_addr); @@ -87,7 +88,6 @@ reg32 |= (1 << 0); pci_write_long(pcidev_dev, 0x50, reg32); - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/internal.c ============================================================================== --- trunk/internal.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/internal.c Tue Jul 6 11:55:48 2010 (r1070) @@ -121,36 +121,45 @@ int force_laptop = 0; char *arg; - arg = extract_param(&programmer_param, "boardenable=", ",:"); + arg = extract_param(&programmer_param, "boardenable", ",:"); if (arg && !strcmp(arg,"force")) { force_boardenable = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for boardenable.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for boardenable: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); - arg = extract_param(&programmer_param, "boardmismatch=", ",:"); + arg = extract_param(&programmer_param, "boardmismatch", ",:"); if (arg && !strcmp(arg,"force")) { force_boardmismatch = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for boardmismatch.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for boardmismatch: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); - arg = extract_param(&programmer_param, "laptop=", ",:"); + arg = extract_param(&programmer_param, "laptop", ",:"); if (arg && !strcmp(arg,"force_I_want_a_brick")) { force_laptop = 1; } else if (arg && !strlen(arg)) { msg_perr("Missing argument for laptop.\n"); + free(arg); + return 1; } else if (arg) { msg_perr("Unknown argument for laptop: %s\n", arg); - exit(1); + free(arg); + return 1; } free(arg); Modified: trunk/it87spi.c ============================================================================== --- trunk/it87spi.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/it87spi.c Tue Jul 6 11:55:48 2010 (r1070) @@ -142,22 +142,24 @@ flashport = sio_read(port, 0x64) << 8; flashport |= sio_read(port, 0x65); msg_pdbg("Serial flash port 0x%04x\n", flashport); - if (programmer_param && !strlen(programmer_param)) { - free(programmer_param); - programmer_param = NULL; - } - if (programmer_param) { - portpos = extract_param(&programmer_param, - "it87spiport=", ",:"); - if (portpos) { - flashport = strtol(portpos, (char **)NULL, 0); - msg_pinfo("Forcing serial flash port 0x%04x\n", - flashport); - sio_write(port, 0x64, (flashport >> 8)); - sio_write(port, 0x65, (flashport & 0xff)); - free(portpos); - } + /* Non-default port requested? */ + portpos = extract_param(&programmer_param, "it87spiport", ",:"); + if (portpos && strlen(portpos)) { + flashport = strtol(portpos, (char **)NULL, 0); + msg_pinfo("Forcing serial flash port 0x%04x\n", + flashport); + sio_write(port, 0x64, (flashport >> 8)); + sio_write(port, 0x65, (flashport & 0xff)); + } else if (portpos) { + msg_perr("Error: it87spiport specified, but no port " + "given.\n"); + free(portpos); + /* FIXME: Return failure here once it87spi_common_init() + * can handle the return value sanely. + */ + exit(1); } + free(portpos); exit_conf_mode_ite(port); break; /* TODO: Handle more IT87xx if they support flash translation */ Modified: trunk/nic3com.c ============================================================================== --- trunk/nic3com.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/nic3com.c Tue Jul 6 11:55:48 2010 (r1070) @@ -59,7 +59,8 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_3COM, PCI_BASE_ADDRESS_0, - nics_3com, programmer_param); + nics_3com); + id = pcidev_dev->device_id; /* 3COM 3C90xB cards need a special fixup. */ @@ -96,7 +97,6 @@ OUTL(internal_conf, io_base_addr + INTERNAL_CONFIG); } - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/nicnatsemi.c ============================================================================== --- trunk/nicnatsemi.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/nicnatsemi.c Tue Jul 6 11:55:48 2010 (r1070) @@ -39,7 +39,7 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_NATSEMI, PCI_BASE_ADDRESS_0, - nics_natsemi, programmer_param); + nics_natsemi); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -48,7 +48,6 @@ int nicnatsemi_shutdown(void) { - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/nicrealtek.c ============================================================================== --- trunk/nicrealtek.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/nicrealtek.c Tue Jul 6 11:55:48 2010 (r1070) @@ -44,7 +44,7 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_REALTEK, PCI_BASE_ADDRESS_0, - nics_realtek, programmer_param); + nics_realtek); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -56,7 +56,7 @@ get_io_perms(); io_base_addr = pcidev_init(PCI_VENDOR_ID_SMC1211, PCI_BASE_ADDRESS_0, - nics_realteksmc1211, programmer_param); + nics_realteksmc1211); buses_supported = CHIP_BUSTYPE_PARALLEL; @@ -66,7 +66,6 @@ int nicrealtek_shutdown(void) { /* FIXME: We forgot to disable software access again. */ - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/pcidev.c ============================================================================== --- trunk/pcidev.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/pcidev.c Tue Jul 6 11:55:48 2010 (r1070) @@ -78,10 +78,11 @@ } uint32_t pcidev_init(uint16_t vendor_id, uint32_t bar, - const struct pcidev_status *devs, char *pcidev_bdf) + const struct pcidev_status *devs) { struct pci_dev *dev; struct pci_filter filter; + char *pcidev_bdf; char *msg = NULL; int found = 0; uint32_t addr = 0, curaddr = 0; @@ -93,12 +94,14 @@ /* Filter by vendor and also bb:dd.f (if supplied by the user). */ filter.vendor = vendor_id; + pcidev_bdf = extract_param(&programmer_param, "pci", ","); if (pcidev_bdf != NULL) { if ((msg = pci_filter_parse_slot(&filter, pcidev_bdf))) { msg_perr("Error: %s\n", msg); exit(1); } } + free(pcidev_bdf); for (dev = pacc->devices; dev; dev = dev->next) { if (pci_filter_match(&filter, dev)) { Modified: trunk/satasii.c ============================================================================== --- trunk/satasii.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/satasii.c Tue Jul 6 11:55:48 2010 (r1070) @@ -46,8 +46,8 @@ get_io_perms(); - pcidev_init(PCI_VENDOR_ID_SII, PCI_BASE_ADDRESS_0, satas_sii, - programmer_param); + pcidev_init(PCI_VENDOR_ID_SII, PCI_BASE_ADDRESS_0, satas_sii); + id = pcidev_dev->device_id; if ((id == 0x3132) || (id == 0x3124)) { @@ -71,7 +71,6 @@ int satasii_shutdown(void) { - free(programmer_param); pci_cleanup(pacc); release_io_perms(); return 0; Modified: trunk/serprog.c ============================================================================== --- trunk/serprog.c Sat Jul 3 14:14:25 2010 (r1069) +++ trunk/serprog.c Tue Jul 6 11:55:48 2010 (r1070) @@ -298,43 +298,80 @@ int serprog_init(void) { uint16_t iface; - int len; unsigned char pgmname[17]; unsigned char rbuf[3]; unsigned char c; - char *num; - char *dev; - msg_pspew("%s\n", __func__); - /* the parameter is either of format "/dev/device:baud" or "ip:port" */ - if ((!programmer_param) || (!strlen(programmer_param))) { - nodevice: - msg_perr("Error: No device/host given for the serial programmer driver.\n" - "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n"); - exit(1); - } - num = strstr(programmer_param, ":"); - len = num - programmer_param; - if (!len) goto nodevice; - if (!num) { - msg_perr("Error: No port or baudrate specified to serial programmer driver.\n" - "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n"); - exit(1); - } - len = num - programmer_param; - dev = malloc(len + 1); - if (!dev) sp_die("Error: memory allocation failure"); - memcpy(dev, programmer_param, len); - dev[len] = 0; - num = strdup(num + 1); - if (!num) sp_die("Error: memory allocation failure"); - free(programmer_param); - programmer_param = NULL; + char *device; + char *baudport; + int have_device = 0; - if (dev[0] == '/') sp_fd = sp_openserport(dev, atoi(num)); - else sp_fd = sp_opensocket(dev, atoi(num)); + /* the parameter is either of format "dev=/dev/device:baud" or "ip=ip:port" */ + device = extract_param(&programmer_param, "dev", ","); + if (device && strlen(device)) { + baudport = strstr(device, ":"); + if (baudport) { + /* Split device from baudrate. */ + *baudport = '\0'; + baudport++; + } + if (!baudport || !strlen(baudport)) { + msg_perr("Error: No baudrate specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud\n"); + free(device); + return 1; + } + if (strlen(device)) { + sp_fd = sp_openserport(device, atoi(baudport)); + have_device++; + } + } + if (device && !strlen(device)) { + msg_perr("Error: No device specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud\n"); + free(device); + return 1; + } + free(device); - free(dev); dev = NULL; - free(num); num = NULL; + device = extract_param(&programmer_param, "ip", ","); + if (have_device && device) { + msg_perr("Error: Both host and device specified.\n" + "Please use either dev= or ip= but not both.\n"); + free(device); + return 1; + } + if (device && strlen(device)) { + baudport = strstr(device, ":"); + if (baudport) { + /* Split host from port. */ + *baudport = '\0'; + baudport++; + } + if (!baudport || !strlen(baudport)) { + msg_perr("Error: No port specified.\n" + "Use flashrom -p serprog:ip=ipaddr:port\n"); + free(device); + return 1; + } + if (strlen(device)) { + sp_fd = sp_opensocket(device, atoi(baudport)); + have_device++; + } + } + if (device && !strlen(device)) { + msg_perr("Error: No host specified.\n" + "Use flashrom -p serprog:ip=ipaddr:port\n"); + free(device); + return 1; + } + free(device); + + if (!have_device) { + msg_perr("Error: Neither host nor device specified.\n" + "Use flashrom -p serprog:dev=/dev/device:baud or " + "flashrom -p serprog:ip=ipaddr:port\n"); + return 1; + } msg_pdbg(MSGHEADER "connected - attempting to synchronize\n");

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom