Angel Pons 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:
(3 comments)
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG Commit Message:
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.
The libgfxinit things are not about external graphics cards
https://review.coreboot.org/c/coreboot/+/34603/97//COMMIT_MSG@35 PS97, Line 35: S3 suspend/resume It would be good to describe what happens instead of the expected outcome.
https://review.coreboot.org/c/coreboot/+/34603/97/src/mainboard/asus/h110m-e... File src/mainboard/asus/h110m-e_m2/ramstage.c:
https://review.coreboot.org/c/coreboot/+/34603/97/src/mainboard/asus/h110m-e... PS97, Line 13: CdClock
How to test it or should I set it to Auto from 3?
AFAIK, remove the line then check if video outputs still work