Namyoon Woo has uploaded this change for review.

View Change

support 4-byte address format for VARIOUS_SIZE dummy flash device

This patch adds a support of 4-byte address format for VARIOUS_SIZE
dummy flash device, so that it can emulate an flash size larger than
16 MBytes.

TEST=ran the command line below:
$ flashrom -p dummy:image=${TMP_FILE},size=33554432, \
emulate=VARIABLE_SIZE -w ${IMG_32MB} -V -f

$ flashrom -p dummy:image=${TMP_FILE},size=16777216, \
emulate=VARIABLE_SIZE -w ${IMG_16MB} -V -f

$ flashrom -p dummy:image=${TMP_FILE},size=8388608, \
emulate=VARIABLE_SIZE -w ${IMG_8MB} -V -f

Signed-off-by: Namyoon Woo <namyoon@google.com>
Change-Id: Ia59eecfcbe798d50f8dacea98c3c508edf8ec77e
---
M dummyflasher.c
M flashchips.c
2 files changed, 25 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/81/44881/1
diff --git a/dummyflasher.c b/dummyflasher.c
index 4024f2a..b5a0f1b 100644
--- a/dummyflasher.c
+++ b/dummyflasher.c
@@ -719,6 +719,13 @@
if (readcnt > 0)
memcpy(readarr, flashchip_contents + offs, readcnt);
break;
+ case JEDEC_READ_4BA:
+ offs = writearr[1] << 24 | writearr[2] << 16 | writearr[3] << 8 | writearr[4];
+ /* Truncate to emu_chip_size. */
+ offs %= emu_chip_size;
+ if (readcnt > 0)
+ memcpy(readarr, flashchip_contents + offs, readcnt);
+ break;
case JEDEC_BYTE_PROGRAM:
offs = writearr[1] << 16 | writearr[2] << 8 | writearr[3];
/* Truncate to emu_chip_size. */
@@ -733,6 +740,20 @@
}
memcpy(flashchip_contents + offs, writearr + 4, writecnt - 4);
break;
+ case JEDEC_BYTE_PROGRAM_4BA:
+ offs = writearr[1] << 24 | writearr[2] << 16 | writearr[3] << 8 | writearr[4];
+ /* Truncate to emu_chip_size. */
+ offs %= emu_chip_size;
+ if (writecnt < 6) {
+ msg_perr("BYTE PROGRAM size too short!\n");
+ return 1;
+ }
+ if (writecnt - 5 > emu_max_byteprogram_size) {
+ msg_perr("Max BYTE PROGRAM size exceeded!\n");
+ return 1;
+ }
+ memcpy(flashchip_contents + offs, writearr + 5, writecnt - 5);
+ break;
case JEDEC_AAI_WORD_PROGRAM:
if (!emu_max_aai_size)
break;
@@ -978,8 +999,8 @@
if (eraser->block_erase == NULL)
break;

- eraser->eraseblocks[0].count = emu_chip_size /
- eraser->eraseblocks[0].size;
+ eraser->eraseblocks[0].count = 1;
+ eraser->eraseblocks[0].size = emu_chip_size;
msg_cdbg("%s: eraser.size=%d, .count=%d\n",
__func__, eraser->eraseblocks[0].size,
eraser->eraseblocks[0].count);
diff --git a/flashchips.c b/flashchips.c
index e99073b..34b7702 100644
--- a/flashchips.c
+++ b/flashchips.c
@@ -18765,13 +18765,14 @@
.model_id = VARIABLE_SIZE_DEVICE_ID,
.total_size = 64, /* This size is set temporarily */
.page_size = 256,
+ .feature_bits = FEATURE_4BA,
.tested = TEST_OK_PREW,
.probe = probe_variable_size,
.block_erasers =
{
{
.eraseblocks = { {64 * 1024, 1} },
- .block_erase = spi_block_erase_d8,
+ .block_erase = spi_block_erase_c7,
}
},
.write = spi_chip_write_256,

To view, visit change 44881. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ia59eecfcbe798d50f8dacea98c3c508edf8ec77e
Gerrit-Change-Number: 44881
Gerrit-PatchSet: 1
Gerrit-Owner: Namyoon Woo <namyoon@google.com>
Gerrit-MessageType: newchange