Vlado Cibic has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33328 )
Change subject: mainboard: Add support for ASUS P8Z77-M PRO desktop mainboard ......................................................................
Patch Set 62: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/33328/61/Documentation/mainboard/asus/p8z77-... File Documentation/mainboard/asus/p8z77-m_pro.md:
https://review.coreboot.org/#/c/33328/61/Documentation/mainboard/asus/p8z77-... PS61, Line 28: ![](p8z77-m_pro.jpg)
do you have the copyright on the picture provided?
Now yes. I just took a photo with my phone.
https://review.coreboot.org/#/c/33328/61/Documentation/mainboard/asus/p8z77-... PS61, Line 34: An external programmer is required
period is missing […]
About the period, our whole policy for the text file was not to use periods at the end of the paragraphs. So, for consistency, we won't add that period.
We added a mention to an external flash method as you suggested (CH341a+flashrom).
thx!