I have very mixed feelings about this patch. On one hand, as you say, it better supports most of the coreboot users. The other side though is that we'd still like to encourage use of blob-free platforms. I thought Ron's idea of actually only showing blob free platforms by default was fantastic, but it caused other issues.

I don't like that this is needed, but even building coreboot without microcode updates is a really bad idea. My main motivation for this user friendliness.

I guess the unfortunate reality of our current situation is that most modern platforms are going to require blobs. I think we're making progress in reversing that, but DAMN, it's slow.

yeah :/

It would probably be good to discuss this in the coreboot leadership meeting, but I won't be in that meeting on January 1st.

View Change

1 comment:

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia0632b9ae7a1f212a8640b3faec2695d17d238c5
Gerrit-Change-Number: 37972
Gerrit-PatchSet: 2
Gerrit-Owner: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Felix Singer <felixsinger@posteo.net>
Gerrit-CC: Martin Roth <martinroth@google.com>
Gerrit-Comment-Date: Sun, 29 Dec 2019 00:40:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Martin Roth <martinroth@google.com>
Gerrit-MessageType: comment