Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41260 )
Change subject: mb/msi/h81m-p33: Add new mainboard ......................................................................
Patch Set 7:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41260/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/41260/4//COMMIT_MSG@25 PS4, Line 25: - SeaBIOS to boot Arch Linux
Versions?
Added SeaBIOS version, Arch Linux doesn't really have versions, and the kernel version changes rather often
https://review.coreboot.org/c/coreboot/+/41260/4/src/mainboard/msi/h81m-p33/... File src/mainboard/msi/h81m-p33/acpi/platform.asl:
https://review.coreboot.org/c/coreboot/+/41260/4/src/mainboard/msi/h81m-p33/... PS4, Line 9: Return(Package(){0, 0})
Add spaces?
What for? I'm not sure if we have any coding style for ASL. Since I don't really care about ASL formatting, maybe we could auto-format them?