diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-12-09 12:16:39 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-12-10 14:30:12 +0000 |
| commit | 92226dc6c3a769c0ba107b002ae05aee907b14e2 (patch) | |
| tree | ae175d332002d47605f526571690fe4e00577c44 /src/soc/intel/cannonlake/vr_config.c | |
| parent | dbd2362caaa36af41efb22a04bdefcc4b52639b7 (diff) | |
| download | coreboot-92226dc6c3a769c0ba107b002ae05aee907b14e2.tar.gz coreboot-92226dc6c3a769c0ba107b002ae05aee907b14e2.tar.bz2 coreboot-92226dc6c3a769c0ba107b002ae05aee907b14e2.zip | |
soc/intel/{skylake/cannonlake}: Fix bug in vr_config
The `cpu_get_power_max()` function returns the TDP in milliwatts, but
the vr_config code interprets the value in watts. Divide the value by
1000 to fix this.
This also fixes an integer overflow when `cpu_get_power_max()` returns
a value greater than 65535 (UINT16_MAX).
Change-Id: Ibe9e0db6762eee5cc363f8b371c8538eb92f6308
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60001
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Diffstat (limited to 'src/soc/intel/cannonlake/vr_config.c')
| -rw-r--r-- | src/soc/intel/cannonlake/vr_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/cannonlake/vr_config.c b/src/soc/intel/cannonlake/vr_config.c index 744564bfa7dc..7ba73eac3f25 100644 --- a/src/soc/intel/cannonlake/vr_config.c +++ b/src/soc/intel/cannonlake/vr_config.c @@ -619,7 +619,7 @@ void fill_vr_domain_config(void *params, FSP_S_CONFIG *vr_params = (FSP_S_CONFIG *)params; const struct vr_config *cfg; static uint16_t mch_id = 0, igd_id = 0; - const uint16_t tdp = cpu_get_power_max(); + const uint16_t tdp = cpu_get_power_max() / 1000; if (!mch_id) { struct device *dev = pcidev_path_on_root(SA_DEVFN_ROOT); |