[coreboot-gerrit] [S] Change in coreboot[main]: soc/amd/phoenix/fch: only init ACPI IO ports in FSP case

[Felix Held (Code Review)]

Attention is currently required from: Fred Reitberger, Jason Glenesk, Matt DeVillier.

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/80297?usp=email )

Change subject: soc/amd/phoenix/fch: only init ACPI IO ports in FSP case
......................................................................

soc/amd/phoenix/fch: only init ACPI IO ports in FSP case

Since openSIL configures the APCI IO port addresses, don't overwrite
those by the coreboot code.

Signed-off-by: Felix Held 
Change-Id: If10e5a9f52ab313ad1afebd7f9e722994d48b0a7
---
M src/soc/amd/phoenix/fch.c
1 file changed, 6 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/80297/1

diff --git a/src/soc/amd/phoenix/fch.c b/src/soc/amd/phoenix/fch.c
index e9bc80a..1e03cda 100644
--- a/src/soc/amd/phoenix/fch.c
+++ b/src/soc/amd/phoenix/fch.c
@@ -89,10 +89,12 @@
     * ACPI tables are generated. Enable these ports indiscriminately.
     */

-	pm_write16(PM_EVT_BLK, ACPI_PM_EVT_BLK);
-	pm_write16(PM1_CNT_BLK, ACPI_PM1_CNT_BLK);
-	pm_write16(PM_TMR_BLK, ACPI_PM_TMR_BLK);
-	pm_write16(PM_GPE0_BLK, ACPI_GPE0_BLK);
+	if (CONFIG(PLATFORM_USES_FSP2_0)) {
+		pm_write16(PM_EVT_BLK, ACPI_PM_EVT_BLK);
+		pm_write16(PM1_CNT_BLK, ACPI_PM1_CNT_BLK);
+		pm_write16(PM_TMR_BLK, ACPI_PM_TMR_BLK);
+		pm_write16(PM_GPE0_BLK, ACPI_GPE0_BLK);
+	}

if (CONFIG(HAVE_SMI_HANDLER)) {
    	/* APMC - SMI Command Port */