[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Check return type against CB_SUCCESS

[Subrata Banik (Code Review)]

Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/58757 )

Change subject: drivers/intel/fsp2_0: Check return type against CB_SUCCESS
......................................................................

drivers/intel/fsp2_0: Check return type against CB_SUCCESS

commit 6af980a2a
(drivers/intel/fsp2_0: Allow `mp_startup_all_cpus()` to run serially)
drops CB_SUCCESS check for mp_run_on_all_aps function hence, this
changes bring back the required return type against CB_SUCCESS.

Change-Id: I9fc81e6a7eebbf0072ea2acb36b3c33539b517a7
Signed-off-by: Subrata Banik 
---
M src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c
1 file changed, 2 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/58757/1

diff --git a/src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c b/src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c
index 5b537a9..9aef1b6 100644
--- a/src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c
+++ b/src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c
@@ -108,7 +108,8 @@
     * due to lack of acquiring a spin lock while accessing common data structure in
     * multiprocessor environment.
     */
-	if (mp_run_on_all_aps((void *)procedure, argument, timeout_usec, false)) {
+	if (mp_run_on_all_aps((void *)procedure, argument, timeout_usec, false) !=
+				CB_SUCCESS) {
    	printk(BIOS_DEBUG, "%s: Exit with Failure\n", __func__);
    	return FSP_NOT_STARTED;
    }