flashrom

flashrom@flashrom.org

6 participants
9186 discussions

Mark MCP51 variant 0x0261 as tested.
by Marcin Kościelnicki 27 Apr '14

27 Apr '14

Signed-off-by: Marcin Kościelnicki <koriakin(a)0x04.net> Index: chipset_enable.c =================================================================== --- chipset_enable.c (revision 1764) +++ chipset_enable.c (working copy) @@ -1418,7 +1418,7 @@ /* Slave, should not be here, to fix known bug for A01. */ {0x10de, 0x00d3, OK, "NVIDIA", "CK804", enable_flash_ck804}, {0x10de, 0x0260, OK, "NVIDIA", "MCP51", enable_flash_ck804}, - {0x10de, 0x0261, NT, "NVIDIA", "MCP51", enable_flash_ck804}, + {0x10de, 0x0261, OK, "NVIDIA", "MCP51", enable_flash_ck804}, {0x10de, 0x0262, NT, "NVIDIA", "MCP51", enable_flash_ck804}, {0x10de, 0x0263, NT, "NVIDIA", "MCP51", enable_flash_ck804}, {0x10de, 0x0360, OK, "NVIDIA", "MCP55", enable_flash_mcp55}, /* M57SLI*/

2 1

[commit] r1781 - in trunk: . util
by repository service 27 Apr '14

27 Apr '14

Author: stefanct Date: Sun Apr 27 07:07:35 2014 New Revision: 1781 URL: http://flashrom.org/trac/flashrom/changeset/1781 Log: ft2232_spi: Add support for TUMPA Lite. http://www.tiaowiki.com/w/TIAO_USB_Multi_Protocol_Adapter_Lite_User's_Manual Initial patch from Jadran Puharic <jpuharic(a)gmail.com>. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> Modified: trunk/flashrom.8.tmpl trunk/ft2232_spi.c trunk/util/z60_flashrom.rules Modified: trunk/flashrom.8.tmpl ============================================================================== --- trunk/flashrom.8.tmpl Sat Apr 26 18:13:09 2014 (r1780) +++ trunk/flashrom.8.tmpl Sun Apr 27 07:07:35 2014 (r1781) @@ -197,7 +197,7 @@ FTDI FT2232H Mini-Module, FTDI FT4232H Mini-Module, openbiosprog-spi, Amontec \ JTAGkey/JTAGkey-tiny/JTAGkey-2, Dangerous Prototypes Bus Blaster, \ Olimex ARM-USB-TINY/-H, Olimex ARM-USB-OCD/-H, TIAO/DIYGADGET USB -Multi-Protocol Adapter (TUMPA), and GOEPEL PicoTAP. +Multi-Protocol Adapter (TUMPA), TUMPA Lite, and GOEPEL PicoTAP. .sp .BR "* serprog" " (for flash ROMs attached to a programmer speaking serprog), \ including AVR flasher by Urja Rannikko, AVR flasher by eightdot, \ @@ -599,7 +599,7 @@ can be .BR 2232H ", " 4232H ", " 232H ", " jtagkey ", " busblaster ", " openmoko ", " \ arm-usb-tiny ", " arm-usb-tiny-h ", " arm-usb-ocd ", " arm-usb-ocd-h \ -", " tumpa ", or " picotap +", " tumpa ", " tumpalite ", or " picotap and .B interface can be Modified: trunk/ft2232_spi.c ============================================================================== --- trunk/ft2232_spi.c Sat Apr 26 18:13:09 2014 (r1780) +++ trunk/ft2232_spi.c Sun Apr 27 07:07:35 2014 (r1781) @@ -43,6 +43,7 @@ #define FTDI_FT4232H_PID 0x6011 #define FTDI_FT232H_PID 0x6014 #define TIAO_TUMPA_PID 0x8a98 +#define TIAO_TUMPA_LITE_PID 0x8a99 #define AMONTEC_JTAGKEY_PID 0xCFF8 #define GOEPEL_VID 0x096C @@ -62,6 +63,7 @@ {FTDI_VID, FTDI_FT4232H_PID, OK, "FTDI", "FT4232H"}, {FTDI_VID, FTDI_FT232H_PID, OK, "FTDI", "FT232H"}, {FTDI_VID, TIAO_TUMPA_PID, OK, "TIAO", "USB Multi-Protocol Adapter"}, + {FTDI_VID, TIAO_TUMPA_LITE_PID, OK, "TIAO", "USB Multi-Protocol Adapter Lite"}, {FTDI_VID, AMONTEC_JTAGKEY_PID, OK, "Amontec", "JTAGkey"}, {GOEPEL_VID, GOEPEL_PICOTAP_PID, OK, "GOEPEL", "PicoTAP"}, {FIC_VID, OPENMOKO_DBGBOARD_PID, OK, "FIC", "OpenMoko Neo1973 Debug board (V2+)"}, @@ -208,6 +210,10 @@ /* Interface A is SPI1, B is SPI2. */ ft2232_type = TIAO_TUMPA_PID; channel_count = 2; + } else if (!strcasecmp(arg, "tumpalite")) { + /* Only one channel is used on lite edition */ + ft2232_type = TIAO_TUMPA_LITE_PID; + channel_count = 1; } else if (!strcasecmp(arg, "busblaster")) { /* In its default configuration it is a jtagkey clone */ ft2232_type = FTDI_FT2232H_PID; Modified: trunk/util/z60_flashrom.rules ============================================================================== --- trunk/util/z60_flashrom.rules Sat Apr 26 18:13:09 2014 (r1780) +++ trunk/util/z60_flashrom.rules Sun Apr 27 07:07:35 2014 (r1781) @@ -80,4 +80,8 @@ # http://www.diygadget.com/tiao-usb-multi-protocol-adapter-jtag-spi-i2c-seria… ATTRS{idVendor}=="0403", ATTRS{idProduct}=="8a98", MODE="664", GROUP="plugdev" +# TIAO/DIYGADGET USB Multi-Protocol Adapter (TUMPA) Lite +# http://www.tiaowiki.com/w/TIAO_USB_Multi_Protocol_Adapter_Lite_User's_Manual +ATTRS{idVendor}=="0403", ATTRS{idProduct}=="8a99", MODE="664", GROUP="plugdev" + LABEL="flashrom_rules_end"

