Attention is currently required from: Angel Pons, Patrick Rudolph.
Anastasios Koutian has posted comments on this change by Anastasios Koutian. ( https://review.coreboot.org/c/coreboot/+/83269?usp=email )
Change subject: cpu/intel/model_206ax: Allow PL1/PL2 configuration ......................................................................
Patch Set 1:
(1 comment)
File src/cpu/intel/model_206ax/chip.h:
https://review.coreboot.org/c/coreboot/+/83269/comment/79dc211e_95e59a04?usp... : PS1, Line 46: int pl1; /* Long-term power limit*/ : int pl2; /* Short-term power limit*/
Huh, that's a pretty small unit. Out of curiosity, any reason to use microwatts instead of watts? […]
I chose this in order to be consistent with the linux kernel's intel-rapl-msr interface (/sys/devices/virtual/powercap/intel-rapl) which allows to configure these limits from userspace. Also, Intel's manuals commonly use microjoules for MSR energy measurements, so microwatts made sense.