[coreboot-gerrit] Change in coreboot[master]: soc/amd,intel: Drop s3_resume parameter on FSP functions

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Mariusz Szafrański, Lee Leahy, Marshall Dawson, Suresh Bellampalli, Vanessa Eusebio, Huang Jin, Michal Motyl, Andrey Petrov, Patrick Rudolph, Felix Held.
Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50360 )

Change subject: soc/amd,intel: Drop s3_resume parameter on FSP functions
......................................................................

Patch Set 2:

(1 comment)

File src/soc/intel/alderlake/cpu.c:

https://review.coreboot.org/c/coreboot/+/50360/comment/49aa5717_47e75c9e 
PS2, Line 27: soc_fsp_load
> We can and should drop `soc_fsp_load` in a follow-up

I am going to question if fsps_load() callsite inside PARALLEL_MP is necessary.

Then again, I don't like placing fsps_load directly as the callback in mp_ops because then the function prototype has to match.