[coreboot-gerrit] Change in coreboot[master]: vendorcode/amd/pi: Integrate Merlin Falcon as a build option

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33622 )

Change subject: vendorcode/amd/pi: Integrate Merlin Falcon as a build option
......................................................................

Patch Set 10:

(1 comment)

https://review.coreboot.org/c/coreboot/+/33622/10/src/vendorcode/amd/pi/Kcon... 
File src/vendorcode/amd/pi/Kconfig:

https://review.coreboot.org/c/coreboot/+/33622/10/src/vendorcode/amd/pi/Kcon... 
PS10, Line 48: if SOC_AMD_STONEYRIDGE_FP4 ||
> I think you meant for this to be "if SOC_AMD_STONEYRIDGE_FP4 &&" […]

If there's no binaries I want 00670F00 AGESA. Though I'm starting to consider doing it differently. The configuration I'm setting when there's no Merlin Falcon binaries are for Prairie Falcon, a valid SOC for padmelon board (I have 2 padmelon boards, one with each SOC).
So I'm considering removing "HAVE_MERLINFALCON_BINARIES" and replacing with "SOC_AMD_PRAIRIEFALCON".