Patch Set 24:
(10 comments)
11 comments:
Patch Set #24, Line 7: mb/ongy/h61m-s1: Add new mainboard
Mention the socket/chipset in the summary, so people can better classify the device at first look.
Done
I have never heard of them.
Done
Patch Set #24, Line 9: Port done with the autoport utility
Please add a dot/period at the end.
Done
As there is no documentation added, please mention how to flash the image. […]
Done
Patch Set #24, Line 14: 4.15.0-99-generic
I thought that is the Linux kernel for 16.04.
Done
Patch Set #24, Line 25: - Tianocore payload (CorebootPayload)
What is the error?
Done
File src/mainboard/ongy/Kconfig:
Please remove the blank line.
Done
File src/mainboard/ongy/h61m-s1/acpi/platform.asl:
Please format the comment correctly. […]
Done
Patch Set #24, Line 11: /*The _WAK method is called on system wakeup*/
Please add spaces.
Done
Patch Set #24, Line 15: Return(Package(){0,0})
Please add spaces. Current autoport should have fixed this.
Done
File src/mainboard/ongy/h61m-s1/mainboard.c:
Patch Set #24, Line 9: install_intel_vga_int15_handler
I'm not sure if you need this as your maindoard is a disktop.
it is used in other H61 mainboards: https://github.com/coreboot/coreboot/blob/master/src/mainboard/gigabyte/ga-h61m-series/mainboard.c
To view, visit change 40813. To unsubscribe, or for help writing mail filters, visit settings.