Attention is currently required from: Uwe Poeche, Werner Zeh, Andrey Petrov. Uwe Poeche has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63546 )
Change subject: intel/common/block: Provide RAPL and min clock ratio switches in common ......................................................................
Patch Set 6:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63546/comment/95f405cf_902658be PS2, Line 7: provide config switches from APL specific
This could be formulated more clear, e.g.: […]
Done
https://review.coreboot.org/c/coreboot/+/63546/comment/a9f9c0a1_227c2111 PS2, Line 10: APL_SKIP_SET_POWER_LIMITS
APL_SET_MIN_CLOCK_RATIO
Done
https://review.coreboot.org/c/coreboot/+/63546/comment/c220a50f_e4952352 PS2, Line 12: Therefore they moved here with other name to common/block.
Maybe 'Move them to common code instead of having them just for one SOC'
Done
https://review.coreboot.org/c/coreboot/+/63546/comment/11221b66_f5e6078a PS2, Line 14: mainboards mc_apl1/4/5: compare cpu clock via MSR 0x198 and RAPL : settings via MSR 0x610 before and after the change.
Maybe: […]
Done
File src/soc/intel/common/block/cpu/Kconfig:
https://review.coreboot.org/c/coreboot/+/63546/comment/bed210f6_2e1fb4cd PS2, Line 144: Some Apollo Lake mainboards do not need the Running Average Power : Limits (RAPL) algorithm for a constant power management. : Set this config option to skip the RAPL configuration.
I would rephrase this text as it is not APL specific anymore
Done