openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-27 15:12:36 +02:00
Code Issues Releases Wiki Activity

realtek_mst_i2c_spi.c: Fix cmd timeout issue

Browse Source 
Chip erasures take much longer than sector and bank
erasures. Allow the wait loop helper to multiply the
max timeout in this very specific case while quickly
timeout for other ops that are expected to be shorter.

V.2: Fix nonsense fall though warn-err

BUG=b:152558985,b:148745673
BRANCH=none
TEST=flashrom -p realtek_mst_i2c_spi:bus=8 -E &&
     flashrom -p realtek_mst_i2c_spi:bus=8 -w foo
    (cycle)..

Change-Id: I4a36aa3101827e69eb244775d25bbb476d4bb780
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/41123
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sam McNally <sammc@google.com>
This commit is contained in:
Edward O'Callaghan 2020-05-07 11:42:56 +10:00 committed by Edward O'Callaghan
parent 6a5472cee9
commit a9abaae74f
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
realtek_mst_i2c_spi.c
View File

@ -96,21 +96,22 @@ static int realtek_mst_i2c_spi_read_register(int fd, uint8_t reg, uint8_t *value
return ret ? SPI_GENERIC_ERROR : 0; return ret ? SPI_GENERIC_ERROR : 0;
} }
static int realtek_mst_i2c_spi_wait_command_done(int fd, unsigned int offset, int mask, int target) static int realtek_mst_i2c_spi_wait_command_done(int fd, unsigned int offset, int mask,
int target, int multiplier)
{ {
uint8_t val; uint8_t val;
int tried = 0; int tried = 0;
int ret = 0; int ret = 0;
do { do {
ret |= realtek_mst_i2c_spi_read_register(fd, offset, &val); ret |= realtek_mst_i2c_spi_read_register(fd, offset, &val);
} while (!ret && ((val & mask) != target) && ++tried < MAX_SPI_WAIT_RETRIES); } while (!ret && ((val & mask) != target) && ++tried < (MAX_SPI_WAIT_RETRIES*multiplier));
if (tried == MAX_SPI_WAIT_RETRIES) { if (tried == MAX_SPI_WAIT_RETRIES) {
msg_perr("%s: Time out on sending command.\n", __func__); msg_perr("%s: Time out on sending command.\n", __func__);
return -MAX_SPI_WAIT_RETRIES; return -MAX_SPI_WAIT_RETRIES;
} }
return (val & mask) ? SPI_GENERIC_ERROR : ret; return (val & mask) != target ? SPI_GENERIC_ERROR : ret;
} }
static int realtek_mst_i2c_spi_enter_isp_mode(int fd) static int realtek_mst_i2c_spi_enter_isp_mode(int fd)
@ -130,7 +131,7 @@ static int realtek_mst_i2c_spi_enter_isp_mode(int fd)
static int realtek_mst_i2c_execute_write(int fd) static int realtek_mst_i2c_execute_write(int fd)
{ {
int ret = realtek_mst_i2c_spi_write_register(fd, MCU_MODE, START_WRITE_XFER); int ret = realtek_mst_i2c_spi_write_register(fd, MCU_MODE, START_WRITE_XFER);
ret |= realtek_mst_i2c_spi_wait_command_done(fd, MCU_MODE, WRITE_XFER_STATUS_MASK, 0); ret |= realtek_mst_i2c_spi_wait_command_done(fd, MCU_MODE, WRITE_XFER_STATUS_MASK, 0, 1);
return ret; return ret;
} }
@ -202,6 +203,7 @@ static int realtek_mst_i2c_spi_send_command(const struct flashctx *flash,
unsigned char *readarr) unsigned char *readarr)
{ {
unsigned i; unsigned i;
int max_timeout_mul = 1;
int ret = 0; int ret = 0;
if (writecnt > 4 || readcnt > 3 || writecnt == 0) { if (writecnt > 4 || readcnt > 3 || writecnt == 0) {
@ -239,8 +241,10 @@ static int realtek_mst_i2c_spi_send_command(const struct flashctx *flash,
ctrl_reg_val |= (2 << 5); ctrl_reg_val |= (2 << 5);
break; break;
/* Erasures require BIT7 && BIT5 set. */ /* Erasures require BIT7 && BIT5 set. */
case JEDEC_CE_60:
case JEDEC_CE_C7: case JEDEC_CE_C7:
max_timeout_mul *= 20; /* chip erasures take much longer! */
/* FALLTHRU */
case JEDEC_CE_60:
case JEDEC_BE_52: case JEDEC_BE_52:
case JEDEC_BE_D8: case JEDEC_BE_D8:
case JEDEC_BE_D7: case JEDEC_BE_D7:
@ -261,7 +265,7 @@ static int realtek_mst_i2c_spi_send_command(const struct flashctx *flash,
if (ret) if (ret)
return ret; return ret;
ret = realtek_mst_i2c_spi_wait_command_done(fd, 0x60, 0x01, 0); ret = realtek_mst_i2c_spi_wait_command_done(fd, 0x60, 0x01, 0, max_timeout_mul);
if (ret) if (ret)
return ret; return ret;
@ -321,7 +325,7 @@ static int realtek_mst_i2c_spi_read(struct flashctx *flash, uint8_t *buf,
if (ret) if (ret)
return ret; return ret;
ret = realtek_mst_i2c_spi_wait_command_done(fd, 0x60, 0x01, 0); ret = realtek_mst_i2c_spi_wait_command_done(fd, 0x60, 0x01, 0, 1);
if (ret) if (ret)
return ret; return ret;
@ -373,7 +377,7 @@ static int realtek_mst_i2c_spi_write_256(struct flashctx *flash, const uint8_t *
break; break;
/* Wait for empty buffer. */ /* Wait for empty buffer. */
ret |= realtek_mst_i2c_spi_wait_command_done(fd, MCU_MODE, 0x10, 0x10); ret |= realtek_mst_i2c_spi_wait_command_done(fd, MCU_MODE, 0x10, 0x10, 1);
if (ret) if (ret)
break; break;