From 6ad701054230e25fa9b2afe414ca8da6d5a626ee Mon Sep 17 00:00:00 2001 From: Sridhar Siricilla Date: Tue, 12 May 2020 21:28:53 +0530 Subject: soc/intel: Correct comment on HMRFPO_ENABLE HECI command Correct comment on HMRFPO_ENABLE flow for CSE Lite SKU. In order to place CSE into SECOVER_MEI_MSG mode, below procedure has to be followed. 1. Ensure CSE boots from RO(BP1). - Set CSE's next boot partition to RO - Issue GLOBAL_RESET HECI command to reset the system 2. Send HMRFPO_ENABLE command to CSE. Further, no reset is required. TEST=Verified on hatch Signed-off-by: Sridhar Siricilla Change-Id: I213e02ba3898194fa6c8fe38fab34b5c19f25aa1 Reviewed-on: https://review.coreboot.org/c/coreboot/+/41340 Reviewed-by: Furquan Shaikh Reviewed-by: Rizwan Qureshi Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/include/intelblocks/cse.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/soc/intel/common/block/include/intelblocks') diff --git a/src/soc/intel/common/block/include/intelblocks/cse.h b/src/soc/intel/common/block/include/intelblocks/cse.h index 103323b7cc..588d3eb79c 100644 --- a/src/soc/intel/common/block/include/intelblocks/cse.h +++ b/src/soc/intel/common/block/include/intelblocks/cse.h @@ -130,10 +130,10 @@ int cse_request_global_reset(enum rst_req_type rst_type); /* * Sends HMRFPO_ENABLE command. * HMRFPO - Host ME Region Flash Protection Override. - * For CSE Firmware SKU Custom, procedure to place CSE in HMRFPO (SECOVER_MEI_MSG) mode: - * 1. Ensure CSE boots from BP1(RO). - * - Send set_next_boot_partition(BP1) - * - Issue CSE Only Reset + * For CSE Lite SKU, procedure to place CSE in HMRFPO (SECOVER_MEI_MSG) mode: + * 1. Ensure CSE boots from RO(BP1). + * - Set CSE's next boot partition to RO + * - Issue GLOBAL_RESET command to reset the system * 2. Send HMRFPO_ENABLE command to CSE. Further, no reset is required. * * The HMRFPO mode prevents CSE to execute SPI I/O cycles to CSE region, and unlocks -- cgit v1.2.3