<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(4 comments)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I agree with you on most this comments that you have put, but my only point is that, if i tries to put this implementation of creating EDK2 interface inside Coreboot to do certain thinks might not justify the purpose of adding those "foreign" interface as your words. Today coreboot code is so independent in nature so that we don;t need to talk help of such EDK2 interface to do certain things, if we really want to increase mp_init.c capability we could have done the same using native c why UEFI?</p><p style="white-space: pre-wrap; word-wrap: break-word;">UEFI is because this implementation only targeted for Intel fsp 9th gen + designs where we might want to do few things using coreboot callback from fsp, hence i could see a direct dependencies between this implementation proposal and supporting FSP model to justify that callback using SkipMpInit=0. if i make this code into common x86 place as well, no one will use this except 9th gen + socs from intel with proper FSP support. </p><p style="white-space: pre-wrap; word-wrap: break-word;">whats the point of keeping certain code into common when there are no user of that code expect intel native FSP support? </p><p style="white-space: pre-wrap; word-wrap: break-word;">Hope i could able to explain my concern, i have written this document so that i can keep it anywhere but myself not very convinced about why other users will interested to make use of EFI code interface inside x86/coreboot when mp_init.c is capable of doing everything without those APIs.</p><p><a href="https://review.coreboot.org/25921">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/25921">change 25921</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/25921"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5b6096ef31d8a523c00cbad39ab9d4884e735fde </div>
<div style="display:none"> Gerrit-Change-Number: 25921 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-CC: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 01 May 2018 12:34:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>