[coreboot-gerrit] Change in coreboot[master]: mb/amd/padmelon: Use Prairie Falcon configuration

[Richard Spiegel (Code Review)]

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

Change subject: mb/amd/padmelon: Use Prairie Falcon configuration
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36824/1/src/mainboard/amd/padmelon/... 
File src/mainboard/amd/padmelon/Kconfig:

https://review.coreboot.org/c/coreboot/+/36824/1/src/mainboard/amd/padmelon/... 
PS1, Line 20: select SOC_AMD_PRAIRIEFALCON
> Why not make this a choice instead, since the board can support multiple APUs?

I could, provided I don't add Brown Falcon as a possible choice because I don't have the code written for it. I suppose that if I found the correct binaries and select them, it should work... but I would need a Padmelon with Brown Falcon in order to try it.