Attention is currently required from: Krystian Hebel, Maciej Pijanowski, Michał Kopeć, Michał Żygowski.
Nicholas Chin has posted comments on this change by Michał Kopeć. ( https://review.coreboot.org/c/coreboot/+/87044?usp=email )
Change subject: Documentation/mainboard/lenovo: Add ThinkCentre M700/M900 Tiny ......................................................................
Patch Set 3:
(3 comments)
File Documentation/mainboard/lenovo/thinkcentre_m900_tiny.md:
https://review.coreboot.org/c/coreboot/+/87044/comment/7cbfc7a3_10c8e903?usp... : PS3, Line 30: 25L12873F MX25L12873F?
https://review.coreboot.org/c/coreboot/+/87044/comment/329dc421_3f4731bb?usp... : PS3, Line 63: MrChromebox' EDK II fork Would be good to mention the version
https://review.coreboot.org/c/coreboot/+/87044/comment/b4047dd2_576f4cfa?usp... : PS3, Line 87: https://www.winbond.com/hq/product/code-storage-flash-memory/serial-nor-flas... This link doesn't seem to work as intended. Seems like the `&` gets converted to `&`, breaking the link (links to https://www.winbond.com/hq/product/code-storage-flash-memory/serial-nor-flas...). I haven't found a fix though; seems like this is a general issue with markdown. Maybe use a different link such as https://www.mouser.com/datasheet/2/949/w25q128jv_revf_03272018_plus-1489608.... ?