diff options
author | Ian Feng <ian_feng@compal.corp-partner.google.com> | 2024-04-19 14:37:07 +0800 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2024-04-22 13:19:14 +0000 |
commit | 835ed7a7ab5cb00136c45c1964c859dfe32df6fa (patch) | |
tree | 0e5788e1c15b4cca2b2a701b0017d674bf47c0d8 | |
parent | 41ba11229a80eb19d97c8052aff1861478ee2486 (diff) |
mb/google/nissa/var/craaskov: modify 6W and 15W DPTF parameters
The DPTF parameters were defined by the thermal team.
Based on thermal table in 330817690#comment33.
Set 6w "tcc_offset" to "15" by fw_config.
BUG=b:330817690, b:290705146
BRUNCH=firmware-nissa-15217.B
TEST=emerge-nissa coreboot chromeos-bootimage
Change-Id: I19100d960919dc3087fd067c24659de467eea276
Signed-off-by: Ian Feng <ian_feng@compal.corp-partner.google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81997
Reviewed-by: Eric Lai <ericllai@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
-rw-r--r-- | src/mainboard/google/brya/variants/craaskov/overridetree.cb | 64 | ||||
-rw-r--r-- | src/mainboard/google/brya/variants/craaskov/variant.c | 7 |
2 files changed, 38 insertions, 33 deletions
diff --git a/src/mainboard/google/brya/variants/craaskov/overridetree.cb b/src/mainboard/google/brya/variants/craaskov/overridetree.cb index eee54026fe..476e642660 100644 --- a/src/mainboard/google/brya/variants/craaskov/overridetree.cb +++ b/src/mainboard/google/brya/variants/craaskov/overridetree.cb @@ -126,8 +126,6 @@ chip soc/intel/alderlake register "usb2_ports[7]" = "USB2_PORT_MID(OC_SKIP)" # Bluetooth port for CNVi WLAN register "usb2_ports[9]" = "USB2_PORT_MID(OC_SKIP)" # Bluetooth port for CNVi WLAN - register "tcc_offset" = "8" - register "power_limits_config[ADL_N_041_6W_CORE]" = "{ .tdp_pl1_override = 20, .tdp_pl2_override = 25, @@ -153,56 +151,56 @@ chip soc/intel/alderlake [0] = { .target = DPTF_CPU, .thresholds = { - TEMP_PCT(70, 100), - TEMP_PCT(60, 65), - TEMP_PCT(42, 60), - TEMP_PCT(39, 55), - TEMP_PCT(38, 50), - TEMP_PCT(35, 43), - TEMP_PCT(31, 30), + TEMP_PCT(95, 100), + TEMP_PCT(83, 64), + TEMP_PCT(72, 59), + TEMP_PCT(65, 54), + TEMP_PCT(52, 49), + TEMP_PCT(42, 43), + TEMP_PCT(38, 29), } }, [1] = { .target = DPTF_TEMP_SENSOR_0, .thresholds = { TEMP_PCT(60, 100), - TEMP_PCT(55, 65), - TEMP_PCT(52, 60), - TEMP_PCT(50, 55), - TEMP_PCT(48, 50), + TEMP_PCT(55, 64), + TEMP_PCT(52, 59), + TEMP_PCT(50, 54), + TEMP_PCT(48, 49), TEMP_PCT(45, 43), - TEMP_PCT(41, 30), + TEMP_PCT(41, 29), } } }" ## Passive Policy register "policies.passive" = "{ - [0] = DPTF_PASSIVE(CPU, CPU, 95, 5000), - [1] = DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 70, 5000), - [2] = DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 70, 5000), - [3] = DPTF_PASSIVE(CPU, TEMP_SENSOR_2, 70, 5000), + [0] = DPTF_PASSIVE(CPU, CPU, 85, 5000), + [1] = DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 65, 5000), + [2] = DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 65, 5000), + [3] = DPTF_PASSIVE(CPU, TEMP_SENSOR_2, 65, 5000), }" ## Critical Policy register "policies.critical" = "{ - [0] = DPTF_CRITICAL(CPU, 100, SHUTDOWN), - [1] = DPTF_CRITICAL(TEMP_SENSOR_0, 95, SHUTDOWN), - [2] = DPTF_CRITICAL(TEMP_SENSOR_1, 95, SHUTDOWN), - [3] = DPTF_CRITICAL(TEMP_SENSOR_2, 95, SHUTDOWN), + [0] = DPTF_CRITICAL(CPU, 110, SHUTDOWN), + [1] = DPTF_CRITICAL(TEMP_SENSOR_0, 75, SHUTDOWN), + [2] = DPTF_CRITICAL(TEMP_SENSOR_1, 75, SHUTDOWN), + [3] = DPTF_CRITICAL(TEMP_SENSOR_2, 75, SHUTDOWN), }" register "controls.power_limits" = "{ .pl1 = { .min_power = 6000, - .max_power = 20000, + .max_power = 6000, .time_window_min = 28 * MSECS_PER_SEC, .time_window_max = 28 * MSECS_PER_SEC, .granularity = 500 }, .pl2 = { - .min_power = 25000, - .max_power = 25000, + .min_power = 6000, + .max_power = 6000, .time_window_min = 32 * MSECS_PER_SEC, .time_window_max = 32 * MSECS_PER_SEC, .granularity = 500 @@ -275,7 +273,7 @@ chip soc/intel/alderlake ## Passive Policy register "policies.passive" = "{ - [0] = DPTF_PASSIVE(CPU, CPU, 95, 5000), + [0] = DPTF_PASSIVE(CPU, CPU, 90, 5000), [1] = DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 70, 5000), [2] = DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 70, 5000), [3] = DPTF_PASSIVE(CPU, TEMP_SENSOR_2, 70, 5000), @@ -283,23 +281,23 @@ chip soc/intel/alderlake ## Critical Policy register "policies.critical" = "{ - [0] = DPTF_CRITICAL(CPU, 100, SHUTDOWN), - [1] = DPTF_CRITICAL(TEMP_SENSOR_0, 95, SHUTDOWN), - [2] = DPTF_CRITICAL(TEMP_SENSOR_1, 95, SHUTDOWN), - [3] = DPTF_CRITICAL(TEMP_SENSOR_2, 95, SHUTDOWN), + [0] = DPTF_CRITICAL(CPU, 105, SHUTDOWN), + [1] = DPTF_CRITICAL(TEMP_SENSOR_0, 80, SHUTDOWN), + [2] = DPTF_CRITICAL(TEMP_SENSOR_1, 80, SHUTDOWN), + [3] = DPTF_CRITICAL(TEMP_SENSOR_2, 80, SHUTDOWN), }" register "controls.power_limits" = "{ .pl1 = { .min_power = 15000, - .max_power = 20000, + .max_power = 15000, .time_window_min = 28 * MSECS_PER_SEC, .time_window_max = 28 * MSECS_PER_SEC, .granularity = 500 }, .pl2 = { - .min_power = 35000, - .max_power = 35000, + .min_power = 15000, + .max_power = 15000, .time_window_min = 32 * MSECS_PER_SEC, .time_window_max = 32 * MSECS_PER_SEC, .granularity = 500 diff --git a/src/mainboard/google/brya/variants/craaskov/variant.c b/src/mainboard/google/brya/variants/craaskov/variant.c index cee0308aaa..4a06f10f13 100644 --- a/src/mainboard/google/brya/variants/craaskov/variant.c +++ b/src/mainboard/google/brya/variants/craaskov/variant.c @@ -3,6 +3,7 @@ #include <baseboard/variants.h> #include <boardid.h> #include <device/device.h> +#include <fw_config.h> #include <sar.h> const char *get_wifi_sar_cbfs_filename(void) @@ -56,4 +57,10 @@ void variant_update_soc_chip_config(struct soc_intel_alderlake_config *config) config->ext_fivr_settings.vnn_icc_max_ma = 500; printk(BIOS_INFO, "Configured external FIVR\n"); } + + if (fw_config_probe(FW_CONFIG(THERMAL_SOLUTION, THERMAL_SOLUTION_6W))) + config->tcc_offset = 15; + if (fw_config_probe(FW_CONFIG(THERMAL_SOLUTION, THERMAL_SOLUTION_15W))) + config->tcc_offset = 8; + } |