Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/49891 )
Change subject: southbridge/intel: Define default value for ME_REGION_ALLOW_CPU_READ_ACCESS ......................................................................
southbridge/intel: Define default value for ME_REGION_ALLOW_CPU_READ_ACCESS
The patch defines default value for ME_REGION_ALLOW_CPU_READ_ACCESS config. It sets value 'y' if CSE Lite SKU is integrated, otherwise value 'n'. The config ME_REGION_ALLOW_CPU_READ_ACCESS ensures host has read access to ME region when the LOCK_MANAGEMENT_ENGINE is enabled and CSE Lite SKU is integrated.
TEST=Verified build for JSL
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: I680a23e27ae2bf4d85bf919134c47882f308af56 Reviewed-on: https://review.coreboot.org/c/coreboot/+/49891 Reviewed-by: Furquan Shaikh furquan@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/southbridge/intel/common/firmware/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved
diff --git a/src/southbridge/intel/common/firmware/Kconfig b/src/southbridge/intel/common/firmware/Kconfig index cd975ba..36dde56 100644 --- a/src/southbridge/intel/common/firmware/Kconfig +++ b/src/southbridge/intel/common/firmware/Kconfig @@ -57,6 +57,7 @@
config ME_REGION_ALLOW_CPU_READ_ACCESS bool "Allows HOST/CPU read access to ME region" + default y if SOC_INTEL_CSE_LITE_SKU default n help The config ensures Host has read access to the ME region if it is locked