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

[Nico Huber (Code Review)]

Nico Huber 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 3:

Sorry for being late. But I don't follow the argumentation here.

If `SkipMpInit` is only deprecated, i.e. not removed yet, why is it
suddenly mandatory to set it to 0? Is `SkipMpInit` broken in current
ICL FSP? Before this change, did something break if
CONFIG_USE_INTEL_FSP_TO_CALL_COREBOOT_PUBLISH_MP_PPI
was deselected?

Also, the commit message doesn't match the change. It was already
the default. This change enforces it.