diff options
author | Nikolai Artemiev <nartemiev@google.com> | 2021-05-14 15:26:47 +1000 |
---|---|---|
committer | Angel Pons <th3fanbus@gmail.com> | 2021-06-10 10:37:04 +0000 |
commit | 085db626fbf609704136aa3cc51c70325d0b40e2 (patch) | |
tree | 7d8e4a0df8d32ceae398dd1ede25eaf24d88afdf /flashchips.c | |
parent | 91254c5bca32c6c2ece8062047f6f3d0953a7113 (diff) | |
download | flashrom-085db626fbf609704136aa3cc51c70325d0b40e2.tar.gz flashrom-085db626fbf609704136aa3cc51c70325d0b40e2.tar.bz2 flashrom-085db626fbf609704136aa3cc51c70325d0b40e2.zip |
flashchips: change chip name from 'W25Q64JW' to 'W25Q64JW...M'
According to the W25Q64JW datasheet rev. E, only devices ending with the
letter 'M' have a device ID of 8017h. There are other variants with
different device IDs. This patch makes the 'W25Q64JW...M' definition
consistent with the 'W25Q32JW...M' definition.
The device ID macro defined in flashchips.h has also been renamed from
WINBOND_NEX_W25Q64JW to WINBOND_NEX_W25Q64JW_M.
BUG=b:166294558
BRANCH=none
TEST=builds
Change-Id: Ib0dc914da286a191d22e666332b1063b88db4251
Signed-off-by: Nikolai Artemiev <nartemiev@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54291
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Diffstat (limited to 'flashchips.c')
-rw-r--r-- | flashchips.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/flashchips.c b/flashchips.c index 1b281aa1..ef5ea5e2 100644 --- a/flashchips.c +++ b/flashchips.c @@ -17860,10 +17860,10 @@ const struct flashchip flashchips[] = { { .vendor = "Winbond", - .name = "W25Q64JW", + .name = "W25Q64JW...M", .bustype = BUS_SPI, .manufacture_id = WINBOND_NEX_ID, - .model_id = WINBOND_NEX_W25Q64JW, + .model_id = WINBOND_NEX_W25Q64JW_M, .total_size = 8192, .page_size = 256, /* OTP: 256B total; read 0x48; write 0x42, erase 0x44, read ID 0x4B */ |