[coreboot-gerrit] Change in coreboot[master]: soc/amd/{common,picasso}: Move GFX device from static ASL to SSDT

[Furquan Shaikh (Code Review)]

Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/40677 )

Change subject: soc/amd/{common,picasso}: Move GFX device from static ASL to SSDT
......................................................................

soc/amd/{common,picasso}: Move GFX device from static ASL to SSDT

This change:
1. Adds PCI device for graphics controller in ACPI SSDT tables using
acpi_device_write_pci_dev().
2. Gets rid of IGFX device from picasso acpi/northbridge.asl.

This makes it easier to ensure that we don't accidentally
make the DSDT and SSDT entries inconsistent w.r.t. ACPI name and
scope.

BUG=b:153858769

Change-Id: I3a967cdc43b74f786e645d3fb666506070851a99
Signed-off-by: Furquan Shaikh 
---
M src/soc/amd/common/block/graphics/graphics.c
M src/soc/amd/picasso/acpi/northbridge.asl
2 files changed, 7 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/40677/1

diff --git a/src/soc/amd/common/block/graphics/graphics.c b/src/soc/amd/common/block/graphics/graphics.c
index 8e14aa7..8ad0d7a 100644
--- a/src/soc/amd/common/block/graphics/graphics.c
+++ b/src/soc/amd/common/block/graphics/graphics.c
@@ -5,6 +5,12 @@
 #include <device/pci.h>
 #include <device/pci_ids.h>

+static void graphics_fill_ssdt(struct device *dev)
+{
+	acpi_device_write_pci_dev(dev);
+	pci_rom_ssdt(dev);
+}
+
 static const struct *graphics_acpi_name(const struct device *dev)
 {
    return "IGFX";
@@ -17,7 +23,7 @@
    .init				= pci_dev_init,
    .ops_pci			= &pci_dev_ops_pci,
    .write_acpi_tables		= pci_rom_write_acpi_tables,
-	.acpi_fill_ssdt_generator	= pci_rom_ssdt,
+	.acpi_fill_ssdt_generator	= graphics_fill_ssdt,
    .acpi_name			= graphics_acpi_name,
 };

diff --git a/src/soc/amd/picasso/acpi/northbridge.asl b/src/soc/amd/picasso/acpi/northbridge.asl
index c807601..6b6bd7c 100644
--- a/src/soc/amd/picasso/acpi/northbridge.asl
+++ b/src/soc/amd/picasso/acpi/northbridge.asl
@@ -32,11 +32,6 @@
    Name(_ADR, 0x00000000)
 } /* end AMRT */

-/* Internal Graphics */
-Device(IGFX) {
-	Name(_ADR, 0x00010000)
-}
-
 /* Gpp 0 */
 Device(PBR4) {
    Name(_ADR, 0x00020001)