Attention is currently required from: Angel Pons, Anastasia Klimchuk, Sergii Dmytruk. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59071 )
Change subject: flashchips: enable write-protection for W25Q{64,128}.V ......................................................................
Patch Set 15:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/59071/comment/8270980f_85d83b48 PS15, Line 9: W25Q64 Which version specifically?
Patchset:
PS15: So, there are two wrinkles with these Winbond chips:
1. Newer versions with an additional, 3rd status register and WPS feature re-use the older RDID numbers. 2. To support write-protection on these newer chips, the code needs to be extended with WPS support first (for a start it would be enough to ensure the WPS bit is 0, we could extend the API for individual block protection later).
So far the only solution I see for 1. is to split the entries in `flashchip.c`. So for W25Q64.V this would become two entries, one for W25Q64BV/W25Q64CV/W25Q64FV w/o WPS support and one for W25Q64JV w/ WPS support (or w/o write-protect support for now).