[XS] Change in coreboot[main]: soc/intel/xeon_sp/spr: Fix IOAT resources

[Lean Sheng Tan (Code Review)]

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

(

1 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/spr: Fix IOAT resources
......................................................................

soc/intel/xeon_sp/spr: Fix IOAT resources

Do not generate empty mem32 resources for CPMx or HQMx stacks.
Switch existing arguments to make sure that base is bigger than
limit to indicate that the resource is invalid.

Change-Id: I679563e97c33c7ee35d402674972e55f521eafa8
Signed-off-by: Patrick Rudolph 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80793
Reviewed-by: Arthur Heymans 
Reviewed-by: Shuo Liu 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/xeon_sp/spr/ioat.c
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Arthur Heymans: Looks good to me, approved
  build bot (Jenkins): Verified
  Shuo Liu: Looks good to me, but someone else must approve

diff --git a/src/soc/intel/xeon_sp/spr/ioat.c b/src/soc/intel/xeon_sp/spr/ioat.c
index 5528efa..bde7eba 100644
--- a/src/soc/intel/xeon_sp/spr/ioat.c
+++ b/src/soc/intel/xeon_sp/spr/ioat.c
@@ -101,7 +101,7 @@
    mem64_limit = mem64_base + CPM_MMIO_SIZE - 1;
    bus_base = sr->BusBase + CPM_BUS_OFFSET;
    bus_limit = bus_base + CPM_RESERVED_BUS;
-	create_ioat_domain(path, bus, bus_base, bus_limit, 0, -1, mem64_base, mem64_limit,
+	create_ioat_domain(path, bus, bus_base, bus_limit, -1, 0, mem64_base, mem64_limit,
    		   DOMAIN_TYPE_CPM0);

/* HQM0 */
@@ -109,7 +109,7 @@
    mem64_limit = mem64_base + HQM_MMIO_SIZE - 1;
    bus_base = sr->BusBase + HQM_BUS_OFFSET;
    bus_limit = bus_base + HQM_RESERVED_BUS;
-	create_ioat_domain(path, bus, bus_base, bus_limit, 0, -1, mem64_base, mem64_limit,
+	create_ioat_domain(path, bus, bus_base, bus_limit, -1, 0, mem64_base, mem64_limit,
    		   DOMAIN_TYPE_HQM0);

/* CPM1 (optional) */
@@ -118,7 +118,7 @@
    bus_base = sr->BusBase + CPM1_BUS_OFFSET;
    bus_limit = bus_base + CPM_RESERVED_BUS;
    if (bus_limit <= sr->BusLimit)
-		create_ioat_domain(path, bus, bus_base, bus_limit, 0, -1, mem64_base, mem64_limit,
+		create_ioat_domain(path, bus, bus_base, bus_limit, -1, 0, mem64_base, mem64_limit,
    			   DOMAIN_TYPE_CPM1);

/* HQM1 (optional) */
@@ -127,7 +127,7 @@
    bus_base = sr->BusBase + HQM1_BUS_OFFSET;
    bus_limit = bus_base + HQM_RESERVED_BUS;
    if (bus_limit <= sr->BusLimit)
-		create_ioat_domain(path, bus, bus_base, bus_limit, 0, -1, mem64_base, mem64_limit,
+		create_ioat_domain(path, bus, bus_base, bus_limit, -1, 0, mem64_base, mem64_limit,
    			   DOMAIN_TYPE_HQM1);

/* DINO */