[S] Change in coreboot[master]: soc/intel: Use `PWRMBASE` over static `Index 0` for PMC

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/69227 )

Change subject: soc/intel: Use `PWRMBASE` over static `Index 0` for PMC
......................................................................

soc/intel: Use `PWRMBASE` over static `Index 0` for PMC

This patch replaces static index 0 for PMC read resources with PCI
configuration offset 0x10 (PWRMBASE).

TEST=Able to build and boot Google, Rex to OS.

Without this change:
[SPEW ]     PCI: 00:1f.2 resource base fe000000 size 10000 align 0 gran
0 limit 0 flags f0000200 index 0

With this change:
[SPEW ]     PCI: 00:1f.2 resource base fe000000 size 10000 align 0 gran
0 limit 0 flags f0000200 index 10

Signed-off-by: Subrata Banik 
Change-Id: Iee2523876a8045e70effd5824afc327d1113038b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69227
Tested-by: build bot (Jenkins) 
Reviewed-by: Werner Zeh 
---
M src/soc/intel/alderlake/pmc.c
M src/soc/intel/cannonlake/pmc.c
M src/soc/intel/elkhartlake/pmc.c
M src/soc/intel/jasperlake/pmc.c
M src/soc/intel/meteorlake/pmc.c
M src/soc/intel/tigerlake/pmc.c
6 files changed, 38 insertions(+), 6 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Werner Zeh: Looks good to me, approved

diff --git a/src/soc/intel/alderlake/pmc.c b/src/soc/intel/alderlake/pmc.c
index 87cc54a..c28ee2d 100644
--- a/src/soc/intel/alderlake/pmc.c
+++ b/src/soc/intel/alderlake/pmc.c
@@ -85,7 +85,8 @@
    struct resource *res;

/* Add the fixed MMIO resource */
-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

/* Add the fixed I/O resource */
    res = new_resource(dev, 1);
diff --git a/src/soc/intel/cannonlake/pmc.c b/src/soc/intel/cannonlake/pmc.c
index f0a32b93..b85a12a 100644
--- a/src/soc/intel/cannonlake/pmc.c
+++ b/src/soc/intel/cannonlake/pmc.c
@@ -74,7 +74,8 @@
    struct resource *res;

/* Add the fixed MMIO resource */
-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

/* Add the fixed I/O resource */
    res = new_resource(dev, 1);
diff --git a/src/soc/intel/elkhartlake/pmc.c b/src/soc/intel/elkhartlake/pmc.c
index e556ac8..27d00d7 100644
--- a/src/soc/intel/elkhartlake/pmc.c
+++ b/src/soc/intel/elkhartlake/pmc.c
@@ -71,7 +71,8 @@
 {
    struct resource *res;

-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

res = new_resource(dev, 1);
    res->base = (resource_t)ACPI_BASE_ADDRESS;
diff --git a/src/soc/intel/jasperlake/pmc.c b/src/soc/intel/jasperlake/pmc.c
index 4a48345..8f79f0c 100644
--- a/src/soc/intel/jasperlake/pmc.c
+++ b/src/soc/intel/jasperlake/pmc.c
@@ -71,7 +71,8 @@
 {
    struct resource *res;

-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

res = new_resource(dev, 1);
    res->base = (resource_t)ACPI_BASE_ADDRESS;
diff --git a/src/soc/intel/meteorlake/pmc.c b/src/soc/intel/meteorlake/pmc.c
index 0ab15c4..dbfc4be 100644
--- a/src/soc/intel/meteorlake/pmc.c
+++ b/src/soc/intel/meteorlake/pmc.c
@@ -80,7 +80,8 @@
    struct resource *res;

/* Add the fixed MMIO resource */
-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

/* Add the fixed I/O resource */
    res = new_resource(dev, 1);
diff --git a/src/soc/intel/tigerlake/pmc.c b/src/soc/intel/tigerlake/pmc.c
index 31a9cc4..5ad8a89 100644
--- a/src/soc/intel/tigerlake/pmc.c
+++ b/src/soc/intel/tigerlake/pmc.c
@@ -84,7 +84,8 @@
    struct resource *res;

/* Add the fixed MMIO resource */
-	mmio_resource_kb(dev, 0, PCH_PWRM_BASE_ADDRESS / KiB, PCH_PWRM_BASE_SIZE / KiB);
+	mmio_resource_kb(dev, PWRMBASE, PCH_PWRM_BASE_ADDRESS / KiB,
+			PCH_PWRM_BASE_SIZE / KiB);

/* Add the fixed I/O resource */
    res = new_resource(dev, 1);