Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39638 )
Change subject: drivers/intel/gma: hook up GMA for Skylake+ ......................................................................
Patch Set 12:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39638/12/src/soc/intel/skylake/acpi... File src/soc/intel/skylake/acpi/pch.asl:
https://review.coreboot.org/c/coreboot/+/39638/12/src/soc/intel/skylake/acpi... PS12, Line 32: /* IGD 0:2.0 */ Shouldn't this be included in the systemagent.asl? That is where the graphics resides. The pch.asl contains the southbridge items.