1 0

Re: [flashrom] HM77 probe, read, write OK
by Stefan Tauner 27 Apr '14

27 Apr '14

On Thu, 27 Mar 2014 16:55:35 +0400 Vasiliy Vylegzhanin <6vasia(a)gmail.com> wrote: > flashrom code is slightly modified for region write and predefined ICH > descriptor regions, but chipset-related code is vanilla. In fact, only > flashrom.c and layout.c is patched. Hello Vasiliy, thanks for your report! I have added your chipset to our list of supported hardware and will commit that later together with other small changes. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

Re: [flashrom] C204 write log
by Stefan Tauner 27 Apr '14

27 Apr '14

On Wed, 16 Apr 2014 14:19:21 +0400 Vasiliy Vylegzhanin <6vasia(a)gmail.com> wrote: > Works fine. flashrom r1711 + layout patches. Hello Vasiliy, thanks for your report! I have added the chipset to our list of supported hardware and will commit that later together with other small changes. -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

1 0

[PATCH] CID1130009: Resource leak
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130009: Resource leak The system resource will not be reclaimed and reused, reducing the future availability of the resource. In sp_opensocket: Leak of memory or pointers to system resources Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: serprog.c =================================================================== --- serprog.c (revision 1763) +++ serprog.c (working copy) @@ -100,6 +100,7 @@ if (NULL == hostPtr) { hostPtr = gethostbyaddr(ip, strlen(ip), AF_INET); if (NULL == hostPtr) { + close(sock); msg_perr("Error: cannot resolve %s\n", ip); return -1; }

2 1

[PATCH] CID1130008: Resource leak
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130008: Resource leak The system resource will not be reclaimed and reused, reducing the future availability of the resource. In read_romlayout: Leak of memory or pointers to system resources Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: layout.c =================================================================== --- layout.c (revision 1763) +++ layout.c (working copy) @@ -65,6 +65,7 @@ if (num_rom_entries >= MAX_ROMLAYOUT) { msg_gerr("Maximum number of ROM images (%i) in layout " "file reached.\n", MAX_ROMLAYOUT); + fclose(romlayout); return 1; } if (2 != fscanf(romlayout, "%s %s\n", tempstr, rom_entries[num_rom_entries].name))

2 1

[PATCH] CID1130007: Resource leak
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130007: Resource leak The system resource will not be reclaimed and reused, reducing the future availability of the resource. In ogp_spi_init: Leak of memory or pointers to system resources Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: ogp_spi.c =================================================================== --- ogp_spi.c (revision 1763) +++ ogp_spi.c (working copy) @@ -120,8 +120,10 @@ ogp_reg_sck = OGA1_XP10_CPROM_SCK; } else { msg_perr("Invalid or missing rom= parameter.\n"); + free(type); return 1; } + free(type); if (rget_io_perms()) return 1;

