From 2f548e597b637d6a66eb45ffca77e2ce2a776ae6 Mon Sep 17 00:00:00 2001 From: Maximilian Brune Date: Thu, 20 Oct 2022 18:56:43 +0200 Subject: drivers/intel/fsp2_0/memory_init.c: clean code No need to call a function that just instantly returns. It greatly enhances readability to just check before calling a funtion and it also removes an extra argument. Change-Id: I4d57c45ede520160ef615725c023b7e92289a995 Signed-off-by: Maximilian Brune Reviewed-on: https://review.coreboot.org/c/coreboot/+/68592 Tested-by: build bot (Jenkins) Reviewed-by: Lean Sheng Tan Reviewed-by: Angel Pons --- src/drivers/intel/fsp2_0/memory_init.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'src/drivers/intel/fsp2_0') diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c index f7aaed48b1..99b0feae49 100644 --- a/src/drivers/intel/fsp2_0/memory_init.c +++ b/src/drivers/intel/fsp2_0/memory_init.c @@ -26,14 +26,11 @@ static uint8_t temp_ram[CONFIG_FSP_TEMP_RAM_SIZE] __aligned(sizeof(uint64_t)); -static void save_memory_training_data(bool s3wake, uint32_t fsp_version) +static void save_memory_training_data(uint32_t fsp_version) { size_t mrc_data_size; const void *mrc_data; - if (!CONFIG(CACHE_MRC_SETTINGS) || s3wake) - return; - mrc_data = fsp_find_nv_storage_data(&mrc_data_size); if (!mrc_data) { printk(BIOS_ERR, "FSP_NON_VOLATILE_STORAGE_HOB missing!\n"); @@ -76,7 +73,8 @@ static void do_fsp_post_memory_init(bool s3wake, uint32_t fsp_version) (uintptr_t)cbmem_find(CBMEM_ID_FSP_RESERVED_MEMORY)) die("Failed to accommodate FSP reserved memory request!\n"); - save_memory_training_data(s3wake, fsp_version); + if (CONFIG(CACHE_MRC_SETTINGS) && !s3wake) + save_memory_training_data(fsp_version); /* Create romstage handof information */ romstage_handoff_init(s3wake); -- cgit v1.2.3