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

[Michael Niewöhner (Code Review)]

Attention is currently required from: Nico Huber, 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:
> > FSP does this: […]

SkipExtGfxScan=0 let's FSP scan for PCIe graphics. If found, it sets PrimaryDisplay=PCI. Then it scans for PEG. If found, PrimaryDisplay becomes PEG, but only of PCI was not found... so, PCI would even take precedence over PEG.

With SkipExtGfxScan=1 the difference between PEG and PCI is gone, so you're right, that we just must not set Auto or IGD.

=> we should keep SkipExtGfxScan=1 (or maybe even drop the dt option and set it statically?) and set PrimaryDisplay=PCI/PEG