flashrom June 2015

flashrom@flashrom.org

30 participants
88 discussions

[PATCH 2/3] serprog: ignore failures when setting serial port options.
by Stefan Tauner 28 Jun '15

28 Jun '15

There is no good reason to abort earlier just because some options did not stick. This should improve compatibility without negative effects. If communication is affected by the missing flag(s) then we abort later anyway. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> --- serial.c | 23 ++++++++++++++++++----- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/serial.c b/serial.c index 63774a5..b216143 100644 --- a/serial.c +++ b/serial.c @@ -209,12 +209,25 @@ int serialport_config(fdtype fd, unsigned int baud) if (observed.c_cflag != wanted.c_cflag || observed.c_lflag != wanted.c_lflag || observed.c_iflag != wanted.c_iflag || - observed.c_oflag != wanted.c_oflag || - cfgetispeed(&observed) != cfgetispeed(&wanted)) { - msg_perr("%s: Some requested options did not stick.\n", __func__); - return 1; + observed.c_oflag != wanted.c_oflag) { + msg_pwarn("Some requested serial options did not stick, continuing anyway.\n"); + msg_pdbg(" observed wanted\n" + "c_cflag: 0x%08X 0x%08X\n" + "c_lflag: 0x%08X 0x%08X\n" + "c_iflag: 0x%08X 0x%08X\n" + "c_oflag: 0x%08X 0x%08X\n", + observed.c_cflag, wanted.c_cflag, + observed.c_lflag, wanted.c_lflag, + observed.c_iflag, wanted.c_iflag, + observed.c_oflag, wanted.c_oflag + ); + } + if (cfgetispeed(&observed) != cfgetispeed(&wanted) || + cfgetospeed(&observed) != cfgetospeed(&wanted)) { + msg_pwarn("Could not set baud rates exactly\n"); + msg_pdbg("Actual baud flags are: ispeed: 0x%08lX, ospeed: 0x%08lX\n", + (long)cfgetispeed(&observed), (long)cfgetispeed(&wanted)); } - msg_pdbg("Baud rate is %d now.\n", entry->baud); #endif return 0; } -- Kind regards, Stefan Tauner

1 0

[PATCH 1/3] serprog: clear NDELAY flag only once after opening the port.
by Stefan Tauner 28 Jun '15

28 Jun '15

Change sp_openserport() to directly clear the O_NONBLOCK flag that is potentially set by opening the device with O_NDELAY instead of clearing ALL flags in serialport_config() unconditionally. Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> --- serial.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/serial.c b/serial.c index 94f9b7d..63774a5 100644 --- a/serial.c +++ b/serial.c @@ -183,10 +183,6 @@ int serialport_config(fdtype fd, unsigned int baud) msg_pdbg("Baud rate is %ld.\n", dcb.BaudRate); #else struct termios wanted, observed; - if (fcntl(fd, F_SETFL, 0) != 0) { - msg_perr_strerror("Could not clear serial port mode: "); - return 1; - } if (tcgetattr(fd, &observed) != 0) { msg_perr_strerror("Could not fetch original serial port configuration: "); return 1; @@ -254,11 +250,23 @@ fdtype sp_openserport(char *dev, unsigned int baud) } return fd; #else - fd = open(dev, O_RDWR | O_NOCTTY | O_NDELAY); + fd = open(dev, O_RDWR | O_NOCTTY | O_NDELAY); // Use O_NDELAY to ignore DCD state if (fd < 0) { msg_perr_strerror("Cannot open serial port: "); return SER_INV_FD; } + + /* Ensure that we use blocking I/O */ + const int flags = fcntl(fd, F_GETFL); + if (flags == -1) { + msg_perr_strerror("Could not get serial port mode: "); + return SER_INV_FD; + } + if (fcntl(fd, F_SETFL, flags & ~O_NONBLOCK) != 0) { + msg_perr_strerror("Could not set serial port mode to blocking: "); + return SER_INV_FD; + } + if (serialport_config(fd, baud) != 0) { close(fd); return SER_INV_FD; -- Kind regards, Stefan Tauner

1 0

[PATCH 0/3] Some serprog improvements
by Stefan Tauner 28 Jun '15

28 Jun '15

