[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/xeon_sp: Add utils to detect domain0 and stack0

[Lean Sheng Tan (Code Review)]

Lean Sheng Tan has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81199?usp=email )

(

5 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: soc/intel/xeon_sp: Add utils to detect domain0 and stack0
......................................................................

soc/intel/xeon_sp: Add utils to detect domain0 and stack0

In Xeon-SP, the domain0, which is located at stack0, usually needs
special handling due to the compatible devices on it (HEPT, IO-APIC
and legacy IOs). This patch adds util function detect whether a
give domain or stack is with such a role.

TEST=intel/archercity CRB

Change-Id: I2f26b4ac54091c24c554f17964502c364288aa40
Signed-off-by: Shuo Liu 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81199
Tested-by: build bot (Jenkins) 
Reviewed-by: Patrick Rudolph 
---
M src/soc/intel/xeon_sp/chip_common.c
M src/soc/intel/xeon_sp/include/soc/chip_common.h
2 files changed, 6 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Rudolph: Looks good to me, approved

diff --git a/src/soc/intel/xeon_sp/chip_common.c b/src/soc/intel/xeon_sp/chip_common.c
index d0e01bc..ef504d7 100644
--- a/src/soc/intel/xeon_sp/chip_common.c
+++ b/src/soc/intel/xeon_sp/chip_common.c
@@ -199,7 +199,7 @@

int index = 0;

-	if (dev->path.domain.domain == 0) {
+	if (is_domain0(dev)) {
    	/* The 0 - 0xfff IO range is not reported by the HOB but still gets decoded */
    	res = new_resource(dev, index++);
    	res->base = 0;
diff --git a/src/soc/intel/xeon_sp/include/soc/chip_common.h b/src/soc/intel/xeon_sp/include/soc/chip_common.h
index 40c7ffa..53deffa 100644
--- a/src/soc/intel/xeon_sp/include/soc/chip_common.h
+++ b/src/soc/intel/xeon_sp/include/soc/chip_common.h
@@ -77,4 +77,9 @@
 #define is_dev_on_ubox_domain(dev) is_ubox_domain(dev_get_pci_domain(dev))
 #define is_dev_on_cxl_domain(dev) is_cxl_domain(dev_get_pci_domain(dev))

+#define is_domain0(dev) (dev && dev->path.type == DEVICE_PATH_DOMAIN &&\
+		dev->path.domain.domain == 0)
+#define is_dev_on_domain0(dev) (is_domain0(dev_get_pci_domain(dev)))
+#define is_stack0(socket, stack) (socket == 0 && stack == IioStack0)
+
 #endif /* _CHIP_COMMON_H_ */