Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/34234 )
Change subject: flashchips: Add Macronix MX25L51245G as known chip ......................................................................
Patch Set 3: Code-Review+2
(2 comments)
https://review.coreboot.org/c/flashrom/+/34234/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/flashrom/+/34234/3//COMMIT_MSG@9 PS3, Line 9: MX25L51245G is identical to handling of MX66L51235F Full stop at the end, please.
https://review.coreboot.org/c/flashrom/+/34234/3/flashchips.h File flashchips.h:
https://review.coreboot.org/c/flashrom/+/34234/3/flashchips.h@493 PS3, Line 493: MX66L51235F This is the same name as on the left. I had too look twice. I guess the original comment shouldn't have been there in the first place.