diff options
| author | Subrata Banik <subratabanik@google.com> | 2022-01-28 03:12:35 +0530 |
|---|---|---|
| committer | Subrata Banik <subratabanik@google.com> | 2022-02-02 07:34:39 +0000 |
| commit | be3e911d53b1df8ba650eba5ddbd057119156684 (patch) | |
| tree | 706cd176905624b4f4ae5c6d244725a2747a0640 /src/soc/intel/skylake/finalize.c | |
| parent | e49a615320c16bd05e26a2d863eb64ddd65eb61b (diff) | |
soc/intel/skylake: Use PCR write to disable HECI1
Set the SOC_INTEL_COMMON_BLOCK_HECI1_DISABLE_USING_PCR config for
Skylake to disable HECI1 device using PCR writes.
BUG=none
TEST=None
Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: Ib6bfa7c48660a6df8d0944de675a4f30fe248d1b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61433
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Diffstat (limited to 'src/soc/intel/skylake/finalize.c')
| -rw-r--r-- | src/soc/intel/skylake/finalize.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/soc/intel/skylake/finalize.c b/src/soc/intel/skylake/finalize.c index 733f037975..50215954b9 100644 --- a/src/soc/intel/skylake/finalize.c +++ b/src/soc/intel/skylake/finalize.c @@ -9,6 +9,7 @@ #include <device/pci.h> #include <device/pci_ops.h> #include <intelblocks/cpulib.h> +#include <intelblocks/cse.h> #include <intelblocks/lpc_lib.h> #include <intelblocks/p2sb.h> #include <intelblocks/pcr.h> @@ -30,7 +31,7 @@ #define PCR_PSFX_T0_SHDW_PCIEN 0x1C #define PCR_PSFX_T0_SHDW_PCIEN_FUNDIS (1 << 8) -static void pch_disable_heci(void) +void soc_disable_heci1_using_pcr(void) { /* unhide p2sb device */ p2sb_unhide(); @@ -60,7 +61,7 @@ static void pch_finalize_script(struct device *dev) /* we should disable Heci1 based on the config */ if (CONFIG(DISABLE_HECI1_AT_PRE_BOOT)) - pch_disable_heci(); + heci1_disable(); /* Hide p2sb device as the OS must not change BAR0. */ p2sb_hide(); |