[coreboot-gerrit] Change in coreboot[master]: nb/intel/ironlake: Simplify GMA SSDT generator

[Matt DeVillier (Code Review)]

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39950 )

Change subject: nb/intel/ironlake: Simplify GMA SSDT generator
......................................................................

nb/intel/ironlake: Simplify GMA SSDT generator

Simplify generation of GMA SSDT, using updated naming convention.
If acpi_fill_ssdt_generator is being invoked, then we know the
IGD device is present and enabled, so we can skip those checks.
Only check we need is if a given board has supplied the i915
controller info to populate the SSDT.

Change-Id: I1b6d57c091441aa7431061b1f16135d54cc97b47
Signed-off-by: Matt DeVillier 
---
M src/northbridge/intel/ironlake/gma.c
1 file changed, 6 insertions(+), 18 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/39950/1

diff --git a/src/northbridge/intel/ironlake/gma.c b/src/northbridge/intel/ironlake/gma.c
index cba25aa..ef9679f 100644
--- a/src/northbridge/intel/ironlake/gma.c
+++ b/src/northbridge/intel/ironlake/gma.c
@@ -214,25 +214,13 @@
    res->size = (resource_t) 0x10000000;
 }

-const struct i915_gpu_controller_info *
-intel_gma_get_controller_info(void)
+static void gma_generate_ssdt(struct device *device)
 {
-	struct device *dev = pcidev_on_root(0x2, 0);
-	if (!dev) {
-		return NULL;
-	}
-	struct northbridge_intel_ironlake_config *chip = dev->chip_info;
-	return &chip->gfx;
-}
+	const struct northbridge_intel_ironlake_config *chip = device->chip_info;
+	const struct i915_gpu_controller_info *gfx = &chip->gfx;

-static void gma_ssdt(struct device *device)
-{
-	const struct i915_gpu_controller_info *gfx = intel_gma_get_controller_info();
-	if (!gfx) {
-		return;
-	}
-
-	drivers_intel_gma_displays_ssdt_generate(gfx);
+	if (gfx)
+		drivers_intel_gma_displays_ssdt_generate(gfx);
 }

static unsigned long
@@ -269,7 +257,7 @@
    .read_resources = gma_read_resources,
    .set_resources = pci_dev_set_resources,
    .enable_resources = pci_dev_enable_resources,
-	.acpi_fill_ssdt_generator = gma_ssdt,
+	.acpi_fill_ssdt_generator = gma_generate_ssdt,
    .init = gma_func0_init,
    .scan_bus = 0,
    .enable = 0,