Attention is currently required from: Edward O'Callaghan, Angel Pons. Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/53946 )
Change subject: flashchips.c: add support for W25Q32JW..IM ......................................................................
Patch Set 4:
(4 comments)
File flashchips.c:
https://review.coreboot.org/c/flashrom/+/53946/comment/c9203535_23293a8c PS1, Line 17583: ..IM
For consistency with W25Q64JW, I'd drop the trailing `.. […]
This only works with `..IM` chips; there are other variants such as `W25Q32JW..IQ` that have a different device ID.
We should probably change `W25Q64JW` to `W25Q64JW..IM` though.
https://review.coreboot.org/c/flashrom/+/53946/comment/d59fb65f_9c64e946 PS1, Line 17589: FEATURE_WRSR_WREN
Datasheet says that OTP is supported
Done
https://review.coreboot.org/c/flashrom/+/53946/comment/828be866_172c078e PS1, Line 17611: },
Add: […]
Done
https://review.coreboot.org/c/flashrom/+/53946/comment/b0d5edb3_5f47a4f4 PS1, Line 17612: spi_disable_blockprotect
spi_disable_blockprotect_bp2_srwd seems more accurate. Not perfect, but better.
Done