2 1

[PATCH] CID1130011: Use after free
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130011: Use after free This could cause an immediate crash or incorrect values might be read subsequently resulting in incorrect computations. In dump_file: A pointer to freed memory is dereferenced, used as a function argument, or otherwise used Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: util/ich_descriptors_tool/ich_descriptors_tool.c =================================================================== --- util/ich_descriptors_tool/ich_descriptors_tool.c (revision 1763) +++ util/ich_descriptors_tool/ich_descriptors_tool.c (working copy) @@ -77,12 +77,13 @@ printf("Dumping %u bytes of the %s region from 0x%08x-0x%08x to %s... ", file_len, region_names[i], base, limit, fn); int fh = open(fn, O_WRONLY | O_CREAT, S_IRUSR | S_IWUSR); - free(fn); if (fh < 0) { fprintf(stderr, "ERROR: couldn't open(%s): %s\n", fn, strerror(errno)); + free(fn); exit(1); } + free(fn); ret = write(fh, &dump[base >> 2], file_len); if (ret != file_len) {

2 1

[PATCH] CID1130012: Double free
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130012: Double free Depending on the implementation of the deallocator function, the memory may by placed on the free list more than once. In pony_spi_init: Memory is deallocated more than once Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: pony_spi.c =================================================================== --- pony_spi.c (revision 1763) +++ pony_spi.c (working copy) @@ -140,6 +140,7 @@ } else if (arg && !strlen(arg)) { msg_perr("Error: Missing argument for programmer type.\n"); free(arg); + return 1; } else if (arg){ msg_perr("Error: Invalid programmer type specified.\n"); free(arg);

2 1

[PATCH] CID1130004: Nesting level does not match indentation
by Stefan Reinauer 26 Apr '14

26 Apr '14

CID1130004: Nesting level does not match indentation Code that is meant to be executed conditionally may be executed unconditionally In print_supported_boards_helper: The indentation of this code suggests it is nested when it is not. Signed-off-by: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Index: print.c =================================================================== --- print.c (revision 1763) +++ print.c (working copy) @@ -436,13 +436,14 @@ msg_ginfo("%s", b->name); for (i = 0; i < maxboardlen - strlen(b->name); i++) msg_ginfo(" "); - if (b->working == OK) - msg_ginfo("OK "); - else if (b->working == NT) - msg_ginfo("NT "); - else - msg_ginfo("BAD "); + if (b->working == OK) + msg_ginfo("OK "); + else if (b->working == NT) + msg_ginfo("NT "); + else + msg_ginfo("BAD "); + for (e = board_matches; e->vendor_name != NULL; e++) { if (strcmp(e->vendor_name, b->vendor) || strcmp(e->board_name, b->name))

2 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom