Keith Hui has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64.V.M ......................................................................
flashchips: Add W25Q64.V.M
Add Winbond W25Q64JV?M SPI flash chip.
Some tests were done on a W25Q64JVSSIM mounted on a Sparkfun SOIC-DIP adapter, connected to an Arduino Mega 2560 r3 loaded with frser-duino. Probe and read works, but instability were observed in the connection between Arduino and the chip, so this is being submitted as untested.
Datasheet: https://www.winbond.com/resource-files/w25q64jv%20revj%2003272018%20plus.pdf
Change-Id: Ib7749835178ee6a1b0fe579e7a0c29e69a67933b Signed-off-by: Keith Hui buurin@gmail.com --- M flashchips.c M flashchips.h 2 files changed, 41 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/78/38578/1
diff --git a/flashchips.c b/flashchips.c index 65dda0e..bfa2927 100644 --- a/flashchips.c +++ b/flashchips.c @@ -16936,6 +16936,46 @@
{ .vendor = "Winbond", + .name = "W25Q64.V.M", + .bustype = BUS_SPI, + .manufacture_id = WINBOND_NEX_ID, + .model_id = WINBOND_NEX_W25Q64_V_M, + .total_size = 8192, + .page_size = 256, + /* supports SFDP */ + /* OTP: 1024B total, 256B reserved; read 0x48; write 0x42, erase 0x44, read ID 0x4B */ + .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_QPI, + .tested = TEST_UNTESTED, + .probe = probe_spi_rdid, + .probe_timing = TIMING_ZERO, + .block_erasers = + { + { + .eraseblocks = { {4 * 1024, 2048} }, + .block_erase = spi_block_erase_20, + }, { + .eraseblocks = { {32 * 1024, 256} }, + .block_erase = spi_block_erase_52, + }, { + .eraseblocks = { {64 * 1024, 128} }, + .block_erase = spi_block_erase_d8, + }, { + .eraseblocks = { {8 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_60, + }, { + .eraseblocks = { {8 * 1024 * 1024, 1} }, + .block_erase = spi_block_erase_c7, + } + }, + .printlock = spi_prettyprint_status_register_plain, + .unlock = spi_disable_blockprotect, + .write = spi_chip_write_256, + .read = spi_chip_read, + .voltage = {2700, 3600}, + }, + + { + .vendor = "Winbond", .name = "W25Q64.W", .bustype = BUS_SPI, .manufacture_id = WINBOND_NEX_ID, diff --git a/flashchips.h b/flashchips.h index 14ab6de..bdfadad 100644 --- a/flashchips.h +++ b/flashchips.h @@ -938,6 +938,7 @@ #define WINBOND_NEX_W25Q32_W 0x6016 /* W25Q32DW; W25Q32FV in QPI mode */ #define WINBOND_NEX_W25Q64_W 0x6017 /* W25Q64DW; W25Q64FV in QPI mode */ #define WINBOND_NEX_W25Q128_W 0x6018 /* W25Q128FW; W25Q128FV in QPI mode */ +#define WINBOND_NEX_W25Q64_V_M 0x7017 /* W25Q64JVSIM */ #define WINBOND_NEX_W25Q128_V_M 0x7018 /* W25Q128JVSM */ #define WINBOND_NEX_W25Q256JV_M 0x7019 /* W25Q256JV_M (QE=0) */ #define WINBOND_NEX_W25Q128_DTR 0x8018 /* W25Q128JW_DTR */
Keith Hui has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64.V.M ......................................................................
Patch Set 1:
This chip does work for probe/read/write/erase, after I switched to another level shifter setup on a breadboard[1].
[1] http://forums.parallax.com/discussion/160963/5v-and-the-propeller (7th comment with schematic. Textual description: 2N3904 transistor, 5V in to E, 3.3V out to C, 1kohm from 5V to E, 1kohm from 3.3V to each of B and C, one set each for CS,MOSI,CLK, I tested OK with 10kohm instead)
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/38578
to look at the new patch set (#2).
Change subject: flashchips: Add W25Q64JV.M ......................................................................
flashchips: Add W25Q64JV.M
Add Winbond W25Q64JV?M SPI flash chip.
Tested on a W25Q64JVSSIM mounted on a Sparkfun SOIC-DIP adapter, connected to an Arduino Mega 2560 r3 loaded with frser-duino. Chip then mounted on ASUS P8Z77-M board where it still works.
Datasheet: https://www.winbond.com/resource-files/w25q64jv%20revj%2003272018%20plus.pdf
Change-Id: Ib7749835178ee6a1b0fe579e7a0c29e69a67933b Signed-off-by: Keith Hui buurin@gmail.com --- M flashchips.c M flashchips.h 2 files changed, 41 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/78/38578/2
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 2:
Looks like a duplicate of CB:40311 ?
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/flashrom/+/38578/2/flashchips.c File flashchips.c:
https://review.coreboot.org/c/flashrom/+/38578/2/flashchips.c@17165 PS2, Line 17165: FEATURE_QPI This seems to be wrong
Keith Hui has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 2:
Patch Set 2:
Looks like a duplicate of CB:40311 ?
Um, you first submitted it on April 10.
This has been sitting here since January 25 with no feedback.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 2:
Patch Set 2:
Patch Set 2:
Looks like a duplicate of CB:40311 ?
Um, you first submitted it on April 10.
This has been sitting here since January 25 with no feedback.
Right, I thought this one was newer. In any case, the only difference is that I removed 4BA support (seems to be unsupported according to the datasheet)
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 4: Code-Review+2
Attention is currently required from: Keith Hui. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4: Seems to need a manual rebase.
Keith Hui has abandoned this change. ( https://review.coreboot.org/c/flashrom/+/38578 )
Change subject: flashchips: Add W25Q64JV.M ......................................................................
Abandoned
merged as 58971
-
Angel Pons (Code Review) -
Keith Hui (Code Review)