Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44302 )
Change subject: mb/lenovo/thinkcentre_m91p add ThinkCentre M91p ......................................................................
Patch Set 13:
(4 comments)
https://review.coreboot.org/c/coreboot/+/44302/13//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44302/13//COMMIT_MSG@7 PS13, Line 7: mb/lenovo/thinkcentre_m91p add ThinkCentre M91p Please add a colon after the prefix.
https://review.coreboot.org/c/coreboot/+/44302/13//COMMIT_MSG@10 PS13, Line 10: This is an OEM board with a Macronix MX25L6406E flash chip 1. Please add dots at the end of sentences. 2. Please mention what chipset/socket this mainboard has.
https://review.coreboot.org/c/coreboot/+/44302/13//COMMIT_MSG@13 PS13, Line 13: - All DIMM slots Do you know the DIMM model?
https://review.coreboot.org/c/coreboot/+/44302/13//COMMIT_MSG@21 PS13, Line 21: SATA Ports SATA ports