Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36823 )
Change subject: soc/amd/stoneyridge/kconfig: Make Prairie Falcon selection explicit ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/36823/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36823/1//COMMIT_MSG@18 PS1, Line 18: Merlin Falcon uses the same internal CPU as Carrizo, thus it has dual memory : channel while Prairie Falcon and Stoney ridge have only one channel.
Will remove
Done
https://review.coreboot.org/c/coreboot/+/36823/1//COMMIT_MSG@20 PS1, Line 20: one particular version of Carrizo binaries
Yes...Maybe I should have said Bristol Ridge instead of Carrizo. […]
Done
https://review.coreboot.org/c/coreboot/+/36823/1/src/soc/amd/stoneyridge/Mak... File src/soc/amd/stoneyridge/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/36823/1/src/soc/amd/stoneyridge/Mak... PS1, Line 140: ifeq ($(CONFIG_HAVE_MERLINFALCON_BINARIES),y)
This section might need expansion, as you could declare Prairie Falcon and have Merlin Falcon binari […]
Done