From 918f21b72d6bc55bd9e43eebfd7e804752a8605e Mon Sep 17 00:00:00 2001 From: Keith Hui Date: Sat, 19 Apr 2025 22:48:02 -0400 Subject: [PATCH] drivers/spi/winbond.c: Add W25Q64JV signature I am using a pair of Winbond W25Q64JVSSIM SPI flash chips for coreboot development. While working on Asus P8x7x Series autonomous soft strap update support, aka CB:85413 and CB:87334, I found that without its signature (mfgr 0xef, dev 0x7017) added, rdev_writeat() would fail to get anything into the flash chip even though it reports success. Its other parameters are copied from W25Q64_V as the two are almost the same. Change-Id: I4af6268a2a1bde4d2ff9c76879c3bc59b91a916d Signed-off-by: Keith Hui Reviewed-on: https://review.coreboot.org/c/coreboot/+/87363 Tested-by: build bot (Jenkins) Reviewed-by: Matt DeVillier --- src/drivers/spi/winbond.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/drivers/spi/winbond.c b/src/drivers/spi/winbond.c index db93ff616c..2999676b08 100644 --- a/src/drivers/spi/winbond.c +++ b/src/drivers/spi/winbond.c @@ -157,6 +157,15 @@ static const struct spi_flash_part_id flash_table[] = { .protection_granularity_shift = 17, .bp_bits = 3, }, + { + /* W25Q64JV */ + .id[0] = 0x7017, + .nr_sectors_shift = 11, + .fast_read_dual_output_support = 1, + .fast_read_dual_io_support = 1, + .protection_granularity_shift = 17, + .bp_bits = 3, + }, { /* W25Q64JW */ .id[0] = 0x8017,