Arthur Heymans 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: 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.