mb/google/octopus/variants/baseboard: Udpate CPU critical temp

Observed thermal shutdown initiated by DPTF due to CPU temperature
reaching critical temperature trip value. During stress testing with
busty workloads like Octane, Aquarium on open yorp board with heat sink,
sometime CPU temperature spikes till 99 degree Celsius and DPTF
initiates system shutdown. With reference to previous APL/reef/coral
platforms, this updates 105 degree Celsius for the CPU critical
temperature trip value to avoid shutdown. This patch also updates power
limit1 value to avoid the abrupt thermal shutdown by DPTF.

BUG=b:79779737
BRANCH=None
TEST=Build coreboot for Octopus board.

Change-Id: Icd786d3c9b5f7c733dac3fd3e22579e2434058a6
Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar@intel.com>
Reviewed-on: https://review.coreboot.org/27294
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Justin TerAvest <teravest@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Sumeet Pawnikar 2018-06-29 23:27:40 +05:30 committed by Furquan Shaikh
parent e577168ae3
commit ebde659503
2 changed files with 11 additions and 5 deletions

View File

@ -42,10 +42,11 @@ chip soc/intel/apollolake
register "gpe0_dw2" = "PMC_GPE_N_95_64"
register "gpe0_dw3" = "PMC_GPE_N_63_32"
# PL1 override 12000 mW: Due to error in the energy calculation for
# PL1 override 8000 mW: Due to error in the energy calculation for
# current VR solution. Experiments show that SoC TDP max (6W) can
# be reached when RAPL PL1 is set to 12W.
register "tdp_pl1_override_mw" = "12000"
# be reached when RAPL PL1 is set to 8W.
# TODO: Need to tune this value on closed chassis system.
register "tdp_pl1_override_mw" = "8000"
# Set RAPL PL2 to 15W.
register "tdp_pl2_override_mw" = "15000"

View File

@ -15,7 +15,11 @@
/* Below values might change after Thermal Tuning. */
#define DPTF_CPU_PASSIVE 90
#define DPTF_CPU_CRITICAL 99
/* During stress testing with busty workloads, sometime CPU temperature
spikes till 99C and DPTF initiates shutdown. With reference to previous
APL/reef/Coral platforms, we had used 105C for this CPU critical
temperature trip value to avoid abrupt thermal shutdown. */
#define DPTF_CPU_CRITICAL 105
#define DPTF_TSR0_SENSOR_ID 0
#define DPTF_TSR0_SENSOR_NAME "Battery"
@ -65,7 +69,8 @@ Name (MPPC, Package ()
Package () { /* Power Limit 1 */
0, /* PowerLimitIndex, 0 for Power Limit 1 */
3000, /* PowerLimitMinimum */
12000, /* PowerLimitMaximum */
/* TODO: Need to tune this value on closed chassis system. */
8000, /* PowerLimitMaximum */
1000, /* TimeWindowMinimum */
1000, /* TimeWindowMaximum */
200 /* StepSize */