From f3314c20dede95a50d9c2ba2806c09b7f8728f03 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Tue, 23 Mar 2021 13:34:49 -0600 Subject: [PATCH] soc/amd/cezanne: Clear eSPI ranges before configuring eSPI The Cezanne PSP configures the eSPI with the assumption that it's a majolica, setting up both the serial port and the majolica EC IO decode ranges. Since guybrush is NOT a majolica, this doesn't work very well there. Clearing the decode ranges allows the guybrush platform to set the decode ranges needed for its EC. BUG=b:183524609 TEST=Set up eSPI on Guybrush Signed-off-by: Martin Roth Change-Id: I77cfb948cb9ae6d1cf001bd9e66cede8d93f50b5 Reviewed-on: https://review.coreboot.org/c/coreboot/+/51749 Tested-by: build bot (Jenkins) Reviewed-by: Raul Rangel Reviewed-by: Marshall Dawson Reviewed-by: Furquan Shaikh --- src/soc/amd/cezanne/early_fch.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/soc/amd/cezanne/early_fch.c b/src/soc/amd/cezanne/early_fch.c index 0c72863ba3..acd2a7a1ce 100644 --- a/src/soc/amd/cezanne/early_fch.c +++ b/src/soc/amd/cezanne/early_fch.c @@ -68,6 +68,7 @@ void fch_early_init(void) lpc_disable_spi_rom_sharing(); if (CONFIG(SOC_AMD_COMMON_BLOCK_USE_ESPI)) { + espi_clear_decodes(); espi_setup(); espi_configure_decodes(); }