Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/80060?usp=email )
Change subject: soc/intel/common/gfx: Only generate a single SSDT entry for the GPU ......................................................................
soc/intel/common/gfx: Only generate a single SSDT entry for the GPU
If a mainboard defines its display outputs using a generic graphics chip driver, then don't generate the default SSDT for the iGPU, since then there will be duplicate _DOD methods under the iGPU, which will cause some OSes (read: Windows) to disable the ACPI brightness controls for the internal panel.
TEST=tested with rest of patch train
Change-Id: If04d1df102c50b561911d7f842ad37b2dd3c6857 Signed-off-by: Matt DeVillier matt.devillier@gmail.com --- M src/soc/intel/common/block/graphics/graphics.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/80060/1
diff --git a/src/soc/intel/common/block/graphics/graphics.c b/src/soc/intel/common/block/graphics/graphics.c index 9b780da..875fc24 100644 --- a/src/soc/intel/common/block/graphics/graphics.c +++ b/src/soc/intel/common/block/graphics/graphics.c @@ -176,7 +176,7 @@ { const struct i915_gpu_controller_info *gfx = intel_igd_get_controller_info(device);
- if (gfx) + if (gfx && !CONFIG(DRIVERS_GFX_GENERIC)) drivers_intel_gma_displays_ssdt_generate(gfx); }