From 0ae7a8b765f27f1df2330334ed25bc41732f9ea1 Mon Sep 17 00:00:00 2001 From: Sridhar Siricilla Date: Tue, 10 Jan 2023 17:12:01 +0530 Subject: [PATCH] soc/intel/common: Use enum csme_failure_reason The patch updates return type for below functions as they uses 'enum csme_failure_reason' type return values. 1. cse_sub_part_trigger_update() 2. handle_cse_sub_part_fw_update_rv() 3. cse_sub_part_fw_update() TEST=Build coreboot code for Gimble Signed-off-by: Sridhar Siricilla Change-Id: I43bc2d518a275894860e4d3c930c3c4d9685fb3a Reviewed-on: https://review.coreboot.org/c/coreboot/+/71792 Reviewed-by: Eric Lai Reviewed-by: Krishna P Bhat D Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/cse/cse_lite.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 11a4624c26..6e9a09e7b6 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -952,8 +952,8 @@ static bool cse_prep_for_component_update(const struct cse_bp_info *cse_bp_info) return cse_hmrfpo_enable(); } -static uint8_t cse_sub_part_trigger_update(enum bpdt_entry_type type, uint8_t bp, - const void *subpart_cbfs_rw, const size_t blob_sz, +static enum csme_failure_reason cse_sub_part_trigger_update(enum bpdt_entry_type type, + uint8_t bp, const void *subpart_cbfs_rw, const size_t blob_sz, struct region_device *target_rdev) { if (region_device_sz(target_rdev) < blob_sz) { @@ -977,7 +977,7 @@ static uint8_t cse_sub_part_trigger_update(enum bpdt_entry_type type, uint8_t bp return CSE_LITE_SKU_PART_UPDATE_SUCCESS; } -static uint8_t handle_cse_sub_part_fw_update_rv(uint8_t rv) +static enum csme_failure_reason handle_cse_sub_part_fw_update_rv(enum csme_failure_reason rv) { switch (rv) { case CSE_LITE_SKU_PART_UPDATE_SUCCESS: @@ -1052,14 +1052,14 @@ error_exit: return rv; } -static uint8_t cse_sub_part_fw_update(const struct cse_bp_info *cse_bp_info) +static enum csme_failure_reason cse_sub_part_fw_update(const struct cse_bp_info *cse_bp_info) { if (skip_cse_sub_part_update()) { printk(BIOS_INFO, "CSE Sub-partition update not required\n"); return CSE_LITE_SKU_SUB_PART_UPDATE_NOT_REQ; } - int rv; + enum csme_failure_reason rv; rv = cse_sub_part_fw_component_update(IOM_FW, cse_bp_info, CONFIG_SOC_INTEL_CSE_IOM_CBFS_NAME);