I still need it for BtG

Ack, let's see if we can make CB:36682 work.

IMHO, I don't see a conflict to support FSP-T. It opens an option for users. Removing the FSP-T support would force a downstream diversity.

Sorry, you might not know: optional FSP-T support has caused much friction
in the community in the past. Its supporters have often just dumped patches
into our repository and ran off, leaving maintenance to the coreboot commu-
nity. Which, then again, usually doesn't get questions about FSP answered
from Intel's FSP folks. Sometimes, it was unnecessarily enabled by default
in a broken state, causing a lot of costs for the active community. So far
it has caused more damage than provided value, Intel will have to make up
for that if they want us to keep supporting FSP-T (which seems more work
than getting rid of it anyway).

Also, downstream diversity seems unavoidable. And we have to consider whose
downstream we are talking about. So far, none of Intel's customers that want
BtG have approached or supported the coreboot community on this matter. So
one could say there is nothing to lose for us. On top, your other patch
shows that one needed downstream patches anyway. Still, the coreboot commu-
nity was leached for years, not to have no downstream patches but just to
have one less?

Here is what one of your colleagues (somewhere in the CB:36328 discussion)
said why he can only help us with FSP issues in his spare time:

 "[Intel's] management is unwilling to formally assign people to
something that won't directly impact chip sales."

I'll leave it to you to infer what that means for something that doesn't
directly benefit the coreboot community. I'm not saying things have to be
or should be like that, but it was Intel who picked this fight...

(oh, and I forgot to mention that FSP-T replaces open-source code, hence
threatens to destroy coreboot)

View Change

To view, visit change 36622. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib791b30b621730f4b7c0a5f668a3b6559245daf5
Gerrit-Change-Number: 36622
Gerrit-PatchSet: 5
Gerrit-Owner: Michael Niewöhner
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: David Guckian
Gerrit-Reviewer: David Guckian <david.guckian@intel.com>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Gaggery Tsai <gaggery.tsai@intel.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Lean Sheng Tan <lean.sheng.tan@intel.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Michael Niewöhner
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@intel.com>
Gerrit-Reviewer: Vincent Zimmer <vincent.zimmer@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-Comment-Date: Fri, 10 Jan 2020 16:36:32 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment