Rob Barnes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/52401 )
Change subject: guybrush: Add Kconfig for PSP eSPI and port80 ......................................................................
guybrush: Add Kconfig for PSP eSPI and port80
Add PSP_DISABLE_PORT80 and PSP_INITIALIZE_ESPI kconfig options for cezanne. Select PSP_DISABLE_PORT80 and unselect PSP_INITIALIZE_ESPI for guybrush. Port80 codes from PSP can cause bus errors on guybrush.
BUG=b:185514903, b:184356693 TEST=Boot guybrush, observe no port80 codes from PSP
Change-Id: I7241e47ec1b89782e699135370c796eb251afcaa Signed-off-by: Rob Barnes robbarnes@google.com --- M src/mainboard/google/guybrush/Kconfig M src/soc/amd/cezanne/Kconfig M src/soc/amd/cezanne/Makefile.inc 3 files changed, 22 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/01/52401/1
diff --git a/src/mainboard/google/guybrush/Kconfig b/src/mainboard/google/guybrush/Kconfig index 9f43d5e..08b09b1 100644 --- a/src/mainboard/google/guybrush/Kconfig +++ b/src/mainboard/google/guybrush/Kconfig @@ -26,6 +26,7 @@ select MAINBOARD_HAS_CHROMEOS select MAINBOARD_HAS_I2C_TPM_CR50 select MAINBOARD_HAS_TPM2 + select PSP_DISABLE_PORT80 select SOC_AMD_CEZANNE select SOC_AMD_COMMON_BLOCK_USE_ESPI
@@ -64,6 +65,9 @@ hex default 0x50
+config PSP_INITIALIZE_ESPI + default n + config EFS_SPI_READ_MODE int default 0 if EM100 # Normal read mode diff --git a/src/soc/amd/cezanne/Kconfig b/src/soc/amd/cezanne/Kconfig index 7a1146f..ca4bb36 100644 --- a/src/soc/amd/cezanne/Kconfig +++ b/src/soc/amd/cezanne/Kconfig @@ -255,6 +255,16 @@ string default "src/soc/amd/cezanne/fw.cfg"
+config PSP_DISABLE_PORT80 + def_bool n + help + Disables the output of port80 codes from PSP. + +config PSP_INITIALIZE_ESPI + def_bool y + help + Enables the initialization of the eSPI bus in PSP. + config PSP_LOAD_MP2_FW bool default n diff --git a/src/soc/amd/cezanne/Makefile.inc b/src/soc/amd/cezanne/Makefile.inc index 80ce622..687233a 100644 --- a/src/soc/amd/cezanne/Makefile.inc +++ b/src/soc/amd/cezanne/Makefile.inc @@ -80,6 +80,14 @@
FIRMWARE_LOCATION=$(shell grep -e FIRMWARE_LOCATION $(CONFIG_AMDFW_CONFIG_FILE) | awk '{print $$2}')
+ifeq ($(CONFIG_PSP_DISABLE_PORT80),y) +PSP_SOFTFUSE_BITS += 7 +endif + +ifeq ($(CONFIG_PSP_INITIALIZE_ESPI),y) +PSP_SOFTFUSE_BITS += 15 +endif + ifeq ($(CONFIG_PSP_UNLOCK_SECURE_DEBUG),y) # Enable secure debug unlock PSP_SOFTFUSE_BITS += 0 @@ -122,9 +130,6 @@ # type = 0xb - See #55758 (NDA) for bit definitions. PSP_SOFTFUSE_BITS += 28 6
-#hardcode post code to eSPI -PSP_SOFTFUSE_BITS += 15 - # Helper function to return a value with given bit set set-bit=$(call int-shift-left, 1 $(call _toint,$1)) PSP_SOFTFUSE=$(shell A=$(call int-add, \