From db7b35a9c97c07a6097779459a7816c38242f569 Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Wed, 19 Apr 2023 20:48:01 +0530 Subject: =?UTF-8?q?soc/intel/cmn/cse:=20Refactor=C2=A0ramstage=5Fcse=5Ffw?= =?UTF-8?q?=5Fsync()=20function?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This patch refactors sleep type check inside ramstage_cse_fw_sync() to avoid additional logic while performing cse_fw_sync() operation. TEST=Able to build and boot google/marasov. Signed-off-by: Subrata Banik Change-Id: I7c7a91c81d51dbf6742e12c58a24b9f52fff5630 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74581 Tested-by: build bot (Jenkins) Reviewed-by: Arthur Heymans Reviewed-by: Kapil Porwal Reviewed-by: Kangheui Won --- src/soc/intel/common/block/cse/cse_lite.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/soc/intel/common/block') diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 4cc9e19d7c..1e2a052378 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -1211,10 +1211,10 @@ enum cb_err cse_get_fpt_partition_info(enum fpt_partition_id id, struct fw_versi static void ramstage_cse_fw_sync(void *unused) { - bool s3wake; - s3wake = acpi_get_sleep_type() == ACPI_S3; + if (acpi_get_sleep_type() == ACPI_S3) + return; - if (CONFIG(SOC_INTEL_CSE_LITE_SYNC_IN_RAMSTAGE) && !s3wake) { + if (CONFIG(SOC_INTEL_CSE_LITE_SYNC_IN_RAMSTAGE)) { timestamp_add_now(TS_CSE_FW_SYNC_START); cse_fw_sync(); timestamp_add_now(TS_CSE_FW_SYNC_END); -- cgit v1.2.3