[coreboot-gerrit] Change in coreboot[master]: Documentation/Intel: Add MultiProcessorInit documentation

Subrata Banik (Code Review) gerrit at coreboot.org
Tue May 1 13:35:42 CEST 2018


Subrata Banik has posted comments on this change. ( https://review.coreboot.org/25921 )

Change subject: Documentation/Intel: Add MultiProcessorInit documentation
......................................................................


Patch Set 3:

> (5 comments)
 > 
 > Thanks for writing this up. The relation to the `SkipMpInit` wasn't
 > clear to me before (I'm not 100% sure yet, but much closer).
 > 
 > Please make it clear if you intend to force usage of this interface
 > or if it is optional (for `SkipMpInit=1`).

Nothing is force here, its optional if you want to avoid FSP to use those interface which coreboot has published, then just set SkipMpInit=1, FSP won't call this APIs to run those feature programming over AP.


-- 
To view, visit https://review.coreboot.org/25921
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b6096ef31d8a523c00cbad39ab9d4884e735fde
Gerrit-Change-Number: 25921
Gerrit-PatchSet: 3
Gerrit-Owner: Subrata Banik <subrata.banik at intel.com>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik at intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Nico Huber <nico.h at gmx.de>
Gerrit-CC: Patrick Georgi <pgeorgi at google.com>
Gerrit-Comment-Date: Tue, 01 May 2018 11:35:42 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180501/d7cb08f6/attachment.html>


More information about the coreboot-gerrit mailing list