Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34603 )
Change subject: mb/asus: Add ASUS H110M-E/M.2 mainboard ......................................................................
Patch Set 97:
(5 comments)
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@12 PS97, Line 12: Seabios SeaBIOS
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@19 PS97, Line 19: - Intel ME(cleaned) Please add a space before the (.
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@20 PS97, Line 20: no difference observed between use of ME and ME(cleaned). Ditto.
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@31 PS97, Line 31: , Remove the first space.
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@31 PS97, Line 31: - External Graphics Cards works only with GOP, libgfxinit just wakes the main display with a blinking cursor even if I log into the system (when nomodeset kernel parameter enabled) through serial port (For MS Windows 8.1, the GUI works but installing the graphics driver trips the machine and for GNU/Linux (Linux 4.18, 5.4, Linux-libre 5.4, 5.5) , display only works with "nomodeset" kernel parameter). 3 out of 4 times the machine trips during the coreboot boot process at FspSiliconInit and if not then in the Graphics initialization process, once out of 100 times it somehow booted into GUI only on Ubuntu (Linux 4.18). Please wrap the lines after 75 characters.