Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38988 )
Change subject: mainboard: New ivybridge board ASUS P8Z77-M ......................................................................
Patch Set 7:
(5 comments)
https://review.coreboot.org/c/coreboot/+/38988/7//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38988/7//COMMIT_MSG@7 PS7, Line 7: mainboard: New ivybridge board ASUS P8Z77-M Please make it a statement by adding a verb (in imperative mood):
mainboard: Add new Ivy Bridge board ASUS P8Z77-M
https://review.coreboot.org/c/coreboot/+/38988/7//COMMIT_MSG@13 PS7, Line 13: - SeaBIOS boot What version, and what OS?
https://review.coreboot.org/c/coreboot/+/38988/7//COMMIT_MSG@18 PS7, Line 18: - Hardware monitoring under Linux What version?
https://review.coreboot.org/c/coreboot/+/38988/7//COMMIT_MSG@24 PS7, Line 24: - 2ch sound playback, Linux and Windows What versions?
https://review.coreboot.org/c/coreboot/+/38988/7/src/mainboard/asus/p8z77-m/... File src/mainboard/asus/p8z77-m/acpi/platform.asl:
https://review.coreboot.org/c/coreboot/+/38988/7/src/mainboard/asus/p8z77-m/... PS7, Line 6: Return(Package(){0,0}) Add spaces (`util/autoport/ec_none.go`).