Patch Set 2:

(1 comment)

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.

IMHO, I would like to have coreboot include microcode updates by default, but only those blobs by default.

Patch set 3:Code-Review +1

View Change

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: 3
Gerrit-Owner: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Felix Singer <felixsinger@posteo.net>
Gerrit-Comment-Date: Sun, 29 Dec 2019 22:56:46 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment