Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30987 )
Change subject: mb/asus: Add Asus A88XM-E FM2+ with documentation ......................................................................
Patch Set 58:
(2 comments)
Great work. What changes were made to the existing boards?
https://review.coreboot.org/c/coreboot/+/30987/58/src/mainboard/asus/a88xm-e... File src/mainboard/asus/a88xm-e/OemCustomize.c:
https://review.coreboot.org/c/coreboot/+/30987/58/src/mainboard/asus/a88xm-e... PS58, Line 134: * CUSTOMER OVERRIDES MEMORY TABLE Could you please fix the typo in the current code first, so that these files can be compared with `diff` easily?
https://review.coreboot.org/c/coreboot/+/30987/58/src/mainboard/asus/a88xm-e... PS58, Line 156: */ Please remove, and mention it in the commit message.