Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36839 )
Change subject: soc/intel/icelake: Make CpuMpPpi implementation default for ICL ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1: Code-Review-1
Patch Set 1:
Patch Set 1:
Patch Set 1:
Patch Set 1:
Not for now. If someone just implements it publicly as open-source solution we can undo the changes
I guess the problem is that, some CPU programming has to be performed between FSP calls so until entire FSP-S is not open source i guess this going to be our only solution :(
In what sense does it has to be performed between FSP calls? Can it be implemented in coreboot, but Intel wants to hide some of the details in FSP (or is not interested in doing it in open source code for other reasons) or does FSP have an artificial reliance on being able to CPU programming itself. If the latter is the case, please document it.
HI Arthur, i believe we have document here https://github.com/coreboot/coreboot/blob/master/Documentation/soc/intel/mp_... and we had great discussion in past while enabling this MP PPI implemetation.
Yes I read it, but your statement that FSP needs to be fully open sourced confused me into thinking that FSP had some undocumented reliance on MP_PPI.
The Kconfig options still allow FSP-S to do the MP init fully. You need to clean that up too.
i'm cleaning this because other else option doesn't make any sense in ICL. as i have documented the feature won't be able to work unless we do MP PPI in ICL
i meant below code doesn't make any sense as either SKipMpInit=1 would allow to skip FSP-S doing MP Init but now on ICL thats not an option also SkipMpinit is getting deprecated
params->SkipMpInit = !CONFIG_USE_INTEL_FSP_MP_INIT;
if you could see TODO that I have written, SkipMpInit is going to deprecate in next FSP releases.