From 46265abc7180550e93612c012ade3722bb347f55 Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Wed, 15 Jun 2022 21:39:06 +0530 Subject: [PATCH] intel/mp_init: Call `intel_reload_microcode()` before post_cpus_init() This patch calls into `intel_reload_microcode() function to load second microcode patch after BIOS Done bit is set and before setting the BIOS Reset CPL bit. Also, remove redundant microcode reloading debug print. BUG=b:233199592 TEST=Build and boot google/kano to ChromeOS. Signed-off-by: Subrata Banik Change-Id: Icb3fcfd7ef5478be0a40f8f1358f55c0247b4914 Reviewed-on: https://review.coreboot.org/c/coreboot/+/65157 Reviewed-by: Lean Sheng Tan Tested-by: build bot (Jenkins) --- src/soc/intel/common/block/cpu/mp_init.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/soc/intel/common/block/cpu/mp_init.c b/src/soc/intel/common/block/cpu/mp_init.c index 0be6e2a0b4..119be3deeb 100644 --- a/src/soc/intel/common/block/cpu/mp_init.c +++ b/src/soc/intel/common/block/cpu/mp_init.c @@ -181,8 +181,7 @@ void before_post_cpus_init(void) if (mp_run_on_all_cpus(&wrapper_set_bios_done, NULL) != CB_SUCCESS) printk(BIOS_ERR, "Set BIOS Done failure\n"); - initialize_microcode(); - printk(BIOS_INFO, "%s: second patch load done\n", __func__); + intel_reload_microcode(); } /* Ensure to re-program all MTRRs based on DRAM resource settings */