Attention is currently required from: Sean Rhodes, Benjamin Doron, Christian Walter, Maximilian Brune, Lean Sheng Tan.
Hello build bot (Jenkins), Benjamin Doron, Christian Walter, Maximilian Brune, Lean Sheng Tan,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/74122
to look at the new patch set (#2).
Change subject: [WIP] mb/prodrive/atlas: Put options in CFR cbtable ......................................................................
[WIP] mb/prodrive/atlas: Put options in CFR cbtable
PLEASE REMOVE THE TESTONLY OPTIONS IF BUILDING THIS FOR PRODUCTION
YOU HAVE BEEN WARNED
Open questions:
- What to do with payload-specific options? The payload needs to know about them anyway, so describing them in coreboot kind of defeats the purpose of avoiding duplication. In our case, we can handle the profile values in the payload, which coreboot can send through CFR.
- Hook up options using the option API
- Figure out the details for some options (e.g. LLC dead line alloc)
- Figure out if any options are missing, and if the defaults are OK
The plan is to avoid having to specify the options in both edk2 and coreboot.
Change-Id: I47585a9a6f94ab5005f2ab63a0df267c0caef231 Signed-off-by: Angel Pons th3fanbus@gmail.com --- M src/mainboard/prodrive/atlas/Kconfig M src/mainboard/prodrive/atlas/Makefile.inc A src/mainboard/prodrive/atlas/cfr.c 3 files changed, 289 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/74122/2