mikeb mikeb has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31448 )
Change subject: src/device/pci: Add support for discrete VGA initialization and OpROM loading ......................................................................
Patch Set 14:
(1 comment)
https://review.coreboot.org/#/c/31448/14/src/device/pci_rom.c File src/device/pci_rom.c:
https://review.coreboot.org/#/c/31448/14/src/device/pci_rom.c@276 PS14, Line 276: if (CONFIG(MULTIPLE_VGA_ADAPTERS)) : return current; : /* Write ACPI VFCT only for Discrete VGA. */ : /* TODO: do this also for Integrated VGA. */ : break; : case PCI_CLASS_DISPLAY_OTHER: : if (!CONFIG(MULTIPLE_VGA_ADAPTERS)) : return current; : break;
So if I understood it correctly only one entry of the VFCT gets used by the radeon driver and this i […]
I don't know how much time it'll take to implement the alternative solution, and for the coreboot users it would be nice if this "hack" could be accepted until the alternative approach is successfully completed. Although then there would be less reasons to develop this alternative better approach ("since the things are already working"), so I could understand the people for not wanting to accept this.