GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common/espi: Reset eSPI registers to known state 

This sets the eSPI registers to the reset values specified in the PPR.

On Cezanne, the PSP modifies these registers such that the eSPI
peripheral cannot send DEFER packets. This causes random bus errors.

These reset values are identical to what is currently used on Zork.

I didn't clear out ESPI_DECODE because it's currently being done by
cb:51749.

BUG=b:183524609
TEST=Boot guybrush to the OS

Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: Ic3a9860747aac78121358b4499d8a38052236c0c
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52058
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Raul E Rangel 2021-04-02 10:32:03 -06:00 committed by Raul Rangel
parent 1d0e4930ba
commit b92383a8a5
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/soc/amd/common/block/lpc/espi_util.c
View File

@ -885,6 +885,11 @@ int espi_setup(void)
uint32_t slave_caps;
const struct espi_config *cfg = espi_get_config();
espi_write32(ESPI_GLOBAL_CONTROL_0, ESPI_AL_STOP_EN);
espi_write32(ESPI_GLOBAL_CONTROL_1, ESPI_RGCMD_INT(23) | ESPI_ERR_INT_SMI);
espi_write32(ESPI_SLAVE0_INT_EN, 0);
espi_clear_status();
/*
* Boot sequence: Step 1
* Set correct initial configuration to talk to the slave:
@ -962,5 +967,8 @@ int espi_setup(void)
/* Enable subtractive decode if configured */
espi_setup_subtractive_decode(cfg);
espi_write32(ESPI_GLOBAL_CONTROL_1,
espi_read32(ESPI_GLOBAL_CONTROL_1) | ESPI_BUS_MASTER_EN);
return 0;
}