Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/43416 )
Change subject: flashchips: Add support for Macronix MX25L5121E ......................................................................
Patch Set 1:
(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 Does this work as intended? I would try to use a layout file to see if partial writes work
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c@8578 PS1, Line 8578: MX25L5121E supports dual I/O Where do you see this?
https://review.coreboot.org/c/flashrom/+/43416/1/flashchips.c@8579 PS1, Line 8579: 2.35 I see "2.7 to 3.6 volt for read, erase and program operations"