[S] Change in coreboot[main]: nb/amd/pi/00730F01: assign northbridge ops in chipset devicetree

[Felix Held (Code Review)]

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/79104?usp=email )

Change subject: nb/amd/pi/00730F01: assign northbridge ops in chipset devicetree
......................................................................

nb/amd/pi/00730F01: assign northbridge ops in chipset devicetree

Since the northbridge is always function 0 of device 0 on bus 0, the
device operations can be statically assigned in the devicetree and
there's no need to bind the northbridge device operations to the PCI
device during runtime via a list of PCI IDs.

TEST=PC Engines APU2 still boots and doesn't show any new problems

Signed-off-by: Felix Held 
Change-Id: Ia7faaa468ff77e05c378c5555622c3584cfe3f81
---
M src/northbridge/amd/pi/00730F01/chipset.cb
M src/northbridge/amd/pi/00730F01/northbridge.c
2 files changed, 4 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/79104/1

diff --git a/src/northbridge/amd/pi/00730F01/chipset.cb b/src/northbridge/amd/pi/00730F01/chipset.cb
index bc79426..6e065c2 100644
--- a/src/northbridge/amd/pi/00730F01/chipset.cb
+++ b/src/northbridge/amd/pi/00730F01/chipset.cb
@@ -7,7 +7,9 @@

device domain 0 on
    	ops amd_fam16_mod30_pci_domain_ops
-		device pci 0.0 alias gnb			on  end
+		device pci 0.0 alias gnb			on
+			ops amd_pi_northbridge_ops
+		end
    	device pci 0.2 alias iommu			off end
    	device pci 1.0 alias gfx			off end
    	device pci 1.1 alias gfx_hda			off end
diff --git a/src/northbridge/amd/pi/00730F01/northbridge.c b/src/northbridge/amd/pi/00730F01/northbridge.c
index be4e75a..9fa9f2d 100644
--- a/src/northbridge/amd/pi/00730F01/northbridge.c
+++ b/src/northbridge/amd/pi/00730F01/northbridge.c
@@ -633,7 +633,7 @@
    return current;
 }

-static struct device_operations northbridge_operations = {
+struct device_operations amd_pi_northbridge_ops = {
    .read_resources	  = nb_read_resources,
    .set_resources	  = nb_set_resources,
    .enable_resources = pci_dev_enable_resources,
@@ -643,12 +643,6 @@
    .write_acpi_tables = agesa_write_acpi_tables,
 };

-static const struct pci_driver family16_northbridge __pci_driver = {
-	.ops	= &northbridge_operations,
-	.vendor = PCI_VID_AMD,
-	.device = PCI_DID_AMD_16H_MODEL_303F_NB_HT,
-};
-
 static void fam16_finalize(void *chip_info)
 {
    struct device *dev;