Steve Markgraf has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43416 )
Change subject: flashchips: Add support for Macronix MX25L5121E ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c File flashchips.c:
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c@8577 PS1, Line 8577: spi_chip_write_256
Yes, I just did some testing with a layout file. […]
Done
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c@8578 PS1, Line 8578: MX25L5121E supports dual I/O
Oops, I thought I removed that.
Done
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c@8579 PS1, Line 8579: 2.35
Same here, 2.7V-3.6V is correct.
Done