Change in coreboot[master]: drivers/intel/fsp2_0: Allow `mp_startup_all_cpus()` to run serially - coreboot-gerrit

29 Oct 2021


      Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57343 )
Change subject: drivers/intel/fsp2_0: Allow `mp_startup_all_cpus()` to run serially
......................................................................
Patch Set 7:
(1 comment)
File src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c:
https://review.coreboot.org/c/coreboot/+/57343/comment/00525bd0_a8265b64 
PS7, Line 99: != CB_SUCCESS
...
after CB:58494 mp_run_on_aps and mp_run_on_all_aps return enum cb_err type, so it should be checked  […]
Sure, let me take a look into this. Thanks for heads up
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/57343
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia95d11408f663212fd40daa9fd9b0881a07f1ce7
Gerrit-Change-Number: 57343
Gerrit-PatchSet: 7
Gerrit-Owner: Subrata Banik subrata.banik@intel.com
Gerrit-Reviewer: Andrey Petrov andrey.petrov@gmail.com
Gerrit-Reviewer: Furquan Shaikh furquan@google.com
Gerrit-Reviewer: Maulik V Vaghela maulik.v.vaghela@intel.com
Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org
Gerrit-Reviewer: Ronak Kanabar ronak.kanabar@intel.com
Gerrit-Reviewer: Subrata Banik subrata.banik@intel.com
Gerrit-Reviewer: Tim Wawrzynczak twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Felix Held felix-coreboot@felixheld.de
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-CC: Ravindra N ravindra@intel.corp-partner.google.com
Gerrit-Comment-Date: Fri, 29 Oct 2021 14:49:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Held felix-coreboot@felixheld.de
Gerrit-MessageType: comment