diff options
| author | Subrata Banik <subratabanik@google.com> | 2022-06-15 21:39:06 +0530 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-06-22 12:36:15 +0000 |
| commit | 46265abc7180550e93612c012ade3722bb347f55 (patch) | |
| tree | 5c1cdfa709c1ac6b774b2718cda2e531eed25697 | |
| parent | bd0aef0f2a9a9cee1258aa22e026f1b6054c3d64 (diff) | |
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 <subratabanik@google.com>
Change-Id: Icb3fcfd7ef5478be0a40f8f1358f55c0247b4914
Reviewed-on: https://review.coreboot.org/c/coreboot/+/65157
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/soc/intel/common/block/cpu/mp_init.c | 3 |
1 files changed, 1 insertions, 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 */ |