[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add support for MP services2 PPI

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Furquan Shaikh, Martin Roth, Meera Ravindranath, Aamir Bohra, Tim Wawrzynczak, Andrey Petrov, Patrick Rudolph.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49474 )

Change subject: drivers/intel/fsp2_0: Add support for MP services2 PPI
......................................................................

Patch Set 9:

(1 comment)

File src/drivers/intel/fsp2_0/include/fsp/ppi/mp_service_ppi.h:

https://review.coreboot.org/c/coreboot/+/49474/comment/7c5fafa2_b2a99ed7 
PS6, Line 49: /*
            :  * switches the requested AP to be the BSP
            :  */
            : efi_return_status_t mp_switch_bsp(void);
            : 
            : /*
            :  * enable or disable an AP. This service may only be called from the BSP.
            :  */
            : efi_return_status_t mp_enable_disable_ap(void);
            :
> I think Tim meant:  […]

Yes, what Furquan said, not function pointer, that way you get the name in the list, and the code clearly says unsupported in it.