Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/36824 )
Change subject: mb/amd/padmelon: Use Prairie Falcon configuration ......................................................................
mb/amd/padmelon: Use Prairie Falcon configuration
While Merlin Falcon binaries are not available, make it explicit that it's compiling for Prairie Falcon (it was being surreptitious about it).
Board Padmelon accepts 3 different SOC, just changing some resistors (soldered or not): Brown Falcon, Prairie Falcon and Merlin Falcon. Code for Brown Falcon is not currently available.
BUG=None TEST=Build with prairie falcon.
Change-Id: I1663e4403a32a7d626dd2fa06763f18f4230457e Signed-off-by: Richard Spiegel richard.spiegel@silverbackltd.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/36824 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/mainboard/amd/padmelon/Kconfig 1 file changed, 17 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Marshall Dawson: Looks good to me, approved
diff --git a/src/mainboard/amd/padmelon/Kconfig b/src/mainboard/amd/padmelon/Kconfig index 1be3a62..e9d2acb 100644 --- a/src/mainboard/amd/padmelon/Kconfig +++ b/src/mainboard/amd/padmelon/Kconfig @@ -18,7 +18,6 @@ config BOARD_SPECIFIC_OPTIONS # dummy def_bool y select SOC_AMD_STONEYRIDGE - select AMD_APU_MERLINFALCON select AMD_APU_PKG_FP4 select BOARD_ROMSIZE_KB_8192 select DRIVERS_I2C_GENERIC @@ -43,6 +42,23 @@ string default "Padmelon"
+choice PADMELON_SOC + prompt "SOC used in padmelon board" + default PADMELON_MERLIN_FALCON + +config PADMELON_MERLIN_FALCON + bool "Merlin Falcon" + +config PADMELON_PRAIRIE_FALCON + bool "Prairie Falcon" + +endchoice + +config PADMELON_SOC_IN_USE + def_bool y + select AMD_APU_MERLINFALCON if PADMELON_MERLIN_FALCON + select AMD_APU_PRAIRIEFALCON if PADMELON_PRAIRIE_FALCON + config HAVE_S3_SUPPORT bool default n