Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30895 )
Change subject: Documentation: Add Asus F2A85-M 2nd version, reworked based on Asus P8H61-M LX as a template and the old wiki ......................................................................
Patch Set 2: Code-Review+1
(4 comments)
Looking much better!
https://review.coreboot.org/#/c/30895/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/30895/2//COMMIT_MSG@8 PS2, Line 8: 2nd version, reworked based on Asus P8H61-M LX as a template and the old wiki Please add an empty line between the commit summary and the rest of the commit message
https://review.coreboot.org/#/c/30895/2/Documentation/mainboard/asus/f2a85-m... File Documentation/mainboard/asus/f2a85-m.md:
https://review.coreboot.org/#/c/30895/2/Documentation/mainboard/asus/f2a85-m... PS2, Line 8: - ASUS F2A85-M PRO - the most available model in the entire F2A85 series. Be sure to check its status to see what works and what doesn't. I think there is a line length limit, this mahy have to be split
https://review.coreboot.org/#/c/30895/2/Documentation/mainboard/asus/f2a85-m... PS2, Line 19: AX What does this mean?
https://review.coreboot.org/#/c/30895/2/Documentation/mainboard/asus/f2a85-m... PS2, Line 104: : ## Known issues Everything regarding what works and what does not is probably outdated. If you have any of the F2A85 series mainboards, could you please test with coreboot master? Thanks!