diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2022-08-17 21:38:07 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-08-20 20:43:17 +0000 |
| commit | f7d86f21e12aa3a368aa87aa4a1406e7c38e07b0 (patch) | |
| tree | dca5058f114765b2fe2bf97767a1394a720c79c4 /src | |
| parent | e943e9fc241a50494ff846dce01da33468fe989b (diff) | |
soc/amd/stoneyridge/early_fch: use common lpc_early_init function
The functionality of sb_enable_lpc is implemented in the common LPC
support code as lpc_enable_controller. This gets called by the common
lpc_early_init which also calls lpc_disable_decodes and lpc_set_spibase.
The lpc_set_spibase call was already done in bootblock_fch_early_init,
so the main change in code behavior is that now lpc_disable_decodes gets
called during early FCH initialization. The lpc_enable_port80 and
sb_lpc_decode calls after the lpc_early_init code will reenable some of
the decodes.
TEST=Successfully boots on google/liara, cbmem and dmesg logs look clean
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ia58a6f609fa149a6c09ed99f08bdc4f05eb56f96
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66841
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/soc/amd/stoneyridge/early_fch.c | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/src/soc/amd/stoneyridge/early_fch.c b/src/soc/amd/stoneyridge/early_fch.c index 1f038f61f1..e0d5fa052d 100644 --- a/src/soc/amd/stoneyridge/early_fch.c +++ b/src/soc/amd/stoneyridge/early_fch.c @@ -9,16 +9,6 @@ #include <soc/southbridge.h> #include <types.h> -static void sb_enable_lpc(void) -{ - u8 byte; - - /* Enable LPC controller */ - byte = pm_io_read8(PM_LPC_GATING); - byte |= PM_LPC_ENABLE; - pm_io_write8(PM_LPC_GATING, byte); -} - static void sb_lpc_decode(void) { u32 tmp = 0; @@ -119,11 +109,9 @@ void bootblock_fch_early_init(void) the GPIO registers. */ enable_acpimmio_decode_pm04(); lpc_enable_rom(); - sb_enable_lpc(); + lpc_early_init(); lpc_enable_port80(); sb_lpc_decode(); - /* Make sure the base address is predictable */ - lpc_set_spibase(SPI_BASE_ADDRESS); fch_spi_early_init(); fch_smbus_init(); fch_enable_cf9_io(); |