1 comment:
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;
I don't know how much time it'll take to implement the alternative solution, and for the coreboot us […]
Getting code upstream is about working on nice and maintainable solutions to problems and getting feedback on how to achieve that. Getting hardware to work is nice ofc, but not sufficient reason to accept it upstream. If you don't want to work on it anymore, users can still get your patches.
If you have questions about the suggestions made in https://review.coreboot.org/c/coreboot/+/31502 , I and others are very willing to answer them. It might also be a good idea to look at the vendor ACPI code to figure out what it does?
To view, visit change 31448. To unsubscribe, or for help writing mail filters, visit settings.