soc/intel/common/block/lpc: Move southbridge_write_acpi_tables declaration

Move the southbridge_write_acpi_tables declaration from acpi.h to common
lpc_lib.h, as common LPC is always the caller. This removes a duplicate
declaration since all soc/intel devices use common LPC, but not all use
common ACPI. The southbridge_write_acpi_tables function is defined in acpi.c
with the other acpi functions.

Note that this would have the reverse problem if there is ever a non-common
LPC device.

Change-Id: I0590a028b11f34e423d8f0007e0653037b0849a0
Signed-off-by: Marc Jones <marcjones@sysproconsulting.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48251
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Jay Talbott <JayTalbott@sysproconsulting.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/soc/intel/common/block/acpi/acpi.c

@@ -13,6 +13,7 @@
 #include <cpu/intel/common/common.h>
 #include <cpu/x86/smm.h>
 #include <intelblocks/acpi.h>
+#include <intelblocks/lpc_lib.h>
 #include <intelblocks/msr.h>
 #include <intelblocks/pmclib.h>
 #include <intelblocks/uart.h>

src/soc/intel/common/block/include/intelblocks/acpi.h

@@ -23,14 +23,6 @@ uint32_t soc_read_sci_irq_select(void);
 /* Write the scis from soc specific register. */
 void soc_write_sci_irq_select(uint32_t scis);
 
-/*
- * Calls acpi_write_hpet which creates and fills HPET table and
- * adds it to the RSDT (and XSDT) structure.
- */
-unsigned long southbridge_write_acpi_tables(const struct device *device,
-					    unsigned long current,
-					    struct acpi_rsdp *rsdp);
-
 /*
  * get_cstate_map returns a table of processor specific acpi_cstate_t entries
  * and number of entries in the table

src/soc/intel/common/block/include/intelblocks/lpc_lib.h

@@ -115,5 +115,12 @@ void pch_pirq_init(void);
  * 2. Disable NMI sources
  */
 void pch_misc_init(void);
+/*
+ * Calls acpi_write_hpet which creates and fills HPET table and
+ * adds it to the RSDT (and XSDT) structure.
+ */
+unsigned long southbridge_write_acpi_tables(const struct device *device,
+					    unsigned long current,
+					    struct acpi_rsdp *rsdp);
 
 #endif /* _SOC_COMMON_BLOCK_LPC_LIB_H_ */

src/soc/intel/common/block/lpc/lpc.c

@@ -4,7 +4,6 @@
 #include <device/device.h>
 #include <device/pci.h>
 #include <device/pci_ids.h>
-#include <intelblocks/acpi.h>
 #include <intelblocks/lpc_lib.h>
 #include <soc/pm.h>
 

src/soc/intel/skylake/include/soc/acpi.h

@@ -13,8 +13,6 @@
 #define PSS_LATENCY_BUSMASTER		10
 
 unsigned long acpi_madt_irq_overrides(unsigned long current);
-unsigned long southbridge_write_acpi_tables(const struct device *device,
-	unsigned long current, struct acpi_rsdp *rsdp);
 unsigned long northbridge_write_acpi_tables(const struct device *,
 	unsigned long current, struct acpi_rsdp *);
 

src/soc/intel/xeon_sp/acpi.c

@@ -7,6 +7,7 @@
 #include <device/pci.h>
 #include <cbmem.h>
 #include <cpu/x86/smm.h>
+#include <intelblocks/lpc_lib.h>
 #include <soc/acpi.h>
 #include <soc/cpu.h>
 #include <soc/intel/common/acpi.h>

src/soc/intel/xeon_sp/include/soc/acpi.h

@@ -19,8 +19,6 @@ typedef struct {
 
 unsigned long northbridge_write_acpi_tables(const struct device *device,
 	unsigned long current, struct acpi_rsdp *rsdp);
-unsigned long southbridge_write_acpi_tables(const struct device *device,
-	unsigned long current, struct acpi_rsdp *rsdp);
 uint32_t soc_read_sci_irq_select(void);
 int soc_madt_sci_irq_polarity(int sci);
 void soc_power_states_generation(int core, int cores_per_package);