Attention is currently required from: Paul Menzel, Yu-Ping Wu.
Dinesh Gehlot has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/83686?usp=email )
Change subject: soc/intel/common/block/cse: Enforce CSE sync with pertinent GBB flag ......................................................................
Patch Set 3:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83686/comment/f0c55d59_5c7ef96c?usp... : PS2, Line 7: src:
Wrong prefix?
Acknowledged
https://review.coreboot.org/c/coreboot/+/83686/comment/b0f2272d_1b4882fe?usp... : PS2, Line 9: The patch enforces CSE sync when the GBB flag GBB_FLAG_FORCE_CSE_SYNC is : enabled and the system is currently booting from the RO section.
Why is enforcing this the right thing to do?
We are intentionally initiating the synchronization process to validate the upstream process, even when the current CSE version matches the CBFS-stored CSE version, hence its `enforcing`.
Please let me know if you would like me to rephrase the description.
https://review.coreboot.org/c/coreboot/+/83686/comment/c5ac6544_f5cfa31d?usp... : PS2, Line 13: CL:5705989
It’s not marked up in the upstream Gerrit. Please add a URL.
This is a downstream patch. I took out the CL details from the description because the existing description and patch comments has sufficient information.