diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2024-07-24 23:58:07 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2024-07-25 22:54:43 +0000 |
| commit | 9af1d3f85727342a0105eecdc2bbee4fca71b0b2 (patch) | |
| tree | c7716cd6da075a30afef2d145c269daac2242d50 /src | |
| parent | a17d22e51aa5bd133fb53e78ab75c171113e9383 (diff) | |
acpi,soc: use is_domain0 function
No need to open-code this when we have a function for this.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Iae570ba750cb29456436349b4263808e2e410e2e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/83643
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Shuo Liu <shuo.liu@intel.com>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/acpi/acpigen_pci_root_resource_producer.c | 2 | ||||
| -rw-r--r-- | src/soc/amd/common/block/data_fabric/domain.c | 2 | ||||
| -rw-r--r-- | src/soc/cavium/cn81xx/soc.c | 3 | ||||
| -rw-r--r-- | src/soc/intel/xeon_sp/uncore_acpi.c | 2 |
4 files changed, 4 insertions, 5 deletions
diff --git a/src/acpi/acpigen_pci_root_resource_producer.c b/src/acpi/acpigen_pci_root_resource_producer.c index 78e1a89a3b..ce57b9cc79 100644 --- a/src/acpi/acpigen_pci_root_resource_producer.c +++ b/src/acpi/acpigen_pci_root_resource_producer.c @@ -60,7 +60,7 @@ void pci_domain_fill_ssdt(const struct device *domain) acpigen_resource_producer_bus_number(domain->downstream->secondary, domain->downstream->max_subordinate); - if (domain->path.domain.domain == 0) { + if (is_domain0(domain)) { /* ACPI 6.4.2.5 I/O Port Descriptor */ acpigen_write_io16(PCI_IO_CONFIG_INDEX, PCI_IO_CONFIG_LAST_PORT, 1, PCI_IO_CONFIG_PORT_COUNT, 1); diff --git a/src/soc/amd/common/block/data_fabric/domain.c b/src/soc/amd/common/block/data_fabric/domain.c index b056d607c7..c419fe02ce 100644 --- a/src/soc/amd/common/block/data_fabric/domain.c +++ b/src/soc/amd/common/block/data_fabric/domain.c @@ -206,7 +206,7 @@ void amd_pci_domain_read_resources(struct device *domain) read_non_pci_resources(domain, &idx); /* Only add the SoC's DRAM memory map and fixed resources once */ - if (domain->path.domain.domain == 0) { + if (is_domain0(domain)) { add_pci_cfg_resources(domain, &idx); read_soc_memmap_resources(domain, &idx); diff --git a/src/soc/cavium/cn81xx/soc.c b/src/soc/cavium/cn81xx/soc.c index 81c9a648e5..e0caea6cd7 100644 --- a/src/soc/cavium/cn81xx/soc.c +++ b/src/soc/cavium/cn81xx/soc.c @@ -386,8 +386,7 @@ static struct device_operations soc_ops = { static void enable_soc_dev(struct device *dev) { - if (dev->path.type == DEVICE_PATH_DOMAIN && - dev->path.domain.domain == 0) { + if (is_domain0(dev)) { dev->ops = &pci_domain_ops_ecam0; } else if (dev->path.type == DEVICE_PATH_CPU_CLUSTER) { dev->ops = &soc_ops; diff --git a/src/soc/intel/xeon_sp/uncore_acpi.c b/src/soc/intel/xeon_sp/uncore_acpi.c index 22f8b6fed0..0b2b9f4107 100644 --- a/src/soc/intel/xeon_sp/uncore_acpi.c +++ b/src/soc/intel/xeon_sp/uncore_acpi.c @@ -560,7 +560,7 @@ unsigned long northbridge_write_acpi_tables(const struct device *device, unsigne struct acpi_rsdp *rsdp) { /* Only write uncore ACPI tables for domain0 */ - if (device->path.domain.domain != 0) + if (!is_domain0(device)) return current; acpi_srat_t *srat; |