Attention is currently required from: Nico Huber, Angel Pons, Anastasia Klimchuk. Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59073 )
Change subject: [RFC][WPTST] dummyflasher: emualte SR2 for W25Q128FV ......................................................................
Patch Set 11:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/59073/comment/4a2e9f6f_9ec80c88 PS10, Line 8: : By the way, W25Q128FV has three status registers, but no plans to use : the third one yet.
If you remove "by the way", it is a very good second paragraph for this commit message! […]
Done
File dummyflasher.c:
https://review.coreboot.org/c/flashrom/+/59073/comment/e787373b_fc8ab20f PS10, Line 922: data->emu_status_len = 2;
I added a comment to previous patch about emu_status_len always 1, I see that it becomes 2 in this p […]
These changes are split into addition of SR2 and use of SR2. Can squash commits into one, but I thought it's better to separate general support from a chip-specific one.
-
Sergii Dmytruk (Code Review)