Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Tim Chu.
Shuo Liu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/80087?usp=email )
Change subject: soc/intel/xeon_sp/spr: Create CXL ACPI resources only for IIO stacks connected with CXL cards ......................................................................
soc/intel/xeon_sp/spr: Create CXL ACPI resources only for IIO stacks connected with CXL cards
When an IIO stack is connected with CXL cards, its bus range will be divided by a PCI host bridge object and a CXL host bridge object, otherwise, all its range will be owned by the PCI host bridge object. Accordingly, CXL ACPI resources should be only created when the IIO stack is connected with a CXL card.
TEST=intel/archercity CRB
Change-Id: I6c1b1343991bc73d90a433d959f6618bbf59532f Signed-off-by: Shuo Liu shuo.liu@intel.com --- M src/soc/intel/xeon_sp/spr/soc_acpi.c 1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/80087/1
diff --git a/src/soc/intel/xeon_sp/spr/soc_acpi.c b/src/soc/intel/xeon_sp/spr/soc_acpi.c index 9d2df2c..cb7fe1d 100644 --- a/src/soc/intel/xeon_sp/spr/soc_acpi.c +++ b/src/soc/intel/xeon_sp/spr/soc_acpi.c @@ -376,7 +376,9 @@
if (stack <= IioStack5) { // TYPE_UBOX_IIO create_dsdt_iou_pci_resource(socket, stack, ri, stack_enabled); - create_dsdt_iou_cxl_resource(socket, stack, ri, stack_enabled); + if (is_iio_cxl_stack_res(ri)) + create_dsdt_iou_cxl_resource(socket, stack, ri, + stack_enabled); create_dsdt_stack_sta(socket, stack, ri, stack_enabled); } else if (stack >= IioStack8 && stack <= IioStack11) { // TYPE_DINO create_dsdt_dino_resource(socket, stack, ri, stack_enabled);