Tristan Corrick has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30767 )
Change subject: mb/asus/h61m-cs: Add ASUS H61M-CS ......................................................................
Patch Set 10: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/30767/10/src/mainboard/asus/h61m-cs/Kconfig File src/mainboard/asus/h61m-cs/Kconfig:
https://review.coreboot.org/#/c/30767/10/src/mainboard/asus/h61m-cs/Kconfig@... PS10, Line 9: select INTEL_INT15 Thanks for adding the VBT. `select INTEL_GMA_HAVE_VBT` should be added too.
https://review.coreboot.org/#/c/30767/10/src/mainboard/asus/h61m-cs/devicetr... File src/mainboard/asus/h61m-cs/devicetree.cb:
https://review.coreboot.org/#/c/30767/10/src/mainboard/asus/h61m-cs/devicetr... PS10, Line 60: device pci 1c.5 on # Realtek Gigabit NIC I thought lspci said the Realtek NIC was under 1c.4, why the change? Have you tested the Ethernet? If 1c.4 is unused, please change it to `off`.