I plan to include them (and probably some of Urja's patches too) in flashrom 0.9.8.1. Stefan Tauner (3): serprog: clear NDELAY flag only once after opening the port. serprog: ignore failures when setting serial port options. serprog: allow to omit specifying a baud rate. flashrom.8.tmpl | 26 ++++++++++++----------- programmer.h | 4 +--- serial.c | 66 ++++++++++++++++++++++++++++++++++++++++----------------- serprog.c | 44 ++++++++++++++++++++------------------ 4 files changed, 84 insertions(+), 56 deletions(-) -- Kind regards, Stefan Tauner

1 0

Re: [flashrom] [PATCH 5/5] dediprog: Enable by default.
by David Hendricks 28 Jun '15

28 Jun '15

Acked-by: David Hendricks <dhendrix(a)chromium.org> On Sun, Jun 21, 2015 at 6:37 PM, Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> wrote: > Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > --- > Makefile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Makefile b/Makefile > index c439d8d..e7c8fde 100644 > --- a/Makefile > +++ b/Makefile > @@ -469,8 +469,8 @@ CONFIG_OGP_SPI ?= yes > # Always enable Bus Pirate SPI for now. > CONFIG_BUSPIRATE_SPI ?= yes > > -# Disable Dediprog SF100 until support is complete and tested. > -CONFIG_DEDIPROG ?= no > +# Always enable Dediprog SF100 for now. > +CONFIG_DEDIPROG ?= yes > > # Always enable Marvell SATA controllers for now. > CONFIG_SATAMV ?= yes > -- > Kind regards, Stefan Tauner > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom

1 0

Re: [flashrom] [PATCH 3/5] dediprog: Show UID.
by David Hendricks 28 Jun '15

28 Jun '15

Apply this patch after patch 4 so that dediprog_read() is defined. Aside from that: Acked-by: David Hendricks <dhendrix(a)chromium.org> On Sun, Jun 21, 2015 at 6:37 PM, Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> wrote: > Whatever that is... > To that end, send the respective command and print result with a "new" > helpers.c function. > > Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > --- > dediprog.c | 19 +++++++++++++++++++ > flash.h | 1 + > helpers.c | 7 +++++++ > 3 files changed, 27 insertions(+) > > diff --git a/dediprog.c b/dediprog.c > index ce7ff7a..a6801be 100644 > --- a/dediprog.c > +++ b/dediprog.c > @@ -730,6 +730,18 @@ static int dediprog_shutdown(void *data) > return 0; > } > > +/* Return the 8-byte UID for the flash */ > +static int get_uid(uint8_t buf[8]) > +{ > + int ret = dediprog_read(CMD_GET_UID, 0, 0, buf, 8); > + if (ret != 8) { > + msg_perr("get_uid failed (%s)!\n", usb_strerror()); > + return 1; > + } > + > + return 0; > +} > + > /* URB numbers refer to the first log ever captured. */ > int dediprog_init(void) > { > @@ -872,6 +884,13 @@ int dediprog_init(void) > return 1; > } > > + uint8_t uid[8]; > + if (get_uid(uid) != 0) > + return 1; > + msg_pdbg("UID:"); > + print_hex_buf(uid, sizeof(uid)); > + msg_pdbg("\n"); > + > register_spi_master(&spi_master_dediprog); > > dediprog_set_leds(LED_NONE); > diff --git a/flash.h b/flash.h > index 2c2839f..f7bd94b 100644 > --- a/flash.h > +++ b/flash.h > @@ -251,6 +251,7 @@ void print_supported_wiki(void); > > /* helpers.c */ > uint32_t address_to_bits(uint32_t addr); > +void print_hex_buf(uint8_t *buf, size_t len); > int bitcount(unsigned long a); > int max(int a, int b); > int min(int a, int b); > diff --git a/helpers.c b/helpers.c > index 7a146c3..a82ba55 100644 > --- a/helpers.c > +++ b/helpers.c > @@ -34,6 +34,13 @@ uint32_t address_to_bits(uint32_t addr) > return 32 - lzb; > } > > +void print_hex_buf(uint8_t *buf, size_t len) > +{ > + size_t i; > + for (i = 0; i < len; i++) > + msg_pdbg(" %02x", buf[i]); > +} > + > int bitcount(unsigned long a) > { > int i = 0; > -- > Kind regards, Stefan Tauner > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom

1 0

Re: [flashrom] [PATCH 1/5] dediprog: Invert the LED polarity in the code.
by David Hendricks 28 Jun '15

28 Jun '15

Acked-by: David Hendricks <dhendrix(a)chromium.org> On Sun, Jun 21, 2015 at 6:37 PM, Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> wrote: > > Previously we have used low-active macros (because the hardware and > old protocol were so too) and set every single LED explicitly although we > only used a limited number of combinations. Using an enumeration for > commonly used values instead makes things easier. > > Based on the following chromiumos change: > Change-Id: Ie481a583e623cdc45e3649a4db69b15570f65a7b > > Signed-off-by: Simon Glass <sjg(a)chromium.org> > Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > --- > dediprog.c | 101 ++++++++++++++++++++++++++++++------------------------------- > 1 file changed, 49 insertions(+), 52 deletions(-) > > diff --git a/dediprog.c b/dediprog.c > index 851dbae..b45e061 100644 > --- a/dediprog.c > +++ b/dediprog.c > @@ -45,6 +45,17 @@ static int dediprog_endpoint; > #define DEDI_SPI_CMD_PAGEWRITE 0x1 > #define DEDI_SPI_CMD_AAIWRITE 0x4 > > +enum dediprog_leds { > + LED_INVALID = -1, > + LED_NONE = 0, > + LED_PASS = 1 << 0, > + LED_BUSY = 1 << 1, > + LED_ERROR = 1 << 2, > + LED_ALL = 7, > +}; > + > +static int dediprog_firmwareversion = FIRMWARE_VERSION(0, 0, 0); > + > #if 0 > /* Might be useful for other pieces of code as well. */ > static void print_hex(void *buf, size_t len) > @@ -77,24 +88,11 @@ static struct usb_device *get_device_by_vid_pid(uint16_t vid, uint16_t pid, unsi > > //int usb_control_msg(usb_dev_handle *dev, int requesttype, int request, int value, int index, char *bytes, int size, int timeout); > > -/* Set/clear LEDs on dediprog */ > -#define PASS_ON (0 << 0) > -#define PASS_OFF (1 << 0) > -#define BUSY_ON (0 << 1) > -#define BUSY_OFF (1 << 1) > -#define ERROR_ON (0 << 2) > -#define ERROR_OFF (1 << 2) > -static int current_led_status = -1; > - > +/* This function sets the GPIOs connected to the LEDs as well as IO1-IO4. */ > static int dediprog_set_leds(int leds) > { > - int ret, target_leds; > - > - if (leds < 0 || leds > 7) > - leds = 0; // Bogus value, enable all LEDs > - > - if (leds == current_led_status) > - return 0; > + if (leds < LED_NONE || leds > LED_ALL) > + leds = LED_ALL; > > /* Older Dediprogs with 2.x.x and 3.x.x firmware only had > * two LEDs, and they were reversed. So map them around if > @@ -103,23 +101,21 @@ static int dediprog_set_leds(int leds) > * bit 2 == 0: green light is on. > * bit 0 == 0: red light is on. > */ > + int target_leds; > if (dediprog_firmwareversion < FIRMWARE_VERSION(5,0,0)) { > - target_leds = ((leds & ERROR_OFF) >> 2) | > - ((leds & PASS_OFF) << 2); > + target_leds = ((leds & LED_ERROR) >> 2) | > + ((leds & LED_PASS) << 2); > } else { > target_leds = leds; > } > > - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, target_leds, > - NULL, 0x0, DEFAULT_TIMEOUT); > + target_leds ^= 7; > + int ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, target_leds, NULL, 0x0, DEFAULT_TIMEOUT); > if (ret != 0x0) { > - msg_perr("Command Set LED 0x%x failed (%s)!\n", > - leds, usb_strerror()); > + msg_perr("Command Set LED 0x%x failed (%s)!\n", leds, usb_strerror()); > return 1; > } > > - current_led_status = leds; > - > return 0; > } > > @@ -271,26 +267,22 @@ static int dediprog_spi_read(struct flashctx *flash, uint8_t *buf, > unsigned int residue = start % chunksize ? chunksize - start % chunksize : 0; > unsigned int bulklen; > > - dediprog_set_leds(PASS_OFF|BUSY_ON|ERROR_OFF); > + dediprog_set_leds(LED_BUSY); > > if (residue) { > msg_pdbg("Slow read for partial block from 0x%x, length 0x%x\n", > start, residue); > ret = spi_read_chunked(flash, buf, start, residue, 16); > - if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > - return ret; > - } > + if (ret) > + goto err; > } > > /* Round down. */ > bulklen = (len - residue) / chunksize * chunksize; > ret = dediprog_spi_bulk_read(flash, buf + residue, start + residue, > bulklen); > - if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > - return ret; > - } > + if (ret) > + goto err; > > len -= residue + bulklen; > if (len) { > @@ -298,14 +290,15 @@ static int dediprog_spi_read(struct flashctx *flash, uint8_t *buf, > start, len); > ret = spi_read_chunked(flash, buf + residue + bulklen, > start + residue + bulklen, len, 16); > - if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > - return ret; > - } > + if (ret) > + goto err; > } > > - dediprog_set_leds(PASS_ON|BUSY_OFF|ERROR_OFF); > + dediprog_set_leds(LED_PASS); > return 0; > +err: > + dediprog_set_leds(LED_ERROR); > + return ret; > } > > /* Bulk write interface, will write multiple chunksize byte chunks aligned to chunksize bytes. > @@ -383,7 +376,7 @@ static int dediprog_spi_write(struct flashctx *flash, const uint8_t *buf, > unsigned int residue = start % chunksize ? chunksize - start % chunksize : 0; > unsigned int bulklen; > > - dediprog_set_leds(PASS_OFF|BUSY_ON|ERROR_OFF); > + dediprog_set_leds(LED_BUSY); > > if (chunksize != 256) { > msg_pdbg("Page sizes other than 256 bytes are unsupported as " > @@ -398,7 +391,7 @@ static int dediprog_spi_write(struct flashctx *flash, const uint8_t *buf, > /* No idea about the real limit. Maybe 12, maybe more. */ > ret = spi_write_chunked(flash, buf, start, residue, 12); > if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > + dediprog_set_leds(LED_ERROR); > return ret; > } > } > @@ -406,10 +399,10 @@ static int dediprog_spi_write(struct flashctx *flash, const uint8_t *buf, > /* Round down. */ > bulklen = (len - residue) / chunksize * chunksize; > ret = dediprog_spi_bulk_write(flash, buf + residue, chunksize, start + residue, bulklen, dedi_spi_cmd); > - if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > - return ret; > - } > + if (ret) > + dediprog_set_leds(LED_ERROR); > + else > + dediprog_set_leds(LED_PASS); > > len -= residue + bulklen; > if (len) { > @@ -418,12 +411,12 @@ static int dediprog_spi_write(struct flashctx *flash, const uint8_t *buf, > ret = spi_write_chunked(flash, buf + residue + bulklen, > start + residue + bulklen, len, 12); > if (ret) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > + dediprog_set_leds(LED_ERROR); > return ret; > } > } > > - dediprog_set_leds(PASS_ON|BUSY_OFF|ERROR_OFF); > + dediprog_set_leds(LED_PASS); > return 0; > } > > @@ -781,6 +774,8 @@ static int dediprog_shutdown(void *data) > return 1; > #endif > > + dediprog_firmwareversion = FIRMWARE_VERSION(0, 0, 0); > + > /* URB 28. Command Set SPI Voltage to 0. */ > if (dediprog_set_spi_voltage(0x0)) > return 1; > @@ -918,23 +913,25 @@ int dediprog_init(void) > if (register_shutdown(dediprog_shutdown, NULL)) > return 1; > > - dediprog_set_leds(PASS_ON|BUSY_ON|ERROR_ON); > - > /* Perform basic setup. */ > if (dediprog_setup(target)) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > return 1; > } > > + /* Set all possible LEDs as soon as possible to indicate activity. > + * Because knowing the firmware version is required to set the LEDs correctly we need to this after > + * dediprog_setup() has queried the device and set dediprog_firmwareversion. */ > + dediprog_set_leds(LED_ALL); > + > /* After setting voltage and speed, perform setup again. */ > if (dediprog_set_spi_voltage(0) || dediprog_set_spi_speed(spispeed_idx) || dediprog_setup(target)) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > + dediprog_set_leds(LED_ERROR); > return 1; > } > > /* URB 11. Command Set SPI Voltage. */ > if (dediprog_set_spi_voltage(millivolt)) { > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON); > + dediprog_set_leds(LED_ERROR); > return 1; > } > > @@ -946,7 +943,7 @@ int dediprog_init(void) > dediprog_do_stuff(); > #endif > > - dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_OFF); > + dediprog_set_leds(LED_NONE); > > return 0; > } > -- > Kind regards, Stefan Tauner > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom

1 0

Re: [flashrom] [PATCH 2/5] dediprog: Tidy up commands and remove dead nonsense code.
by David Hendricks 28 Jun '15

28 Jun '15

Mostly looks good to me, just a couple minor comments. On Sun, Jun 21, 2015 at 6:37 PM, Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> wrote: > Use names for the commands and request types instead of magic > numbers and remove some of the unnecessary unexplained arguments. > Also, cleanup the nonsense code left over from RE. Most of it can > not explained by official documentation and was recorded with > ancient firmware/software. > > Based on the following chromiumos changes: > Change-Id: I80a0dcdf40eedc89da48fb2c54cd9d9fd13e6fa1 > Change-Id: If61bac2c8194b3ec30a80422d871842c66f0cd74 > > Signed-off-by: Simon Glass <sjg(a)chromium.org> > Signed-off-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > Acked-by: Stefan Tauner <stefan.tauner(a)alumni.tuwien.ac.at> > --- > dediprog.c | 386 +++++++++++++++++++++---------------------------------------- > 1 file changed, 132 insertions(+), 254 deletions(-) > > diff --git a/dediprog.c b/dediprog.c > index b45e061..ce7ff7a 100644 > --- a/dediprog.c > +++ b/dediprog.c > @@ -2,6 +2,8 @@ > * This file is part of the flashrom project. > * > * Copyright (C) 2010 Carl-Daniel Hailfinger > + * Copyright (C) 2015 Simon Glass > + * Copyright (C) 2015 Stefan Tauner > * > * 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 > @@ -38,13 +40,14 @@ > > #define FIRMWARE_VERSION(x,y,z) ((x << 16) | (y << 8) | z) > #define DEFAULT_TIMEOUT 3000 > +#define REQTYPE_OTHER_OUT (USB_ENDPOINT_IN | USB_TYPE_VENDOR | USB_RECIP_OTHER) /* 0x43 */ > +#define REQTYPE_OTHER_IN (USB_ENDPOINT_IN | USB_TYPE_VENDOR | USB_RECIP_OTHER) /* 0xC3 */ > +#define REQTYPE_EP_OUT (USB_ENDPOINT_OUT | USB_TYPE_VENDOR | USB_RECIP_ENDPOINT) /* 0x42 */ > +#define REQTYPE_EP_IN (USB_ENDPOINT_IN | USB_TYPE_VENDOR | USB_RECIP_ENDPOINT) /* 0xC2 */ > static usb_dev_handle *dediprog_handle; > static int dediprog_firmwareversion; > static int dediprog_endpoint; > > -#define DEDI_SPI_CMD_PAGEWRITE 0x1 > -#define DEDI_SPI_CMD_AAIWRITE 0x4 > - > enum dediprog_leds { > LED_INVALID = -1, > LED_NONE = 0, > @@ -54,6 +57,77 @@ enum dediprog_leds { > LED_ALL = 7, > }; > > +/* IO bits for CMD_SET_IO_LED message */ > +enum dediprog_ios { > + IO1 = 1 << 0, > + IO2 = 1 << 1, > + IO3 = 1 << 2, > + IO4 = 1 << 3, > +}; > + > +enum dediprog_cmds { > + CMD_TRANSCEIVE = 0x01, > + CMD_POLL_STATUS_REG = 0x02, > + CMD_SET_VPP = 0x03, > + CMD_SET_TARGET = 0x04, > + CMD_READ_EEPROM = 0x05, > + CMD_WRITE_EEPROM = 0x06, > + CMD_SET_IO_LED = 0x07, > + CMD_READ_PROG_INFO = 0x08, > + CMD_SET_VCC = 0x09, > + CMD_SET_STANDALONE = 0x0A, > + CMD_GET_BUTTON = 0x11, > + CMD_GET_UID = 0x12, > + CMD_SET_CS = 0x14, > + CMD_IO_MODE = 0x15, > + CMD_FW_UPDATE = 0x1A, > + CMD_FPGA_UPDATE = 0x1B, > + CMD_READ_FPGA_VERSION = 0x1C, > + CMD_SET_HOLD = 0x1D, > + CMD_READ = 0x20, > + CMD_WRITE = 0x30, > + CMD_WRITE_AT45DB = 0x31, > + CMD_NAND_WRITE = 0x32, > + CMD_NAND_READ = 0x33, > + CMD_SET_SPI_CLK = 0x61, > + CMD_CHECK_SOCKET = 0x62, > + CMD_DOWNLOAD_PRJ = 0x63, > + CMD_READ_PRJ_NAME = 0x64, > + // New protocol/firmware only > + CMD_CHECK_SDCARD = 0x65, > + CMD_READ_PRJ = 0x66, > +}; > + > +enum dediprog_target { > + FLASH_TYPE_APPLICATION_FLASH_1 = 0, > + FLASH_TYPE_FLASH_CARD, > + FLASH_TYPE_APPLICATION_FLASH_2, FLASH_TYPE_SOCKET, /* SF600 */ > +}; > + > +enum dediprog_readmode { > + READ_MODE_STD = 1, > + READ_MODE_FAST = 2, > + READ_MODE_ATMEL45 = 3, > + READ_MODE_4B_ADDR_FAST = 4, > + READ_MODE_4B_ADDR_FAST_0x0C = 5, /* New protocol only */ > +}; > + > +enum dediprog_writemode { > + WRITE_MODE_PAGE_PGM = 1, > + WRITE_MODE_PAGE_WRITE = 2, > + WRITE_MODE_1B_AAI = 3, > + WRITE_MODE_2B_AAI = 4, > + WRITE_MODE_128B_PAGE = 5, > + WRITE_MODE_PAGE_AT26DF041 = 6, > + WRITE_MODE_SILICON_BLUE_FPGA = 7, > + WRITE_MODE_64B_PAGE_NUMONYX_PCM = 8, /* unit of length 512 bytes */ > + WRITE_MODE_4B_ADDR_256B_PAGE_PGM = 9, > + WRITE_MODE_32B_PAGE_PGM_MXIC_512K = 10, /* unit of length 512 bytes */ > + WRITE_MODE_4B_ADDR_256B_PAGE_PGM_0x12 = 11, 11 seems to only apply for the new protocol? The old protocol spec just says "For SF100" but I have no idea what that actually means. > + WRITE_MODE_4B_ADDR_256B_PAGE_PGM_FLAGS = 12, > +}; > + > + > static int dediprog_firmwareversion = FIRMWARE_VERSION(0, 0, 0); > > #if 0 > @@ -86,8 +160,6 @@ static struct usb_device *get_device_by_vid_pid(uint16_t vid, uint16_t pid, unsi > return NULL; > } > > -//int usb_control_msg(usb_dev_handle *dev, int requesttype, int request, int value, int index, char *bytes, int size, int timeout); > - > /* This function sets the GPIOs connected to the LEDs as well as IO1-IO4. */ > static int dediprog_set_leds(int leds) > { > @@ -110,7 +182,8 @@ static int dediprog_set_leds(int leds) > } > > target_leds ^= 7; > - int ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, target_leds, NULL, 0x0, DEFAULT_TIMEOUT); > + int ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_SET_IO_LED, 0x09, target_leds, > + NULL, 0x0, DEFAULT_TIMEOUT); > if (ret != 0x0) { > msg_perr("Command Set LED 0x%x failed (%s)!\n", leds, usb_strerror()); > return 1; > @@ -149,8 +222,8 @@ static int dediprog_set_spi_voltage(int millivolt) > /* Wait some time as the original driver does. */ > programmer_delay(200 * 1000); > } > - ret = usb_control_msg(dediprog_handle, 0x42, 0x9, voltage_selector, > - 0xff, NULL, 0x0, DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_SET_VCC, voltage_selector, 0, > + NULL, 0x0, DEFAULT_TIMEOUT); > if (ret != 0x0) { > msg_perr("Command Set SPI Voltage 0x%x failed!\n", > voltage_selector); > @@ -180,13 +253,6 @@ static const struct dediprog_spispeeds spispeeds[] = { > { NULL, 0x0 }, > }; > > -/* After dediprog_set_spi_speed, the original app always calls > - * dediprog_set_spi_voltage(0) and then > - * dediprog_check_devicestring() four times in a row. > - * After that, dediprog_command_a() is called. > - * This looks suspiciously like the microprocessor in the SF100 has to be > - * restarted/reinitialized in case the speed changes. > - */ > static int dediprog_set_spi_speed(unsigned int spispeed_idx) > { > if (dediprog_firmwareversion < FIRMWARE_VERSION(5, 0, 0)) { > @@ -194,12 +260,13 @@ static int dediprog_set_spi_speed(unsigned int spispeed_idx) > return 0; > } > > - msg_pdbg("SPI speed is %s Hz\n", spispeeds[spispeed_idx].name); > + const struct dediprog_spispeeds *spispeed = &spispeeds[spispeed_idx]; > + msg_pdbg("SPI speed is %sHz\n", spispeed->name); > > - int ret = usb_control_msg(dediprog_handle, 0x42, 0x61, spispeeds[spispeed_idx].speed, 0xff, > + int ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_SET_SPI_CLK, spispeed->speed, 0xff, > NULL, 0x0, DEFAULT_TIMEOUT); > if (ret != 0x0) { > - msg_perr("Command Set SPI Speed 0x%x failed!\n", spispeeds[spispeed_idx].speed); > + msg_perr("Command Set SPI Speed 0x%x failed!\n", spispeed->speed); > return 1; > } > return 0; > @@ -235,7 +302,7 @@ static int dediprog_spi_bulk_read(struct flashctx *flash, uint8_t *buf, > /* Command Read SPI Bulk. No idea which read command is used on the > * SPI side. > */ > - ret = usb_control_msg(dediprog_handle, 0x42, 0x20, start % 0x10000, > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_READ, start % 0x10000, > start / 0x10000, (char *)count_and_chunk, > sizeof(count_and_chunk), DEFAULT_TIMEOUT); > if (ret != sizeof(count_and_chunk)) { > @@ -344,7 +411,7 @@ static int dediprog_spi_bulk_write(struct flashctx *flash, const uint8_t *buf, u > /* Command Write SPI Bulk. No idea which write command is used on the > * SPI side. > */ > - ret = usb_control_msg(dediprog_handle, 0x42, 0x30, start % 0x10000, start / 0x10000, > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_WRITE, start % 0x10000, start / 0x10000, > (char *)count_and_cmd, sizeof(count_and_cmd), DEFAULT_TIMEOUT); > if (ret != sizeof(count_and_cmd)) { > msg_perr("Command Write SPI Bulk failed, %i %s!\n", ret, > @@ -422,12 +489,12 @@ static int dediprog_spi_write(struct flashctx *flash, const uint8_t *buf, > > static int dediprog_spi_write_256(struct flashctx *flash, const uint8_t *buf, unsigned int start, unsigned int len) > { > - return dediprog_spi_write(flash, buf, start, len, DEDI_SPI_CMD_PAGEWRITE); > + return dediprog_spi_write(flash, buf, start, len, WRITE_MODE_PAGE_PGM); > } > > static int dediprog_spi_write_aai(struct flashctx *flash, const uint8_t *buf, unsigned int start, unsigned int len) > { > - return dediprog_spi_write(flash, buf, start, len, DEDI_SPI_CMD_AAIWRITE); > + return dediprog_spi_write(flash, buf, start, len, WRITE_MODE_2B_AAI); > } > > static int dediprog_spi_send_command(struct flashctx *flash, > @@ -439,29 +506,26 @@ static int dediprog_spi_send_command(struct flashctx *flash, > int ret; > > msg_pspew("%s, writecnt=%i, readcnt=%i\n", __func__, writecnt, readcnt); > - /* Paranoid, but I don't want to be blamed if anything explodes. */ > - if (writecnt > 16) { > - msg_perr("Untested writecnt=%i, aborting.\n", writecnt); > + if (writecnt > UINT16_MAX) { > + msg_perr("Invalid writecnt=%i, aborting.\n", writecnt); > return 1; > } > - /* 16 byte reads should work. */ > - if (readcnt > 16) { > - msg_perr("Untested readcnt=%i, aborting.\n", readcnt); > + if (readcnt > UINT16_MAX) { > + msg_perr("Invalid readcnt=%i, aborting.\n", readcnt); > return 1; > } > > - ret = usb_control_msg(dediprog_handle, 0x42, 0x1, 0xff, > - readcnt ? 0x1 : 0x0, (char *)writearr, writecnt, > - DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_TRANSCEIVE, 0, readcnt ? 0x1 : 0x0, > + (char *)writearr, writecnt, DEFAULT_TIMEOUT); > if (ret != writecnt) { > msg_perr("Send SPI failed, expected %i, got %i %s!\n", > writecnt, ret, usb_strerror()); > return 1; > } > - if (!readcnt) > + if (readcnt == 0) > return 0; > - memset(readarr, 0, readcnt); > - ret = usb_control_msg(dediprog_handle, 0xc2, 0x01, 0xbb8, 0x0000, > + > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_IN, CMD_TRANSCEIVE, 0, 0, > (char *)readarr, readcnt, DEFAULT_TIMEOUT); > if (ret != readcnt) { > msg_perr("Receive SPI failed, expected %i, got %i %s!\n", > @@ -477,27 +541,27 @@ static int dediprog_check_devicestring(void) > int fw[3]; > char buf[0x11]; > > +#if 0 > /* Command Prepare Receive Device String. */ > - memset(buf, 0, sizeof(buf)); > - ret = usb_control_msg(dediprog_handle, 0xc3, 0x7, 0x0, 0xef03, buf, > - 0x1, DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_OTHER_IN, 0x7, 0x0, 0xef03, > + buf, 0x1, DEFAULT_TIMEOUT); > /* The char casting is needed to stop gcc complaining about an always true comparison. */ > if ((ret != 0x1) || (buf[0] != (char)0xff)) { > msg_perr("Unexpected response to Command Prepare Receive Device" > " String!\n"); > return 1; > } > +#endif > /* Command Receive Device String. */ > - memset(buf, 0, sizeof(buf)); > - ret = usb_control_msg(dediprog_handle, 0xc2, 0x8, 0xff, 0xff, buf, > - 0x10, DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_EP_IN, CMD_READ_PROG_INFO, 0, 0, > + buf, 0x10, DEFAULT_TIMEOUT); > if (ret != 0x10) { > msg_perr("Incomplete/failed Command Receive Device String!\n"); > return 1; > } > buf[0x10] = '\0'; > msg_pdbg("Found a %s\n", buf); > - if (memcmp(buf, "SF100", 0x5)) { > + if (memcmp(buf, "SF100", 0x5) != 0) { > msg_perr("Device not a SF100!\n"); > return 1; > } > @@ -515,24 +579,20 @@ static int dediprog_check_devicestring(void) > return 0; > } > > -/* Command A seems to be some sort of device init. It is either followed by > - * dediprog_check_devicestring (often) or Command A (often) or > - * Command F (once). > - */ > -static int dediprog_command_a(void) > +static int dediprog_device_init(void) > { > int ret; > char buf[0x1]; > > memset(buf, 0, sizeof(buf)); > - ret = usb_control_msg(dediprog_handle, 0xc3, 0xb, 0x0, 0x0, buf, > - 0x1, DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_OTHER_IN, 0x0B, 0x0, 0x0, > + buf, 0x1, DEFAULT_TIMEOUT); > if (ret < 0) { > msg_perr("Command A failed (%s)!\n", usb_strerror()); > return 1; > } > if ((ret != 0x1) || (buf[0] != 0x6f)) { > - msg_perr("Unexpected response to Command A!\n"); > + msg_perr("Unexpected response to init!\n"); > return 1; > } > return 0; > @@ -548,9 +608,8 @@ static int dediprog_command_b(void) > int ret; > char buf[0x3]; > > - memset(buf, 0, sizeof(buf)); > - ret = usb_control_msg(dediprog_handle, 0xc3, 0x7, 0x0, 0xef00, buf, > - 0x3, DEFAULT_TIMEOUT); > + ret = usb_control_msg(dediprog_handle, REQTYPE_OTHER_IN, 0x7, 0x0, 0xef00, > + buf, 0x3, DEFAULT_TIMEOUT); > if (ret < 0) { > msg_perr("Command B failed (%s)!\n", usb_strerror()); > return 1; > @@ -565,120 +624,29 @@ static int dediprog_command_b(void) > } > #endif > > -/* Command Chip Select is only sent after dediprog_check_devicestring, but not after every > - * invocation of dediprog_check_devicestring. It is only sent after the first > - * dediprog_command_a(); dediprog_check_devicestring() sequence in each session. > - * Bit #1 of the value changes the chip select: 0 is target 1, 1 is target 2 and parameter target can be 1 or 2 > - * respectively. We don't know how to encode "3, Socket" and "0, reference card" yet. On SF100 the vendor > - * software "DpCmd 6.0.4.06" selects target 2 when requesting 3 (which is unavailable on that hardware). > - */ > -static int dediprog_chip_select(int target) > +static int set_target_flash(enum dediprog_target target) > { > - int ret; > - uint16_t value = ((target - 1) & 1) << 1; > - msg_pdbg("Selecting target chip %i\n", target); > - ret = usb_control_msg(dediprog_handle, 0x42, 0x4, value, 0x0, NULL, > - 0x0, DEFAULT_TIMEOUT); > - if (ret != 0x0) { > - msg_perr("Command Chip Select failed (%s)!\n", usb_strerror()); > + int ret = usb_control_msg(dediprog_handle, REQTYPE_EP_OUT, CMD_SET_TARGET, target, 0, > + NULL, 0, DEFAULT_TIMEOUT); > + if (ret != 0) { > + msg_perr("set_target_flash failed (%s)!\n", usb_strerror()); > return 1; > } > return 0; > } > > #if 0 > -/* Very strange. Seems to be a programmer keepalive or somesuch. > - * Wait unsuccessfully for timeout ms to read one byte. > - * Is usually called after setting voltage to 0. > - * Present in all logs with Firmware 2.1.1 and 3.1.8 > - */ > -static int dediprog_command_f(int timeout) > +/* Returns true if the button is currently pressed. */ > +static bool dediprog_get_button(void) > { > - int ret; > - char buf[0x1]; > - > - memset(buf, 0, sizeof(buf)); > - ret = usb_control_msg(dediprog_handle, 0xc2, 0x11, 0xff, 0xff, buf, > - 0x1, timeout); > - /* This check is most probably wrong. Command F always causes a timeout > - * in the logs, so we should check for timeout instead of checking for > - * success. > - */ > - if (ret != 0x1) { > - msg_perr("Command F failed (%s)!\n", usb_strerror()); > - return 1; > - } > - return 0; > -} > - > -/* Start/stop blinking? > - * Present in eng_detect_blink.log with firmware 3.1.8 > - * Preceded by Command J > - */ > -static int dediprog_command_g(void) > -{ > - int ret; > - > - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, 0x03, NULL, 0x0, DEFAULT_TIMEOUT); > - if (ret != 0x0) { > - msg_perr("Command G failed (%s)!\n", usb_strerror()); > - return 1; > - } > - return 0; > -} > - > -/* Something. > - * Present in all logs with firmware 5.1.5 > - * Always preceded by Command Receive Device String > - * Always followed by Command Set SPI Voltage nonzero > - */ > -static int dediprog_command_h(void) > -{ > - int ret; > - > - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, 0x05, NULL, 0x0, DEFAULT_TIMEOUT); > - if (ret != 0x0) { > - msg_perr("Command H failed (%s)!\n", usb_strerror()); > + char buf[1]; > + int ret = usb_control_msg(dediprog_handle, REQTYPE_EP_IN, CMD_GET_BUTTON, 0, 0, > + buf, 0x1, DEFAULT_TIMEOUT); > + if (ret != 0) { > + msg_perr("Could not get button state (%s)!\n", usb_strerror()); > return 1; > } > - return 0; > -} > - > -/* Shutdown for firmware 5.x? > - * Present in all logs with firmware 5.1.5 > - * Often preceded by a SPI operation (Command Read SPI Bulk or Receive SPI) > - * Always followed by Command Set SPI Voltage 0x0000 > - */ > -static int dediprog_command_i(void) > -{ > - int ret; > - > - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, 0x06, NULL, 0x0, DEFAULT_TIMEOUT); > - if (ret != 0x0) { > - msg_perr("Command I failed (%s)!\n", usb_strerror()); > - return 1; > - } > - return 0; > -} > - > -/* Start/stop blinking? > - * Present in all logs with firmware 5.1.5 > - * Always preceded by Command Receive Device String on 5.1.5 > - * Always followed by Command Set SPI Voltage nonzero on 5.1.5 > - * Present in eng_detect_blink.log with firmware 3.1.8 > - * Preceded by Command B in eng_detect_blink.log > - * Followed by Command G in eng_detect_blink.log > - */ > -static int dediprog_command_j(void) > -{ > - int ret; > - > - ret = usb_control_msg(dediprog_handle, 0x42, 0x07, 0x09, 0x07, NULL, 0x0, DEFAULT_TIMEOUT); > - if (ret != 0x0) { > - msg_perr("Command J failed (%s)!\n", usb_strerror()); > - return 1; > - } > - return 0; > + return buf[0] != 1; > } > #endif > > @@ -730,28 +698,6 @@ static int parse_voltage(char *voltage) > return millivolt; > } > > -static int dediprog_setup(long target) > -{ > - /* URB 6. Command A. */ > - if (dediprog_command_a()) { > - return 1; > - } > - /* URB 7. Command A. */ > - if (dediprog_command_a()) { > - return 1; > - } > - /* URB 8. Command Prepare Receive Device String. */ > - /* URB 9. Command Receive Device String. */ > - if (dediprog_check_devicestring()) { > - return 1; > - } > - /* URB 10. Command Chip Select */ > - if (dediprog_chip_select(target)) { > - return 1; > - } > - return 0; > -} > - > static const struct spi_master spi_master_dediprog = { > .type = SPI_CONTROLLER_DEDIPROG, > .max_data_read = MAX_DATA_UNSPECIFIED, > @@ -767,13 +713,6 @@ static int dediprog_shutdown(void *data) > { > msg_pspew("%s\n", __func__); > > -#if 0 > - /* Shutdown on firmware 5.x */ > - if (dediprog_firmwareversion == 5) > - if (dediprog_command_i()) > - return 1; > -#endif > - > dediprog_firmwareversion = FIRMWARE_VERSION(0, 0, 0); > > /* URB 28. Command Set SPI Voltage to 0. */ > @@ -819,6 +758,7 @@ int dediprog_init(void) > } > free(spispeed); > } > + > voltage = extract_programmer_param("voltage"); > if (voltage) { > millivolt = parse_voltage(voltage); > @@ -914,90 +854,28 @@ int dediprog_init(void) > return 1; > > /* Perform basic setup. */ > - if (dediprog_setup(target)) { > + if (dediprog_device_init()) > + return 1; > + if (dediprog_check_devicestring()) > return 1; > - } > > /* Set all possible LEDs as soon as possible to indicate activity. > * Because knowing the firmware version is required to set the LEDs correctly we need to this after > * dediprog_setup() has queried the device and set dediprog_firmwareversion. */ > dediprog_set_leds(LED_ALL); > > - /* After setting voltage and speed, perform setup again. */ > - if (dediprog_set_spi_voltage(0) || dediprog_set_spi_speed(spispeed_idx) || dediprog_setup(target)) { > - dediprog_set_leds(LED_ERROR); > - return 1; > - } > - > - /* URB 11. Command Set SPI Voltage. */ > - if (dediprog_set_spi_voltage(millivolt)) { > + /* Select target/socket, frequency and VCC. */ > + if (set_target_flash(FLASH_TYPE_APPLICATION_FLASH_1) || > + dediprog_set_spi_speed(spispeed_idx) || > + dediprog_set_spi_voltage(millivolt)) { > dediprog_set_leds(LED_ERROR); > return 1; > } > > register_spi_master(&spi_master_dediprog); > > - /* RE leftover, leave in until the driver is complete. */ > -#if 0 > - /* Execute RDID by hand if you want to test it. */ > - dediprog_do_stuff(); > -#endif > - > dediprog_set_leds(LED_NONE); > > return 0; > } > > -#if 0 > -/* Leftovers from reverse engineering. Keep for documentation purposes until > - * completely understood. > - */ > -static int dediprog_do_stuff(void) > -{ > - char buf[0x4]; > - /* SPI command processing starts here. */ > - > - /* URB 12. Command Send SPI. */ > - /* URB 13. Command Receive SPI. */ > - memset(buf, 0, sizeof(buf)); > - /* JEDEC RDID */ > - msg_pdbg("Sending RDID\n"); > - buf[0] = JEDEC_RDID; > - if (dediprog_spi_send_command(JEDEC_RDID_OUTSIZE, JEDEC_RDID_INSIZE, > - (unsigned char *)buf, (unsigned char *)buf)) > - return 1; > - msg_pdbg("Receiving response: "); > - print_hex(buf, JEDEC_RDID_INSIZE); > - /* URB 14-27 are more SPI commands. */ > - /* URB 28. Command Set SPI Voltage. */ > - if (dediprog_set_spi_voltage(0x0)) > - return 1; > - /* URB 29-38. Command F, unsuccessful wait. */ > - if (dediprog_command_f(544)) > - return 1; > - /* URB 39. Command Set SPI Voltage. */ > - if (dediprog_set_spi_voltage(0x10)) > - return 1; > - /* URB 40. Command Set SPI Speed. */ > - if (dediprog_set_spi_speed(0x2)) > - return 1; > - /* URB 41 is just URB 28. */ > - /* URB 42,44,46,48,51,53 is just URB 8. */ > - /* URB 43,45,47,49,52,54 is just URB 9. */ > - /* URB 50 is just URB 6/7. */ > - /* URB 55-131 is just URB 29-38. (wait unsuccessfully for 4695 (maybe 4751) ms)*/ > - /* URB 132,134 is just URB 6/7. */ > - /* URB 133 is just URB 29-38. */ > - /* URB 135 is just URB 8. */ > - /* URB 136 is just URB 9. */ > - /* URB 137 is just URB 11. */ > - > - /* Command Start Bulk Read. Data is u16 blockcount, u16 blocksize. */ > - /* Command Start Bulk Write. Data is u16 blockcount, u16 blocksize. */ > - /* Bulk transfer sizes for Command Start Bulk Read/Write are always > - * 512 bytes, rest is filled with 0xff. > - */ > - > - return 0; > -} > -#endif > -- > Kind regards, Stefan Tauner > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom

1 0

Re: [flashrom] 404 error on link to precompiled windows exe
by Idwer Vollering 27 Jun '15

27 Jun '15

Hi, 2015-06-25 12:52 GMT+02:00 Michael Ayles <michael.ayles(a)ttelectronics-ims.com>: > Hi, > > I’m a test development engineer and one of our clients has developed a board > with an EON EN25F20 SPI Flash. They provided a Bus Pirate and the firmware > file to be programmed and told us they use linux with flashrom to program > the parts. > > Unfortunately the test solution we have developed runs windows for our test > solution. I’ve been unable to find a valid link to a precompiled executable > of flashrom for windows. The link on the (flashrom.org/windows) page > (http://ra.openbios.org/~idwer/flashrom/mingw/) returns a 404 error. Indeed. Server configuration (which is outside my control) has rendered this URL unreahable. However, you can find win32 builds over here: http://buildbot.flashrom.org/buildresults/ > > Could you please help me locate a .exe, I’m more of a hardware guy than > software and am struggling with the MINGW stuff. You can find an (alpha quality) prebuilt .exe here: http://buildbot.flashrom.org/buildresults/flashrom-000174-CcG/win32/ If you wish to reproduce the build yourself, here are in-a-nutshell instructions: http://flashrom.org/Windows#Building_.28cross-compiling.29_flashrom_on_Linu…. > > Many thanks, > > > > Michael Ayles > > Test Development Engineer > > > > IMS Testing Solutions (Abtest) > > Abtest Ltd is a UKAS approved facility : Accreditation Number 0042 > > Tregwilym Industrial Estate, Rogerstone, Newport, South Wales, UK, NP10 9YA > > T: +44 (0) 1443 290030 F: +44 (0) 1633 892770 > > E: Michael.Ayles(a)ttelectronics-ims.com > > > > www.abtest.com > > > > Abtest Ltd. Registration Number 620992 trading as IMS Testing Solutions > > > > General Note: Please be aware,due to new security requirements , all visits > to Abtest must be booked in at least 24 hours in advance. > > > > Confidentiality Notice: > > > > This message is private and confidential. If received in error, please > destroy and notify sender. Sender does not intend to waive confidentiality > or privilege. Dissemination, use of or reliance upon this email is > prohibited when received in error. Email may be susceptible to data > corruption, interception and unauthorised amendment, and no liability is > accepted by the sender for any of the foregoing. It is the recipient’s > responsibility to scan the email and any attachment for viruses. flashrom is open source software (GPL), and its mailing list is public as well. Meeting such confidentiality disclaimers is impossible. See http://www.flashrom.org/pipermail/flashrom/ HTH, Idwer > > > > TT electronics integrated manufacturing services Limited is a company > registered in England and Wales. > > Registered number: 896672 > > Registered Office: Tregwilym Industrial Estate, Rogerstone, Newport, South > Wales, NP10 9YA, UK > > > > Please consider your environmental responsibility before printing this > e-mail > > > > > _______________________________________________ > flashrom mailing list > flashrom(a)flashrom.org > http://www.flashrom.org/mailman/listinfo/flashrom

1 0

Re: [flashrom] MX25L6445E erase and write failed!
by Stefan Tauner 26 Jun '15

26 Jun '15

On Wed, 4 Mar 2015 11:32:04 +0200 Urja Rannikko <urjaman(a)gmail.com> wrote: > Hi, > > > I've an idea. Maybe you've managed to switch the MX25L6445E into the > > "individual block protection mode" (read the datasheet page 30 - its > > an OTP, once done once its for good) and now it isnt actually > > erasing/writing anything at all (that 1M just happens to have > > succeeded previously) because flashrom currently doesnt know of the > > invidividual block unlock commands (or even the GBULK gang block > > unlock) and thinks everything is ok because the status register looks > > like unlocked. > > > > So maybe read the current contents into a file and modify a little bit > > in the erased area (eg. swap out the first 4k to random data*) and see > > if it writes. My theory is it wont. > > > > *here's example commands: > > flashrom <insert your parameters> -r current.bin > > dd if=/dev/urandom of=4krandom.bin bs=1024 count=4 > > dd if=current.bin bs=1024 skip=4 of=current-4.bin > > cat 4krandom.bin current-4.bin > modified.bin > > flashrom <parameters> -w modified.bin > > > > Somebody'd need to write a patch to flashrom to use the RDSCUR command > > to see if WPSEL is set to see for sure. But support for this wouldnt > > hurt anyways. I'll see if i have time later in the evening to make a > > prototype/test patch for this... > > I already had the datasheet open, so... here it is (in slightly crude form). > https://github.com/urjaman/flashrom/commit/b51f09e5f21bf14d492c7d006e0bdd86… > there's the commit if somebody wants to just web see. > To test, in the flashrom-serprog git tree you have: > git pull > git checkout mx25l6445e-debug > make > ./flashrom ... Hi, was this ever tested? I have a MX25L25635F which is yet unsupported due to the larger address space etc. and I could test this eventually some time in the future... but I would prefer to get support for this into flashrom earlier than that (2016? :) -- Kind regards/Mit freundlichen Grüßen, Stefan Tauner

2 1

404 error on link to precompiled windows exe
by Michael Ayles 25 Jun '15

25 Jun '15

Hi, I'm a test development engineer and one of our clients has developed a board with an EON EN25F20 SPI Flash. They provided a Bus Pirate and the firmware file to be programmed and told us they use linux with flashrom to program the parts. Unfortunately the test solution we have developed runs windows for our test solution. I've been unable to find a valid link to a precompiled executable of flashrom for windows. The link on the (flashrom.org/windows) page (http://ra.openbios.org/~idwer/flashrom/mingw/) returns a 404 error. Could you please help me locate a .exe, I'm more of a hardware guy than software and am struggling with the MINGW stuff. Many thanks, Michael Ayles Test Development Engineer IMS Testing Solutions (Abtest) Abtest Ltd is a UKAS approved facility : Accreditation Number 0042 Tregwilym Industrial Estate, Rogerstone, Newport, South Wales, UK, NP10 9YA T: +44 (0) 1443 290030 F: +44 (0) 1633 892770 E: Michael.Ayles(a)ttelectronics-ims.com www.abtest.com<http://www.abtest.com/> Abtest Ltd. Registration Number 620992 trading as IMS Testing Solutions General Note: Please be aware,due to new security requirements , all visits to Abtest must be booked in at least 24 hours in advance. Confidentiality Notice: This message is private and confidential. If received in error, please destroy and notify sender. Sender does not intend to waive confidentiality or privilege. Dissemination, use of or reliance upon this email is prohibited when received in error. Email may be susceptible to data corruption, interception and unauthorised amendment, and no liability is accepted by the sender for any of the foregoing. It is the recipient's responsibility to scan the email and any attachment for viruses. TT electronics integrated manufacturing services Limited is a company registered in England and Wales. Registered number: 896672 Registered Office: Tregwilym Industrial Estate, Rogerstone, Newport, South Wales, NP10 9YA, UK Please consider your environmental responsibility before printing this e-mail

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

flashrom June 2015