Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31603 )
Change subject: mainboard: Add ASRock H110M-DVS ......................................................................
Patch Set 2: Code-Review+1
(8 comments)
Once this builds on Jenkins, it should be submitted, and other fixes done in follow-up commits.
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@9 PS2, Line 9: Kabylake Kaby Lake
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@11 PS2, Line 11: OS Maybe add *(Ubuntu 16.04.2 with Linux 4.15)* already here.
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@13 PS2, Line 13: Intel FSP2.0 I do not know much about that, but is there a revision or something similar?
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@16 PS2, Line 16: Work Works
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@17 PS2, Line 17: - Integrated graphics (only DVI port); What resolution did you use?
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@28 PS2, Line 28: Seabios SeaBIOS (what version)
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@28 PS2, Line 28: Tianocore TianoCore (what version)?
https://review.coreboot.org/#/c/31603/2//COMMIT_MSG@30 PS2, Line 30: What about suspend and resume?