Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44058 )
Change subject: soc/intel/{icl,jsl,tgl}: Remove SkipMpInit UPD as deprecated ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44058/1/src/soc/intel/tigerlake/fsp... File src/soc/intel/tigerlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/44058/1/src/soc/intel/tigerlake/fsp... PS1, Line 111: USE_INTEL_FSP_MP_INIT
This option won't do anything for these platforms, right? If so, we shouldn't be showing it.
no... it can do entire MP init by FSP. i'm working to validate this path as well when user select USE_INTEL_FSP_MP_INIT and not the USE_INTEL_FSP_TO_CALL_COREBOOT_PUBLISH_MP_PPI, at least platform should behave in same way without any brokenness.