Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37676 )
Change subject: vc/amd/pi: Fix typo ......................................................................
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.)
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...