<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 46:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">If Intel finds ways to make their customers ignore coreboot's license, I'll probably find ways to ignore theirs.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Intel understands and respects that coreboot is a GPL licensed project. None of this work is intended or expected to cause licensing issues, otherwise we would have never started this work in the first place. We have been very open with the community on the intent of this change - we wish to bring more openness and customization to the FSP. PI spec complaint firmware implementations have the freedom to change the MP services implementation used by the FSP, the intent is to extend that freedom to coreboot users.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Got it but we need to clarify this with our own legal team. We contacted the Software Freedom Conservancy as our organization in order to get the process started.<br>> <br>> All our actions here are in good faith. We don't need coreboot's MP code to get FSP's job done... we have our own MP code. Our only reason for making this change is for coreboot's benefit. Why would Intel make the effort to implement a change solely for the community's benefit if we were trying to be disrespectful? </p><p style="white-space: pre-wrap; word-wrap: break-word;">Sure I believe that's true, please give us some time to get feedback from our legal team.<br>What is your deadline/roadmap?<br>> <br>> > What I was trying to say between the lines: If Intel makes it unbearable to use FSP, people will try to work around it. What makes it unbearable? Most of all that it has its own control flow (which brings us issues like these here with MP init).<br>> <br>> This is the technical issue that we are trying to make better here. If the community is unwilling to accept any changes to the FSP API, then we cannot improve it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's true.<br>> <br>> > Well, I can't say if implementing #2 can save FSP so there is probably little use for it.<br>> <br>> I understand that you don't like the existence of FSP, I am trying to make it better given the business constraints that mandate FSP's existence. Are you willing to work with me on this?</p><p><a href="https://review.coreboot.org/c/coreboot/+/25634">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/25634">change 25634</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/c/coreboot/+/25634"/><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-Change-Id: Ie844e3f15f759ea09a8f3fd24825ee740151c956 </div>
<div style="display:none"> Gerrit-Change-Number: 25634 </div>
<div style="display:none"> Gerrit-PatchSet: 46 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Idwer Vollering <vidwer@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> </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-Reviewer: ron minnich <rminnich@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Aaron Durbin <adurbin@chromium.org> </div>
<div style="display:none"> Gerrit-CC: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-CC: Nathaniel L Desimone <nathaniel.l.desimone@intel.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Dec 2018 12:36:43 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>