Marshall Dawson 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 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/36823/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36823/1//COMMIT_MSG@16 PS1, Line 16: only Stoneyridge is combo capable
So I confused some... yes SMU firmware. […]
The spec says Stoney and PF should support using either 0x10 or 0x14. The other two must use only 0x10.
https://review.coreboot.org/c/coreboot/+/36823/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36823/2//COMMIT_MSG@13 PS2, Line 13: Stoneyridge Use "Stoney Ridge" when using the code name.
https://review.coreboot.org/c/coreboot/+/36823/2//COMMIT_MSG@16 PS2, Line 16: only Stoneyridge is combo capable. I still believe this sounds incorrect.
https://review.coreboot.org/c/coreboot/+/36823/2/src/soc/amd/stoneyridge/Mak... File src/soc/amd/stoneyridge/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/36823/2/src/soc/amd/stoneyridge/Mak... PS2, Line 140: && ($(CONFIG_HAVE_MERLINFALCON_BINARIES),y)) Need to nest it