[coreboot-gerrit] Change in ...coreboot[master]: soc/amd/stoneyridge: Add merlinfalcon configuration

[Richard Spiegel (Code Review)]

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

Change subject: soc/amd/stoneyridge: Add merlinfalcon configuration
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/#/c/33621/2/src/soc/amd/stoneyridge/Makefile.inc 
File src/soc/amd/stoneyridge/Makefile.inc:

https://review.coreboot.org/#/c/33621/2/src/soc/amd/stoneyridge/Makefile.inc... 
PS2, Line 146: MF
> BTW you might want to force the FILE_ variables for the fanless SKUs to be empty assignments. […]

I figured out a different way, with
ifeq ($(CONFIG_SOC_AMD_PSP_SELECTABLE_SMU_FW),y)

https://review.coreboot.org/#/c/33621/2/src/soc/amd/stoneyridge/Makefile.inc... 
PS2, Line 239: ifeq ($(CONFIG_SOC_AMD_MERLINFALCON_FP4),y)
> Somewhere above, add a variable assignment that depends on the device, e.g.  […]

Seems to work, thanks.