summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJonathan Zhang <jonzhang@meta.com>2023-01-25 11:28:49 -0800
committerLean Sheng Tan <sheng.tan@9elements.com>2023-03-25 16:42:06 +0000
commit532e8c059e3209530ebf56545df3f1f7a82c6ccf (patch)
tree900abd03b02a097a1e9fc73f5b72605a8934ace0 /src
parent5c1964058f5a9e1f144cbb0a004181f1d046afe4 (diff)
soc/intel/xeon_sp/chip_common.c: Probe all buses in attach_iio_stacks()
For some Xeon-SP (such as SPR-SP), more buses should be probed. Signed-off-by: Jonathan Zhang <jonzhang@meta.com> Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com> Change-Id: Ica3c61493a0ff6c699b500f30788b2cf5a06c250 Reviewed-on: https://review.coreboot.org/c/coreboot/+/71965 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Jonathan Zhang <jon.zhixiong.zhang@gmail.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/intel/xeon_sp/chip_common.c72
1 files changed, 45 insertions, 27 deletions
diff --git a/src/soc/intel/xeon_sp/chip_common.c b/src/soc/intel/xeon_sp/chip_common.c
index a5062fcf80..f6805b2032 100644
--- a/src/soc/intel/xeon_sp/chip_common.c
+++ b/src/soc/intel/xeon_sp/chip_common.c
@@ -493,45 +493,63 @@ void xeonsp_pci_domain_set_resources(struct device *dev)
/* Attach IIO stack bus numbers with dummy device to PCI DOMAIN 0000 device */
void attach_iio_stacks(struct device *dev)
{
- struct bus *iiostack_bus;
struct device dummy;
struct iiostack_resource stack_info = {0};
DEV_FUNC_ENTER(dev);
get_iiostack_info(&stack_info);
+
for (int s = 0; s < stack_info.no_of_stacks; ++s) {
+ STACK_RES *sr = &stack_info.res[s];
/* only non zero bus no. needs to be enumerated */
- if (stack_info.res[s].BusBase == 0)
+ if (sr->BusBase == 0) {
+ /* Update BUS 0 BusLimit */
+ dev->link_list->max_subordinate = sr->BusLimit;
continue;
+ }
+
+ for (int b = sr->BusBase; b <= sr->BusLimit; ++b) {
+ struct bus tmp_bus;
+ memset(&tmp_bus, 0, sizeof(tmp_bus));
+ memcpy(&tmp_bus, dev->bus, sizeof(tmp_bus));
+ tmp_bus.secondary = b;
+ tmp_bus.subordinate = b;
+ tmp_bus.max_subordinate = sr->BusLimit;
+ tmp_bus.dev = NULL;
+ tmp_bus.children = NULL;
+ tmp_bus.next = NULL;
+ tmp_bus.link_num = 1;
+
+ dummy.bus = &tmp_bus;
+ dummy.path.type = DEVICE_PATH_PCI;
+ dummy.path.pci.devfn = 0;
+ uint32_t id = pci_read_config32(&dummy, PCI_VENDOR_ID);
+ if (id == 0xffffffff) {
+ printk(BIOS_DEBUG, "IIO Stack device %s not visible\n",
+ dev_path(&dummy));
+ continue;
+ }
- iiostack_bus = malloc(sizeof(struct bus));
- if (!iiostack_bus)
- die("%s: out of memory.\n", __func__);
- memset(iiostack_bus, 0, sizeof(*iiostack_bus));
- memcpy(iiostack_bus, dev->bus, sizeof(*iiostack_bus));
- iiostack_bus->secondary = stack_info.res[s].BusBase;
- iiostack_bus->subordinate = stack_info.res[s].BusBase;
- iiostack_bus->dev = NULL;
- iiostack_bus->children = NULL;
- iiostack_bus->next = NULL;
- iiostack_bus->link_num = 1;
-
- dummy.bus = iiostack_bus;
- dummy.path.type = DEVICE_PATH_PCI;
- dummy.path.pci.devfn = 0;
- uint32_t id = pci_read_config32(&dummy, PCI_VENDOR_ID);
- if (id == 0xffffffff)
- printk(BIOS_WARNING, "IIO Stack device %s not visible\n",
+ printk(BIOS_DEBUG, "%s Attaching IIO Bus %s\n", __func__,
dev_path(&dummy));
+ printk(BIOS_DEBUG, " %s attach secondary: 0x%x, subordinate: 0x%x, dev: %s\n",
+ __func__, tmp_bus.secondary,
+ tmp_bus.subordinate, dev_path(&dummy));
- if (!dev->link_list) {
- dev->link_list = iiostack_bus;
- } else {
- struct bus *nlink = dev->link_list;
- while (nlink->next)
- nlink = nlink->next;
- nlink->next = iiostack_bus;
+ struct bus *iiostack_bus = malloc(sizeof(struct bus));
+ if (iiostack_bus == NULL)
+ die("%s: out of memory.\n", __func__);
+ memcpy(iiostack_bus, &tmp_bus, sizeof(*iiostack_bus));
+
+ if (dev->link_list == NULL) {
+ dev->link_list = iiostack_bus;
+ } else {
+ struct bus *nlink = dev->link_list;
+ while (nlink->next != NULL)
+ nlink = nlink->next;
+ nlink->next = iiostack_bus;
+ }
}
}