summaryrefslogtreecommitdiff
path: root/src/soc/amd
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2024-07-12 14:06:49 +0200
committerFelix Held <felix-coreboot@felixheld.de>2024-07-25 22:55:15 +0000
commite9e71132a308acbe5c6d5371fbe5f7d6ffe30cf4 (patch)
treeeb872c338ef1a006600815e874c2d8b01b062e7c /src/soc/amd
parent9af1d3f85727342a0105eecdc2bbee4fca71b0b2 (diff)
soc/amd/*/root_complex: introduce and use domain_iohc_info struct
Instead of implementing the functions get_iohc_misc_smn_base and get_iohc_fabric_id in the SoC code, move those functions to the common AMD code, and implement get_iohc_info in the SoC code that returns a pointer to and the size of a SoC-specific array of domain_iohc_info structs that contains the info needed by the common code instead. This allows to iterate over the domain_iohc_info structs which will be used in a later patch to find the PSP MMIO base address in both ramstage and smm. TEST=Mandolin still boots and all non-PCI MIO resources are still reported to the resource allocator Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: Ifce3d2b540d14ba3cba36f7cbf248fb7c63483fe Reviewed-on: https://review.coreboot.org/c/coreboot/+/83443 Reviewed-by: Paul Menzel <paulepanter@mailbox.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com> Reviewed-by: Varshit Pandya <pandyavarshit@gmail.com> Reviewed-by: Martin Roth <martin.roth@amd.corp-partner.google.com>
Diffstat (limited to 'src/soc/amd')
-rw-r--r--src/soc/amd/cezanne/root_complex.c22
-rw-r--r--src/soc/amd/common/block/include/amdblocks/root_complex.h6
-rw-r--r--src/soc/amd/common/block/root_complex/Makefile.mk1
-rw-r--r--src/soc/amd/common/block/root_complex/root_complex.c43
-rw-r--r--src/soc/amd/genoa_poc/root_complex.c55
-rw-r--r--src/soc/amd/glinda/root_complex.c22
-rw-r--r--src/soc/amd/mendocino/root_complex.c22
-rw-r--r--src/soc/amd/phoenix/root_complex.c22
-rw-r--r--src/soc/amd/picasso/root_complex.c22
9 files changed, 122 insertions, 93 deletions
diff --git a/src/soc/amd/cezanne/root_complex.c b/src/soc/amd/cezanne/root_complex.c
index 72a24199e2..04fb90ca67 100644
--- a/src/soc/amd/cezanne/root_complex.c
+++ b/src/soc/amd/cezanne/root_complex.c
@@ -75,9 +75,17 @@ struct device_operations cezanne_root_complex_operations = {
.acpi_fill_ssdt = root_complex_fill_ssdt,
};
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = IOMS0_FABRIC_ID,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- return SMN_IOHC_MISC_BASE_13B1;
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -99,13 +107,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return IOMS0_FABRIC_ID;
- default:
- return -1;
- }
-}
diff --git a/src/soc/amd/common/block/include/amdblocks/root_complex.h b/src/soc/amd/common/block/include/amdblocks/root_complex.h
index 9ef5d05925..bfabf973a7 100644
--- a/src/soc/amd/common/block/include/amdblocks/root_complex.h
+++ b/src/soc/amd/common/block/include/amdblocks/root_complex.h
@@ -15,6 +15,11 @@
#define NON_PCI_RES_IDX_AUTO 0
+struct domain_iohc_info {
+ uint16_t fabric_id;
+ uint32_t misc_smn_base;
+};
+
struct non_pci_mmio_reg {
uint32_t iohc_misc_offset;
uint64_t mask;
@@ -27,6 +32,7 @@ void read_non_pci_resources(struct device *domain, unsigned long *idx);
void read_soc_memmap_resources(struct device *domain, unsigned long *idx);
uint32_t get_iohc_misc_smn_base(struct device *domain);
+const struct domain_iohc_info *get_iohc_info(size_t *count);
const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count);
signed int get_iohc_fabric_id(struct device *domain);
diff --git a/src/soc/amd/common/block/root_complex/Makefile.mk b/src/soc/amd/common/block/root_complex/Makefile.mk
index 07f3ab108e..a6aa0da502 100644
--- a/src/soc/amd/common/block/root_complex/Makefile.mk
+++ b/src/soc/amd/common/block/root_complex/Makefile.mk
@@ -3,5 +3,6 @@ ifeq ($(CONFIG_SOC_AMD_COMMON_BLOCK_ROOT_COMPLEX),y)
ramstage-y += ioapic.c
ramstage-y += non_pci_resources.c
+ramstage-y += root_complex.c
endif # CONFIG_SOC_AMD_COMMON_BLOCK_ROOT_COMPLEX
diff --git a/src/soc/amd/common/block/root_complex/root_complex.c b/src/soc/amd/common/block/root_complex/root_complex.c
new file mode 100644
index 0000000000..2bcdc1d432
--- /dev/null
+++ b/src/soc/amd/common/block/root_complex/root_complex.c
@@ -0,0 +1,43 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <amdblocks/root_complex.h>
+#include <console/console.h>
+#include <device/device.h>
+#include <types.h>
+
+static const struct domain_iohc_info *get_domain_iohc_info(struct device *domain)
+{
+ const unsigned int domain_id = domain->path.domain.domain;
+ const struct domain_iohc_info *iohc;
+ size_t iohc_count;
+
+ iohc = get_iohc_info(&iohc_count);
+
+ if (domain_id < iohc_count) {
+ return &iohc[domain_id];
+ } else {
+ printk(BIOS_ERR, "Invalid domain 0x%x with no corresponding IOHC device.\n",
+ domain_id);
+ return NULL;
+ }
+}
+
+uint32_t get_iohc_misc_smn_base(struct device *domain)
+{
+ const struct domain_iohc_info *iohc_info = get_domain_iohc_info(domain);
+
+ if (iohc_info)
+ return iohc_info->misc_smn_base;
+ else
+ return 0;
+}
+
+signed int get_iohc_fabric_id(struct device *domain)
+{
+ const struct domain_iohc_info *iohc_info = get_domain_iohc_info(domain);
+
+ if (iohc_info)
+ return iohc_info->fabric_id;
+ else
+ return -1;
+}
diff --git a/src/soc/amd/genoa_poc/root_complex.c b/src/soc/amd/genoa_poc/root_complex.c
index 849bf77a78..3171febe9f 100644
--- a/src/soc/amd/genoa_poc/root_complex.c
+++ b/src/soc/amd/genoa_poc/root_complex.c
@@ -2,26 +2,31 @@
#include <amdblocks/ioapic.h>
#include <amdblocks/root_complex.h>
-#include <console/console.h>
-#include <device/device.h>
#include <types.h>
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = 0x22,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13C1,
+ },
+ [1] = {
+ .fabric_id = 0x23,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+ [2] = {
+ .fabric_id = 0x21,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13E1,
+ },
+ [3] = {
+ .fabric_id = 0x20,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13D1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- switch (domain->path.domain.domain) {
- case 0:
- return SMN_IOHC_MISC_BASE_13C1;
- case 1:
- return SMN_IOHC_MISC_BASE_13B1;
- case 2:
- return SMN_IOHC_MISC_BASE_13E1;
- case 3:
- return SMN_IOHC_MISC_BASE_13D1;
- default:
- printk(BIOS_ERR, "Invalid domain 0x%x with no corresponding IOHC device.\n",
- domain->path.domain.domain);
- return 0;
- }
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -44,19 +49,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return 0x22;
- case 1:
- return 0x23;
- case 2:
- return 0x21;
- case 3:
- return 0x20;
- default:
- return -1;
- }
-}
diff --git a/src/soc/amd/glinda/root_complex.c b/src/soc/amd/glinda/root_complex.c
index d51fef2deb..776fc3279e 100644
--- a/src/soc/amd/glinda/root_complex.c
+++ b/src/soc/amd/glinda/root_complex.c
@@ -105,9 +105,17 @@ struct device_operations glinda_root_complex_operations = {
.acpi_fill_ssdt = root_complex_fill_ssdt,
};
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = IOMS0_FABRIC_ID,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- return SMN_IOHC_MISC_BASE_13B1;
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -132,13 +140,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return IOMS0_FABRIC_ID;
- default:
- return -1;
- }
-}
diff --git a/src/soc/amd/mendocino/root_complex.c b/src/soc/amd/mendocino/root_complex.c
index 217983331f..a7cbe5131a 100644
--- a/src/soc/amd/mendocino/root_complex.c
+++ b/src/soc/amd/mendocino/root_complex.c
@@ -266,9 +266,17 @@ struct device_operations mendocino_root_complex_operations = {
.acpi_fill_ssdt = root_complex_fill_ssdt,
};
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = IOMS0_FABRIC_ID,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- return SMN_IOHC_MISC_BASE_13B1;
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -290,13 +298,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return IOMS0_FABRIC_ID;
- default:
- return -1;
- }
-}
diff --git a/src/soc/amd/phoenix/root_complex.c b/src/soc/amd/phoenix/root_complex.c
index 918b7bd9bd..7fed3bdb31 100644
--- a/src/soc/amd/phoenix/root_complex.c
+++ b/src/soc/amd/phoenix/root_complex.c
@@ -105,9 +105,17 @@ struct device_operations phoenix_root_complex_operations = {
.acpi_fill_ssdt = root_complex_fill_ssdt,
};
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = IOMS0_FABRIC_ID,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- return SMN_IOHC_MISC_BASE_13B1;
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -132,13 +140,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return IOMS0_FABRIC_ID;
- default:
- return -1;
- }
-}
diff --git a/src/soc/amd/picasso/root_complex.c b/src/soc/amd/picasso/root_complex.c
index dd39cb284b..a3bd729ce0 100644
--- a/src/soc/amd/picasso/root_complex.c
+++ b/src/soc/amd/picasso/root_complex.c
@@ -84,9 +84,17 @@ struct device_operations picasso_root_complex_operations = {
.acpi_fill_ssdt = root_complex_fill_ssdt,
};
-uint32_t get_iohc_misc_smn_base(struct device *domain)
+static const struct domain_iohc_info iohc_info[] = {
+ [0] = {
+ .fabric_id = IOMS0_FABRIC_ID,
+ .misc_smn_base = SMN_IOHC_MISC_BASE_13B1,
+ },
+};
+
+const struct domain_iohc_info *get_iohc_info(size_t *count)
{
- return SMN_IOHC_MISC_BASE_13B1;
+ *count = ARRAY_SIZE(iohc_info);
+ return iohc_info;
}
static const struct non_pci_mmio_reg non_pci_mmio[] = {
@@ -107,13 +115,3 @@ const struct non_pci_mmio_reg *get_iohc_non_pci_mmio_regs(size_t *count)
*count = ARRAY_SIZE(non_pci_mmio);
return non_pci_mmio;
}
-
-signed int get_iohc_fabric_id(struct device *domain)
-{
- switch (domain->path.domain.domain) {
- case 0:
- return IOMS0_FABRIC_ID;
- default:
- return -1;
- }
-}