Patch Set 1: Code-Review+2

Somewhat related, if google/kahlee ships with AGESA_SPLIT_MEMORY_FILES=y, who would need AGESA_SPLIT_MEMORY_FILES=n? Is it that a different set of blobs are under 3rdparty/ or what's going on here?

I doubt anyone would _need_ a non-split image. There's only one blob that's under 3rdparty currently, and that's a non-split one. It seems like amd/padmelon w/Prarie Falcon has landed, or is close, and will use the single image. (BTW, I'm not sure who's driving that but it's not something I'm working on besides a few reviews. So I don't know much more than you do.)

Well at the moment I consider You Marshall to be the 'coreboot man inside AMD'. If you are not driving to get things improved about these blob formats and licenses and repositories then who is.

View Change

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic3d1b9f90c6ed3d85ff209f433de9ab939d760a6
Gerrit-Change-Number: 37676
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 12 Dec 2019 16:19:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment