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

[Matt DeVillier (Code Review)]

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

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

nb/intel/gm45: 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: Ideddfc3d327c4421faffb6583e347cd2b094e155
Signed-off-by: Matt DeVillier 
---
M src/northbridge/intel/gm45/gma.c
1 file changed, 6 insertions(+), 18 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/39959/1

diff --git a/src/northbridge/intel/gm45/gma.c b/src/northbridge/intel/gm45/gma.c
index eed0b92..176a511 100644
--- a/src/northbridge/intel/gm45/gma.c
+++ b/src/northbridge/intel/gm45/gma.c
@@ -219,25 +219,13 @@
    intel_gma_restore_opregion();
 }

-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_gm45_config *chip = dev->chip_info;
-	return &chip->gfx;
-}
+	const struct northbridge_intel_gm45_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
@@ -279,7 +267,7 @@
    .read_resources = pci_dev_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,