[coreboot-gerrit] Change in ...coreboot[master]: soc/amd/stoneyridge: Move acpi_fill_mcfg to northbridge

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32550 )

Change subject: soc/amd/stoneyridge: Move acpi_fill_mcfg to northbridge
......................................................................

soc/amd/stoneyridge: Move acpi_fill_mcfg to northbridge

Relocate the function to the more appropriate file.

Change-Id: I92a3e8d0461ae228f6c01567db159e2458de5f6b
Signed-off-by: Marshall Dawson 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32550
Tested-by: build bot (Jenkins) 
Reviewed-by: Martin Roth 
---
M src/soc/amd/stoneyridge/lpc.c
M src/soc/amd/stoneyridge/northbridge.c
2 files changed, 12 insertions(+), 13 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Martin Roth: Looks good to me, approved

diff --git a/src/soc/amd/stoneyridge/lpc.c b/src/soc/amd/stoneyridge/lpc.c
index 87da9f1..9a8c8ef 100644
--- a/src/soc/amd/stoneyridge/lpc.c
+++ b/src/soc/amd/stoneyridge/lpc.c
@@ -25,7 +25,6 @@
 #include <pc80/mc146818rtc.h>
 #include <pc80/isa-dma.h>
 #include <arch/ioapic.h>
-#include <arch/acpi.h>
 #include <pc80/i8254.h>
 #include <pc80/i8259.h>
 #include <soc/acpi.h>
@@ -326,18 +325,6 @@
    lpc_enable_childrens_resources(dev);
 }

-unsigned long acpi_fill_mcfg(unsigned long current)
-{
-
-	current += acpi_create_mcfg_mmconfig((acpi_mcfg_mmconfig_t *)current,
-					     CONFIG_MMCONF_BASE_ADDRESS,
-					     0,
-					     0,
-					     CONFIG_MMCONF_BUS_NUMBER);
-
-	return current;
-}
-
 static struct pci_operations lops_pci = {
    .set_subsystem = pci_dev_set_subsystem,
 };
diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c
index 42963c0..5985832 100644
--- a/src/soc/amd/stoneyridge/northbridge.c
+++ b/src/soc/amd/stoneyridge/northbridge.c
@@ -176,6 +176,18 @@
    setup_ioapic((u8 *)IO_APIC2_ADDR, CONFIG_MAX_CPUS+1);
 }

+unsigned long acpi_fill_mcfg(unsigned long current)
+{
+
+	current += acpi_create_mcfg_mmconfig((acpi_mcfg_mmconfig_t *)current,
+					     CONFIG_MMCONF_BASE_ADDRESS,
+					     0,
+					     0,
+					     CONFIG_MMCONF_BUS_NUMBER);
+
+	return current;
+}
+
 static unsigned long acpi_fill_hest(acpi_hest_t *hest)
 {
    void *addr, *current;