Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/55619 )
Change subject: soc/intel/common: Check CSE RW partition status ......................................................................
soc/intel/common: Check CSE RW partition status
The patch moves CSE Lite RW status check out of CSE RW update logic as the RW sanity check has to be done irrespective of CSE RW update logic is enabled or not.
TEST=Verified boot on Brya
Signed-off-by: Sridahr Siricilla sridhar.siricilla@intel.com Change-Id: I582b6cf24f8894c80ab461ca21f7c6e8caa738bc --- M src/soc/intel/common/block/cse/cse_lite.c 1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/55619/1
diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index c7dec5c..95938d0 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -768,9 +768,6 @@ return cse_trigger_fw_update(cse_bp_info, &source_metadata, &target_rdev); }
- if (!cse_is_rw_bp_status_valid(cse_bp_info)) - return CSE_LITE_SKU_RW_JUMP_ERROR; - return 0; }
@@ -808,6 +805,9 @@ cse_trigger_recovery(rv); }
+ if (!cse_is_rw_bp_status_valid(&cse_bp_info.bp_info)) + cse_trigger_recovery(CSE_LITE_SKU_RW_JUMP_ERROR); + if (!cse_boot_to_rw(&cse_bp_info.bp_info)) { printk(BIOS_ERR, "cse_lite: Failed to switch to RW\n"); cse_trigger_recovery(CSE_LITE_SKU_RW_SWITCH_ERROR);