diff options
| author | Nico Huber <nico.huber@secunet.com> | 2021-07-27 12:43:25 +0000 |
|---|---|---|
| committer | Paul Fagerburg <pfagerburg@chromium.org> | 2021-08-04 15:15:35 +0000 |
| commit | 6ac8a9f826072b72b064e5885bb8cda65c92a7bb (patch) | |
| tree | c47bad496e1e53379cb17db15c449711c03b505d | |
| parent | b06d847ea905e5e78523a5833c1a55e77c83a424 (diff) | |
| download | coreboot-6ac8a9f826072b72b064e5885bb8cda65c92a7bb.tar.gz coreboot-6ac8a9f826072b72b064e5885bb8cda65c92a7bb.tar.bz2 coreboot-6ac8a9f826072b72b064e5885bb8cda65c92a7bb.zip | |
soc/intel/cannonlake/vr_config: Print configured values
These values are quite important and our default tables sometimes have
holes. We should at least make it visible what the resulting settings
are.
Change-Id: Ic716d073da1c2638c4b16f2eac01b83a0768d22f
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56659
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | src/soc/intel/cannonlake/vr_config.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/soc/intel/cannonlake/vr_config.c b/src/soc/intel/cannonlake/vr_config.c index fa96b6fddecd..744564bfa7dc 100644 --- a/src/soc/intel/cannonlake/vr_config.c +++ b/src/soc/intel/cannonlake/vr_config.c @@ -647,17 +647,27 @@ void fill_vr_domain_config(void *params, vr_params->Psi4Enable[domain] = cfg->psi4enable; vr_params->ImonSlope[domain] = cfg->imon_slope; vr_params->ImonOffset[domain] = cfg->imon_offset; + printk(BIOS_INFO, "VR config[%d]:\n", domain); + printk(BIOS_INFO, " Psi1Threshold: %u\n", cfg->psi1threshold); + printk(BIOS_INFO, " Psi2Threshold: %u\n", cfg->psi2threshold); + printk(BIOS_INFO, " Psi3Threshold: %u\n", cfg->psi3threshold); + printk(BIOS_INFO, " Psi3Enable: %u\n", cfg->psi3enable); + printk(BIOS_INFO, " Psi4Enable: %u\n", cfg->psi4enable); + printk(BIOS_INFO, " ImonSlope: %u\n", cfg->imon_slope); + printk(BIOS_INFO, " ImonOffset: %u\n", cfg->imon_offset); /* If board provided non-zero value, use it. */ if (cfg->voltage_limit) vr_params->VrVoltageLimit[domain] = cfg->voltage_limit; else vr_params->VrVoltageLimit[domain] = get_sku_voltagelimit(domain); + printk(BIOS_INFO, " VrVoltageLimit: %u\n", vr_params->VrVoltageLimit[domain]); if (cfg->icc_max) vr_params->IccMax[domain] = cfg->icc_max; else vr_params->IccMax[domain] = get_sku_icc_max(domain, tdp, mch_id, igd_id); + printk(BIOS_INFO, " IccMax: %u\n", vr_params->IccMax[domain]); if (cfg->ac_loadline) vr_params->AcLoadline[domain] = cfg->ac_loadline; @@ -665,6 +675,7 @@ void fill_vr_domain_config(void *params, vr_params->AcLoadline[domain] = load_table(vr_config_ll, ARRAY_SIZE(vr_config_ll), domain, tdp, mch_id); + printk(BIOS_INFO, " AcLoadline: %u\n", vr_params->AcLoadline[domain]); if (cfg->dc_loadline) vr_params->DcLoadline[domain] = cfg->dc_loadline; @@ -672,8 +683,10 @@ void fill_vr_domain_config(void *params, vr_params->DcLoadline[domain] = load_table(vr_config_ll, ARRAY_SIZE(vr_config_ll), domain, tdp, mch_id); + printk(BIOS_INFO, " DcLoadline: %u\n", vr_params->DcLoadline[domain]); vr_params->TdcEnable[domain] = !cfg->tdc_disable; + printk(BIOS_INFO, " TdcEnable: %u\n", vr_params->TdcEnable[domain]); if (cfg->tdc_powerlimit) vr_params->TdcPowerLimit[domain] = cfg->tdc_powerlimit; @@ -681,4 +694,5 @@ void fill_vr_domain_config(void *params, vr_params->TdcPowerLimit[domain] = load_table(vr_config_tdc, ARRAY_SIZE(vr_config_tdc), domain, tdp, mch_id); + printk(BIOS_INFO, " TdcPowerLimit: %u\n", vr_params->TdcPowerLimit[domain]); } |