Attention is currently required from: Intel coreboot Reviewers.
Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/87266?usp=email )
Change subject: soc/intel/common/cse: Skip CSE state setting with LITE_SYNC_BY_PAYLOAD ......................................................................
soc/intel/common/cse: Skip CSE state setting with LITE_SYNC_BY_PAYLOAD
The CSE state setting logic in `cse_set_state` is not required when the `SOC_INTEL_CSE_LITE_SYNC_BY_PAYLOAD` configuration option is enabled.
This commit adds a conditional check to skip the state setting in such cases, potentially optimizing the CBFS travering time to locate `option/me_state` file.
TEST=Able to build and boot google/fatcat.
Change-Id: I43f5daab450989307d9b3529949e9f03cba4404d Signed-off-by: Subrata Banik subratabanik@google.com --- M src/soc/intel/common/block/cse/cse.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/66/87266/1
diff --git a/src/soc/intel/common/block/cse/cse.c b/src/soc/intel/common/block/cse/cse.c index 2e0f2db..68d0c21 100644 --- a/src/soc/intel/common/block/cse/cse.c +++ b/src/soc/intel/common/block/cse/cse.c @@ -1298,6 +1298,9 @@
static void cse_set_state(struct device *dev) { + if (CONFIG(SOC_INTEL_CSE_LITE_SYNC_BY_PAYLOAD)) + return; + /* (CS)ME Disable Command */ struct me_disable_command { struct mkhi_hdr hdr;