[coreboot-gerrit] Change in coreboot[master]: supermicro/x11-lga1151-series: Remove SkipExtGfxScan = 1

[Michael Niewöhner (Code Review)]

Attention is currently required from: Jan Tatje.
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56071 )

Change subject: supermicro/x11-lga1151-series: Remove SkipExtGfxScan = 1
......................................................................

Patch Set 3:

(1 comment)

Patchset:

PS2:
> Without the change and CONFIG_GENERIC_LINEAR_FRAMEBUFFER=y set it also works, I hadn't tried that be […]

Thanks! On IRC we figured out what's going on. VGA does not work because IGD uses the legacy VGA resources when setting SkipExtGfxScan=1.

PrimaryDisplay=AUTO for X11SSH is already correct, but we should set this for the others, too, so FSP does not prefer IGD over VGA (which leads to the same problem). So, let's move the setting from X11SSH to the mainboard devicetree, since all four boards need it.