[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/icelake: Fix AG3E programming in PMC

[Lijian Zhao (Code Review)]

Lijian Zhao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30947

Change subject: soc/intel/icelake: Fix AG3E programming in PMC
......................................................................

soc/intel/icelake: Fix AG3E programming in PMC

According to EDS #571034 4.3.2, GEN_PMCON_A stays in pmc mmio mapped
register but not pci configuration spaces, hence change the programming
method in icelake pmc driver.

Signed-off-by: Lijian Zhao 
Change-Id: I29883b50cdca99b45f5362f78cbee32beaa669f7
---
M src/soc/intel/icelake/pmc.c
1 file changed, 3 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/30947/1

diff --git a/src/soc/intel/icelake/pmc.c b/src/soc/intel/icelake/pmc.c
index 754aa50..9d1c47f 100644
--- a/src/soc/intel/icelake/pmc.c
+++ b/src/soc/intel/icelake/pmc.c
@@ -31,8 +31,9 @@
 static void pmc_set_afterg3(struct device *dev, int s5pwr)
 {
    uint8_t reg8;
+	uint8_t *pmcbase = pmc_mmio_regs();

-	reg8 = pci_read_config8(dev, GEN_PMCON_B);
+	reg8 = read8(pmcbase + GEN_PMCON_A);

switch (s5pwr) {
    case MAINBOARD_POWER_STATE_OFF:
@@ -46,7 +47,7 @@
    	break;
    }

-	pci_write_config8(dev, GEN_PMCON_B, reg8);
+	write8(pmcbase + GEN_PMCON_A, reg8);
 }

/*