Attention is currently required from: Anastasios Koutian.
Angel Pons has posted comments on this change by Anastasios Koutian. ( https://review.coreboot.org/c/coreboot/+/83271?usp=email )
Change subject: cpu/intel/model_206ax: Allow turbo boost ratio limit configuration ......................................................................
Patch Set 2:
(2 comments)
Patchset:
PS2: Also see CB:42547 which admittedly I haven't really taken care of.
Commit Message:
https://review.coreboot.org/c/coreboot/+/83271/comment/badb8fe0_42df4d8c?usp... : PS2, Line 9: Tested on ThinkPad T420 with the i7-3940XM. Hmmm, I don't see any changes to the T420. If these settings are meant to be user-configurable, perhaps they should be Kconfig options and/or CMOS options.
The option API's "get" functions take a fallback value as parameter, which is returned when the selected option backend fails to read the option's value (the "null" backend always fails). This fallback value can be a Kconfig option, so a common pattern is the following:
``` const unsigned int some_value = get_uint_option("some_option", CONFIG_SOME_OPTION); ```