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.)

Splitting into two blobs was work that Google did. I've wanted to push images to coreboot.org but I need to double-check the legality of it first. For example, maybe that's now considered a derivative work and AMD doesn't have rights to redistribute it -- I'm just not certain without digging...

Patch set 1:Code-Review +2

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: 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-Comment-Date: Thu, 12 Dec 2019 15:47:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment