Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33361 )
Change subject: soc/intel/block/cpu: Add a MP init choice selection ......................................................................
Patch Set 2:
Patch Set 2:
(4 comments)
Subrata, I just realized that a choice doesn't work well if one wants to select a specific option from the mainboard Kconfig. Currently, intel/saddlebrook does that. I'm not sure if that's necessary.
Do you think we need to be able to select things or should it be always up to the person that configures coreboot? If the latter, the choice makes sense, otherwise it would get too complex, I fear.
[Subrata] I think this is better option "we need to be able to select things" choice might be easy to understand but select makes more sense from SOC