Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34152 )
Change subject: soc/intel: Move some PARALLEL_MP prototypes ......................................................................
Patch Set 4:
That these share the same declaration raises a question: Why have 6 copies of the (nearly) same code in the first place?
The same thing happens for pretty much every cpu/soc, not just the 7 here. Initialisers of struct mp_ops are split across two or more files, otherwise we would just declare the functions with static together in the same file with const initialiser of mp_ops, right?
Yeah, I see the pattern. I just want to make sure that this doesn't become a copy-pasta supporting change. There is some more weirdness, e.g. smm_lock() seems to be declared somewhere in arch/x86/ already. I'm not sure if it's the same function.
Nvm, found the related patch.