Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44076 )
Change subject: soc/intel/common/block/cpu: Refactor init_cpus function ......................................................................
Patch Set 6: Code-Review+2
(2 comments)
https://review.coreboot.org/c/coreboot/+/44076/6/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c:
https://review.coreboot.org/c/coreboot/+/44076/6/src/soc/intel/common/block/... PS6, Line 136: init_cpus
yes can be done as well, bt wondering if user would like to run some multithreded programming immedi […]
Sounds good. We can go ahead with this.
https://review.coreboot.org/c/coreboot/+/44076/6/src/soc/intel/common/block/... PS6, Line 149: : microcode_patch = intel_microcode_find(); : intel_microcode_load_unlocked(microcode_patch);
valid point, FSP might not need to update ucode as we don't have SGX enable request bt if it does th […]
I was actually referring to coreboot applying the microcode update. I believe currently it does not happen in case where USE_INTEL_FSP_MP_INIT is selected?