Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48862 )
Change subject: drivers/intel/gma: Rework brightness level includes ......................................................................
Patch Set 1:
I'm somehow missing, what the improvement here really is besides that the custom brightness table version is more explicit and more flexible.
the improvement is that this allows gma.asl to be included to provide the GFX0 device without forcing the inclusion of unnecessary brightness controls for devices without an internal panel.
the use case driving this change is running MacOS on Chromeboxes where the GFX0 device is required
We had investigated the GFX0 requirement lately and only found two references in coreboot code (drivers/intel/gma and drivers/gfx/generic). Neither of them should be used for Chromeboxes AFAICS. So maybe there is some unwanted boilerplate left that depends on it? Or is the GFX0 hardcoded in a MacOS driver?