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

mb/google/guybrush/bootblock: drop redundant clearing of LPC decodes 

The writes were originally added due to being part of the initialization
sequence in the reference code, but coreboot already has those registers
cleared by the time we reach this part of the code, so we can drop these
redundant writes.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I43344460e5355664841d77daf1df3fd386e047e9
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58341
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
Felix Held 2021-10-12 21:39:47 +02:00 committed by Felix Held
parent 1c03da5f5a
commit 718a3cba25
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/mainboard/google/guybrush/bootblock.c
View File

@ -39,8 +39,11 @@ void bootblock_mainboard_early_init(void)
dword |= LPC_LDRQ0_PD_EN;
pci_write_config32(SOC_LPC_DEV, LPC_MISC_CONTROL_BITS, dword);
pci_write_config32(SOC_LPC_DEV, LPC_IO_PORT_DECODE_ENABLE, 0);
pci_write_config32(SOC_LPC_DEV, LPC_IO_OR_MEM_DECODE_ENABLE, 0);
/*
* All LPC decodes need to be cleared before we can configure the LPC pads as secondary
* eSPI interface that gets used for the EC communication. This is already done by
* lpc_disable_decodes that gets called before this function.
*/
if (CONFIG(VBOOT_STARTS_BEFORE_BOOTBLOCK))
return;