[M] Change in coreboot[main]: soc/intel/cmn/blk: Remove boot partition check for forced cse sync

[Dinesh Gehlot (Code Review)]

Attention is currently required from: Intel coreboot Reviewers, Kapil Porwal, Subrata Banik.

Dinesh Gehlot has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/86153?usp=email )

Change subject: soc/intel/cmn/blk: Remove boot partition check for forced cse sync
......................................................................

Patch Set 13:

(4 comments)

This change is ready for review.

File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/86153/comment/c51b2cfa_674d6baa?usp... :
PS12, Line 748: if (!(CONFIG(VBOOT) && vboot_is_gbb_flag_set(VB2_GBB_FLAG_FORCE_CSE_SYNC)))
              : 		return;
> shouldn't this logic be like either VBOOT not enable or GBB not set ? […]

Acknowledged

https://review.coreboot.org/c/coreboot/+/86153/comment/87fbb503_fed3bf66?usp... :
PS12, Line 755: 	//cse_info_in_cmos.cse_sync_status &= ~CSE_ENFORCED_SYNC_REQ;
> remove

Acknowledged

https://review.coreboot.org/c/coreboot/+/86153/comment/277557c1_7af87052?usp... :
PS12, Line 765: is_cse_sync_enforced
> this is just prerequisite code that tells if CSE sync enforcement is required or not. […]

The ESOL screen function at romstage also calls `is_cse_sync_enforced()`. At that stage, the enforce CSE sync status flags have not been updated. Therefore, an update to these status flags is required.

File src/soc/intel/common/block/include/intelblocks/cse.h:

https://review.coreboot.org/c/coreboot/+/86153/comment/1e4f6a7e_489d726d?usp... :
PS12, Line 187: REQ
> write in full.

Acknowledged