[coreboot-gerrit] Change in coreboot[master]: soc/intel/icelake: Make CpuMpPpi implementation default for ICL

[Arthur Heymans (Code Review)]

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