Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/49912 )
Change subject: soc/intel/xeon_sp/skx: Add soc_acpi_name ......................................................................
soc/intel/xeon_sp/skx: Add soc_acpi_name
Add the soc_scpi_name to the soc pci_ops. This is used by ACPI table generation and required by the intel common XHCI device.
Change-Id: Idc09d53f14dfb1e42f904dfd4e87e8c09e155135 Signed-off-by: Marc Jones marcjones@sysproconsulting.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/49912 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Jonathan Zhang jonzhang@fb.com Reviewed-by: Jay Talbott JayTalbott@sysproconsulting.com --- M src/soc/intel/xeon_sp/skx/chip.c 1 file changed, 13 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Jay Talbott: Looks good to me, but someone else must approve Jonathan Zhang: Looks good to me, approved
diff --git a/src/soc/intel/xeon_sp/skx/chip.c b/src/soc/intel/xeon_sp/skx/chip.c index 8b08326..4d50a25 100644 --- a/src/soc/intel/xeon_sp/skx/chip.c +++ b/src/soc/intel/xeon_sp/skx/chip.c @@ -3,6 +3,7 @@ #include <cbfs.h> #include <console/console.h> #include <device/pci.h> +#include <intelblocks/acpi.h> #include <intelblocks/gpio.h> #include <soc/acpi.h> #include <soc/chip_common.h> @@ -11,12 +12,24 @@ #include <soc/soc_util.h> #include <soc/util.h>
+#if CONFIG(HAVE_ACPI_TABLES) +const char *soc_acpi_name(const struct device *dev) +{ + if (dev->path.type == DEVICE_PATH_DOMAIN) + return "PC00"; + return NULL; +} +#endif + static struct device_operations pci_domain_ops = { .read_resources = &pci_domain_read_resources, .set_resources = &xeonsp_pci_domain_set_resources, .scan_bus = &xeonsp_pci_domain_scan_bus, #if CONFIG(HAVE_ACPI_TABLES) .write_acpi_tables = &northbridge_write_acpi_tables, + #if CONFIG(HAVE_ACPI_TABLES) + .acpi_name = soc_acpi_name +#endif #endif };