Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38202 )
Change subject: src/device/pci: Add support for discrete VGA initialization and OpROM loading ......................................................................
Patch Set 3:
(1 comment)
This change makes a difference only if your discrete VGA doesn't have its own memory for storing a VBIOS (or maybe if you'd like to load a custom VBIOS on your GPU without flashing it to its' memory - but I haven't explored this yet). Otherwise there shouldn't be any difference. So currently it benefits only a few boards like Lenovo G505S.
https://review.coreboot.org/c/coreboot/+/38202/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38202/2//COMMIT_MSG@11 PS2, Line 11: If CONFIG_MULTIPLE_VGA_ADAPTERS is enabled: coreboot will try to initialize the : discrete VGA of PCI_CLASS_DISPLAY_OTHER class and load OpROM on it. ACPI VFCT : table will be created only for discrete VGA of PCI_CLASS_DISPLAY_OTHER class : instead of integrated (to avoid the discrete VGA init problems), but everything : is working fine. Later will try to make a common VFCT table for both adapters.
Please wrap at 72 characters
Done.