From b4de2612289a2a58863671318f5aac619c05f5d2 Mon Sep 17 00:00:00 2001 From: Sridhar Siricilla Date: Thu, 28 Apr 2022 14:36:38 +0530 Subject: soc/intel/common: Use coreboot error codes The patch uses coreboot error codes instead of uint8_t data type in the pre_mem_debug_init function. TEST=build code for Gimble Signed-off-by: Sridhar Siricilla Change-Id: I957ff764900cb789bf2aacf0472dcb281f48af07 Reviewed-on: https://review.coreboot.org/c/coreboot/+/64301 Tested-by: build bot (Jenkins) Reviewed-by: Maulik V Vaghela --- src/soc/intel/common/basecode/debug/debug_feature.c | 6 +++--- src/soc/intel/common/basecode/include/intelbasecode/debug_feature.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/soc/intel/common/basecode/debug/debug_feature.c b/src/soc/intel/common/basecode/debug/debug_feature.c index 1c55f0b887..df32e1dbf2 100644 --- a/src/soc/intel/common/basecode/debug/debug_feature.c +++ b/src/soc/intel/common/basecode/debug/debug_feature.c @@ -27,12 +27,12 @@ bool is_debug_cse_fw_update_disable(void) return pre_mem_debug.cse_fw_update_disable == 1; } -uint8_t pre_mem_debug_init(void) +enum cb_err pre_mem_debug_init(void) { if (spi_flash_read(boot_device_spi_flash(), PRE_MEM_FEATURE_CTRL_OFFSET, PRE_MEM_FEATURE_CTRL_SZ, &pre_mem_debug)) { printk(BIOS_ERR, "Failed to read Descriptor Region from SPI Flash\n"); - return 1; + return CB_ERR; } - return 0; + return CB_SUCCESS; } diff --git a/src/soc/intel/common/basecode/include/intelbasecode/debug_feature.h b/src/soc/intel/common/basecode/include/intelbasecode/debug_feature.h index 51e3f4d05a..664a739932 100644 --- a/src/soc/intel/common/basecode/include/intelbasecode/debug_feature.h +++ b/src/soc/intel/common/basecode/include/intelbasecode/debug_feature.h @@ -12,6 +12,6 @@ bool is_debug_cse_fw_update_disable(void); * Reads OEM Section area in the Descriptor Region and * populates pre_mem_debug structure. */ -uint8_t pre_mem_debug_init(void); +enum cb_err pre_mem_debug_init(void); #endif -- cgit v1.2.3