summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Feng <ian_feng@compal.corp-partner.google.com>2024-04-19 14:37:07 +0800
committerFelix Held <felix-coreboot@felixheld.de>2024-04-22 13:19:14 +0000
commit835ed7a7ab5cb00136c45c1964c859dfe32df6fa (patch)
tree0e5788e1c15b4cca2b2a701b0017d674bf47c0d8
parent41ba11229a80eb19d97c8052aff1861478ee2486 (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.cb64
-rw-r--r--src/mainboard/google/brya/variants/craaskov/variant.c7
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;
+
}