diff options
| author | Werner Zeh <werner.zeh@siemens.com> | 2018-04-05 07:36:40 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-04-06 07:02:40 +0000 |
| commit | 3caf34167c0d05f53bf1b466117ff234f5c7e14e (patch) | |
| tree | 4c3bbee980b1495af1e1883bf152d82bef4f0a03 | |
| parent | 438ca724609e24c627c1f6f243634bbc5bf6d6d9 (diff) | |
fsp_broadwell_de: Provide valid ACPI path names for domain and LPC
Provide ACPI path names for PCI domain and LPC device so that generated
ACPI tables have valid device paths.
Change-Id: I5a97e45ef50ec5ee9d64c5d2834968a02455cf72
Signed-off-by: Werner Zeh <werner.zeh@siemens.com>
Reviewed-on: https://review.coreboot.org/25534
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rw-r--r-- | src/soc/intel/fsp_broadwell_de/chip.c | 12 | ||||
| -rw-r--r-- | src/soc/intel/fsp_broadwell_de/southcluster.c | 13 |
2 files changed, 25 insertions, 0 deletions
diff --git a/src/soc/intel/fsp_broadwell_de/chip.c b/src/soc/intel/fsp_broadwell_de/chip.c index b8db395350..118fb993fb 100644 --- a/src/soc/intel/fsp_broadwell_de/chip.c +++ b/src/soc/intel/fsp_broadwell_de/chip.c @@ -30,6 +30,15 @@ static void pci_domain_set_resources(device_t dev) assign_resources(dev->link_list); } +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) +static const char *domain_acpi_name(const struct device *dev) +{ + if (dev->path.type == DEVICE_PATH_DOMAIN) + return "PCI0"; + return NULL; +} +#endif + static struct device_operations pci_domain_ops = { .read_resources = pci_domain_read_resources, .set_resources = pci_domain_set_resources, @@ -37,6 +46,9 @@ static struct device_operations pci_domain_ops = { .init = NULL, .scan_bus = pci_domain_scan_bus, .ops_pci_bus = pci_bus_default_ops, +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) + .acpi_name = domain_acpi_name +#endif }; static struct device_operations cpu_bus_ops = { diff --git a/src/soc/intel/fsp_broadwell_de/southcluster.c b/src/soc/intel/fsp_broadwell_de/southcluster.c index 3bf5429e39..8ad27afc72 100644 --- a/src/soc/intel/fsp_broadwell_de/southcluster.c +++ b/src/soc/intel/fsp_broadwell_de/southcluster.c @@ -263,6 +263,16 @@ void southcluster_enable_dev(device_t dev) } } +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) +static const char *lpc_acpi_name(const struct device *dev) +{ + if (dev->path.pci.devfn == LPC_DEV_FUNC) + return "LPC0"; + else + return NULL; +} +#endif + static struct device_operations device_ops = { .read_resources = sc_read_resources, .set_resources = pci_dev_set_resources, @@ -271,6 +281,9 @@ static struct device_operations device_ops = { .enable = southcluster_enable_dev, .scan_bus = scan_lpc_bus, .ops_pci = &soc_pci_ops, +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) + .acpi_name = lpc_acpi_name, +#endif }; static const struct pci_driver southcluster __pci_driver = { |