4 comments:
Patch Set #5, Line 9: In preparation to add board padmelon,
There's no reason to talk about preparing for a board. Instead, make your commit message more descriptive about Merlin Falcon.
File src/soc/amd/stoneyridge/Kconfig:
Never did a prompt, so was not aware I could. Will investigate how.
Look at line 202.
It seems that something like a symbol name of HAVE_MERLINFALCON_BINARIES would better match other coreboot Kconfig files.
Patch Set #5, Line 347: <codename>
Not sure, we are already at soc/stoneyridge. Maybe something like: […]
You forgot to address this comment https://review.coreboot.org/c/coreboot/+/33621/3/src/soc/amd/stoneyridge/Kconfig#43
Perhaps simply make this depends on SOC_AMD_STONEYRIDGE instead of defaulting to a value for MF. (I believe that should work cleanly.)
File src/soc/amd/stoneyridge/Makefile.inc:
Patch Set #3, Line 204: endif
I tried, unfortunately no. It would require dummy *_FN images, otherwise build fails.
Can you try again please? I tried it before I suggested it to you, and it still builds OK on my side when I test w/Grunt by deleting the ST fanless files.
To view, visit change 33621. To unsubscribe, or for help writing mail filters, visit settings.