Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33357 )
Change subject: soc/intel/block/cpu: remove unused USE_COREBOOT_NATIVE_MP_INIT ......................................................................
Patch Set 3:
I guess we could still make it a choice, but not sure if that's necessary:
-> coreboot MP init -> coreboot MP init w/ PPI -> FSP MP init
You actually still can do all those options -> coreboot MP init : don't select USE_INTEL_FSP_MP_INIT or USE_INTEL_FSP_TO_CALL_COREBOOT_PUBLISH_MP_PPI
- coreboot MP init w/ PPI: select USE_INTEL_FSP_TO_CALL_COREBOOT_PUBLISH_MP_PPI
-> FSP MP init: select USE_INTEL_FSP_MP_INIT
a choice would just be there for more clarity I suppose?
Yes, choice would only be for usability.