Attention is currently required from: Vadim Bendebury. Hello Vadim Bendebury,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/flashrom/+/64405
to review the following change.
Change subject: Add W25Q512NW-IM ID to flashrom ......................................................................
Add W25Q512NW-IM ID to flashrom
Add Winbond W25Q512NW-IM chip ID and specs to flashrom.
BUG=b:200173901 BRANCH=none TEST=flash W25Q512NW-IM using CCD.
Change-Id: I9debeda01d77444a5ebe9808ff80a337f320ef65 Signed-off-by: Atul Dhudase adhudase@codeaurora.org Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/flashrom/+... Reviewed-by: Shelley Chen shchen@chromium.org Reviewed-by: Vadim Bendebury vbendeb@chromium.org Tested-by: Shelley Chen shchen@chromium.org Commit-Queue: Shelley Chen shchen@chromium.org (cherry picked from commit facb282e8939b8e4ad15d2478ed9ef86d98aed61) Signed-off-by: Nikolai Artemiev nartemiev@google.com --- M flashchips.c M include/flashchips.h 2 files changed, 44 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/05/64405/1
diff --git a/flashchips.c b/flashchips.c index d155b24..7e37980 100644 --- a/flashchips.c +++ b/flashchips.c @@ -17962,6 +17962,49 @@ .voltage = {2700, 3600}, },
+ { + .vendor = "Winbond", + .name = "W25Q512NW-IM", + .bustype = BUS_SPI, + .manufacture_id = WINBOND_NEX_ID, + .model_id = WINBOND_NEX_W25Q512NW_IM, + .total_size = 64 * 1024, + .page_size = 256, + .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_4BA | FEATURE_4BA_ENTER_WREN | FEATURE_4BA_READ | FEATURE_4BA_FAST_READ, + .tested = TEST_OK_PREW, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 16384} }, + .block_erase = spi_block_erase_21, + }, { + .eraseblocks = { {4 * 1024, 16384} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {32 * 1024, 2048} }, + .block_erase = spi_block_erase_52, + }, { + .eraseblocks = { {64 * 1024, 1024} }, + .block_erase = spi_block_erase_dc, + }, { + .eraseblocks = { {64 * 1024, 1024} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {64 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {64 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .unlock = spi_disable_blockprotect, + .write = spi_chip_write_256, + .read = spi_chip_read, + .voltage = {1650, 1950}, + }, + { .vendor = "Winbond", .name = "W25Q64BV/W25Q64CV/W25Q64FV", diff --git a/include/flashchips.h b/include/flashchips.h index ba97719..999139e 100644 --- a/include/flashchips.h +++ b/include/flashchips.h @@ -983,6 +983,7 @@ #define WINBOND_NEX_W25Q64JW_M 0x8017 /* W25Q64JW...M */ #define WINBOND_NEX_W25Q128_DTR 0x8018 /* W25Q128JW_DTR */ #define WINBOND_NEX_W25Q256_DTR 0x8019 /* W25Q256JW_DTR aka W25Q256256JW-IM */ +#define WINBOND_NEX_W25Q512NW_IM 0x8020 /* W25Q512NW-IM */
#define WINBOND_ID 0xDA /* Winbond */ #define WINBOND_W19B160BB 0x49