soc/amd/*/acpi: use common soc_acpi_write_tables prototype

Since the definition is the same for all SoCs, move it to the common
amdblock/acpi.h header. Since the Stoneyridge northbridge.c file also
includes this prototype, remove the static attribute of the function
there.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ib9aa215f2b4ba58f43fed2c751d989f1719e0a17
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80221
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/soc/amd/cezanne/chip.c

@@ -1,11 +1,11 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
+#include <amdblocks/acpi.h>
 #include <amdblocks/data_fabric.h>
 #include <amdblocks/fsp.h>
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
-#include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/pci_devs.h>
 #include <soc/southbridge.h>

src/soc/amd/cezanne/include/soc/acpi.h

@@ -3,14 +3,6 @@
 #ifndef AMD_CEZANNE_ACPI_H
 #define AMD_CEZANNE_ACPI_H
 
-#include <acpi/acpi.h>
-#include <amdblocks/acpi.h>
-#include <device/device.h>
-#include <stdint.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    acpi_rsdp_t *rsdp);
-
 #endif /* AMD_CEZANNE_ACPI_H */

src/soc/amd/common/block/include/amdblocks/acpi.h

@@ -50,6 +50,9 @@ struct chipset_power_state {
 	struct gpio_wake_state gpio_state;
 };
 
+unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
+				    acpi_rsdp_t *rsdp);
+
 unsigned long southbridge_write_acpi_tables(const struct device *device, unsigned long current,
 					    struct acpi_rsdp *rsdp);
 

src/soc/amd/genoa_poc/chip.c

@@ -1,8 +1,8 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
+#include <amdblocks/acpi.h>
 #include <device/device.h>
 #include <soc/southbridge.h>
-#include <soc/acpi.h>
 #include <soc/southbridge.h>
 #include <vendorcode/amd/opensil/genoa_poc/opensil.h>
 

src/soc/amd/genoa_poc/include/soc/acpi.h

@@ -3,12 +3,6 @@
 #ifndef AMD_GENOA_POC_ACPI_H
 #define AMD_GENOA_POC_ACPI_H
 
-#include <acpi/acpi.h>
-#include <device/device.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    struct acpi_rsdp *rsdp);
-
 #endif /* AMD_GENOA_POC_ACPI_H */

src/soc/amd/glinda/chip.c

@@ -2,12 +2,12 @@
 
 /* TODO: Update for Glinda */
 
+#include <amdblocks/acpi.h>
 #include <amdblocks/data_fabric.h>
 #include <amdblocks/fsp.h>
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
-#include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/pci_devs.h>
 #include <soc/southbridge.h>

src/soc/amd/glinda/include/soc/acpi.h

@@ -5,14 +5,6 @@
 #ifndef AMD_GLINDA_ACPI_H
 #define AMD_GLINDA_ACPI_H
 
-#include <acpi/acpi.h>
-#include <amdblocks/acpi.h>
-#include <device/device.h>
-#include <stdint.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    acpi_rsdp_t *rsdp);
-
 #endif /* AMD_GLINDA_ACPI_H */

src/soc/amd/mendocino/chip.c

@@ -1,11 +1,11 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
+#include <amdblocks/acpi.h>
 #include <amdblocks/data_fabric.h>
 #include <amdblocks/fsp.h>
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
-#include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/pci_devs.h>
 #include <soc/southbridge.h>

src/soc/amd/mendocino/include/soc/acpi.h

@@ -3,14 +3,6 @@
 #ifndef AMD_MENDOCINO_ACPI_H
 #define AMD_MENDOCINO_ACPI_H
 
-#include <acpi/acpi.h>
-#include <amdblocks/acpi.h>
-#include <device/device.h>
-#include <stdint.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    acpi_rsdp_t *rsdp);
-
 #endif /* AMD_MENDOCINO_ACPI_H */

src/soc/amd/phoenix/chip.c

@@ -2,12 +2,12 @@
 
 /* TODO: Update for Phoenix */
 
+#include <amdblocks/acpi.h>
 #include <amdblocks/data_fabric.h>
 #include <amdblocks/fsp.h>
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
-#include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/pci_devs.h>
 #include <soc/southbridge.h>

src/soc/amd/phoenix/include/soc/acpi.h

@@ -5,14 +5,6 @@
 #ifndef AMD_PHOENIX_ACPI_H
 #define AMD_PHOENIX_ACPI_H
 
-#include <acpi/acpi.h>
-#include <amdblocks/acpi.h>
-#include <device/device.h>
-#include <stdint.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    acpi_rsdp_t *rsdp);
-
 #endif /* AMD_PHOENIX_ACPI_H */

src/soc/amd/picasso/chip.c

@@ -1,12 +1,12 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
+#include <amdblocks/acpi.h>
 #include <amdblocks/data_fabric.h>
 #include <amdblocks/fsp.h>
 #include <console/console.h>
 #include <device/device.h>
 #include <device/pci.h>
 #include <drivers/i2c/designware/dw_i2c.h>
-#include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/iomap.h>
 #include <soc/pci_devs.h>

src/soc/amd/picasso/include/soc/acpi.h

@@ -3,14 +3,6 @@
 #ifndef AMD_PICASSO_ACPI_H
 #define AMD_PICASSO_ACPI_H
 
-#include <acpi/acpi.h>
-#include <amdblocks/acpi.h>
-#include <device/device.h>
-#include <stdint.h>
-
 #define ACPI_SCI_IRQ 9
 
-unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
-				    acpi_rsdp_t *rsdp);
-
 #endif /* AMD_PICASSO_ACPI_H */

src/soc/amd/stoneyridge/northbridge.c

@@ -131,9 +131,8 @@ static unsigned long acpi_fill_hest(acpi_hest_t *hest)
 	return (unsigned long)current;
 }
 
-static unsigned long soc_acpi_write_tables(const struct device *device,
-					   unsigned long current,
-					   acpi_rsdp_t *rsdp)
+unsigned long soc_acpi_write_tables(const struct device *device, unsigned long current,
+				    acpi_rsdp_t *rsdp)
 {
 	acpi_srat_t *srat;
 	acpi_slit_t *slit;