soc/intel/(adl, cmn, mtl): Refactor cse_fw_sync() function
This patch refactors cse_fw_sync() function to include timestamp
associated with the CSE sync operation.This effort will ensure the
SoC code just makes a call into the cse_fw_sync() without bothering
about adding timestamp entries.
TEST=Able to build and boot google/marasov.
Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: Ib5e8fc2b8c3b605103f7b1238df5a8405e363f83
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74582
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Kapil Porwal <kapilporwal@google.com>
Reviewed-by: Kangheui Won <khwon@chromium.org>
diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c
index 1e2a052..14a7381 100644
--- a/src/soc/intel/common/block/cse/cse_lite.c
+++ b/src/soc/intel/common/block/cse/cse_lite.c
@@ -1077,7 +1077,7 @@
return handle_cse_sub_part_fw_update_rv(rv);
}
-void cse_fw_sync(void)
+static void do_cse_fw_sync(void)
{
static struct get_bp_info_rsp cse_bp_info;
@@ -1152,6 +1152,13 @@
}
}
+void cse_fw_sync(void)
+{
+ timestamp_add_now(TS_CSE_FW_SYNC_START);
+ do_cse_fw_sync();
+ timestamp_add_now(TS_CSE_FW_SYNC_END);
+}
+
static enum cb_err send_get_fpt_partition_info_cmd(enum fpt_partition_id id,
struct fw_version_resp *resp)
{
@@ -1214,11 +1221,8 @@
if (acpi_get_sleep_type() == ACPI_S3)
return;
- if (CONFIG(SOC_INTEL_CSE_LITE_SYNC_IN_RAMSTAGE)) {
- timestamp_add_now(TS_CSE_FW_SYNC_START);
+ if (CONFIG(SOC_INTEL_CSE_LITE_SYNC_IN_RAMSTAGE))
cse_fw_sync();
- timestamp_add_now(TS_CSE_FW_SYNC_END);
- }
}
BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_EXIT, ramstage_cse_fw_sync, NULL);