[coreboot-gerrit] Change in coreboot[master]: mb/gigabyte/ga-b75m-d3h: add libgfxinit support

[Bill XIE (Code Review)]

Bill XIE has posted comments on this change. ( https://review.coreboot.org/19522 )

Change subject: mb/gigabyte/ga-b75m-d3h: add libgfxinit support
......................................................................


Patch Set 3:

(2 comments)

https://review.coreboot.org/#/c/19522/3/src/mainboard/gigabyte/ga-b75m-d3h/gma-mainboard.ads
File src/mainboard/gigabyte/ga-b75m-d3h/gma-mainboard.ads:

PS3, Line 10:      (DP1,
            :       DP2,
> They are, the pins on the PCH are always DP++. Since your DVI-D
So the statements for DPs can actually be removed?


Line 13:       HDMI2,
> Linux can skip gaps in the numbering. So what you see in xrandr
But can this output prove that there is no HDMI3?


-- 
To view, visit https://review.coreboot.org/19522
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If00a7247df0c32b3d1f489fb92d86baaa8fdf8ba
Gerrit-PatchSet: 3
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Bill XIE <persmule at gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Bill XIE <persmule at gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-HasComments: Yes