From 8711d7644a67aac6396cb8ff0c983ee581910d7a Mon Sep 17 00:00:00 2001 From: ChromeOS Developer Date: Thu, 12 Dec 2013 15:41:15 -0800 Subject: [PATCH] Lower critical temps below CONIFG_PECI_TJMAX This lowers, the WARN, HIGH, and HALT temp thresholds for x86 boards to below their CONFIG_PECI_TJMAX value. Also lowers the FAN_MIN and FAN_MAX temps by 5 degrees on Haswell boards to compensate for lowering TJ_MAX by 5 degrees in an earlier patch. BUG=chrome-os-partner:24455 BRANCH=none TEST=Manual. Run boards without a fan and without any host-side throttling. Verify that board either reaches a steady state temp due to throttling or hits SHUTDOWN and turns off before EC reset is triggered. Change-Id: I499baa0b4100201525e69752af3465feb592262c Signed-off-by: Dave Parker Reviewed-on: https://chromium-review.googlesource.com/179886 Reviewed-by: Bill Richardson --- board/falco/board.c | 2 +- board/link/board.c | 2 +- board/peppy/board.c | 2 +- board/samus/board.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/board/falco/board.c b/board/falco/board.c index 3b50447802..bfec566608 100644 --- a/board/falco/board.c +++ b/board/falco/board.c @@ -224,7 +224,7 @@ BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT); */ struct ec_thermal_config thermal_params[] = { /* Only the AP affects the thermal limits and fan speed. */ - { {C_TO_K(100), C_TO_K(114), C_TO_K(110)}, C_TO_K(60), C_TO_K(90)}, + { {C_TO_K(95), C_TO_K(97), C_TO_K(99)}, C_TO_K(55), C_TO_K(85)}, { {0, 0, 0}, 0, 0}, { {0, 0, 0}, 0, 0}, { {0, 0, 0}, 0, 0}, diff --git a/board/link/board.c b/board/link/board.c index 60b654c2b0..a080345764 100644 --- a/board/link/board.c +++ b/board/link/board.c @@ -247,7 +247,7 @@ struct ec_thermal_config thermal_params[] = { {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, /* Only the AP affects the thermal limits and fan speed. */ - {{C_TO_K(100), C_TO_K(114), C_TO_K(110)}, C_TO_K(60), C_TO_K(90)}, + {{C_TO_K(100), C_TO_K(102), C_TO_K(104)}, C_TO_K(60), C_TO_K(90)}, }; BUILD_ASSERT(ARRAY_SIZE(thermal_params) == TEMP_SENSOR_COUNT); diff --git a/board/peppy/board.c b/board/peppy/board.c index 11f4e44f5d..35f7d5002b 100644 --- a/board/peppy/board.c +++ b/board/peppy/board.c @@ -217,7 +217,7 @@ BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT); */ struct ec_thermal_config thermal_params[] = { /* Only the AP affects the thermal limits and fan speed. */ - {{C_TO_K(100), C_TO_K(114), C_TO_K(110)}, C_TO_K(60), C_TO_K(90)}, + {{C_TO_K(95), C_TO_K(97), C_TO_K(99)}, C_TO_K(55), C_TO_K(85)}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, diff --git a/board/samus/board.c b/board/samus/board.c index f943250f18..bca84b61c7 100644 --- a/board/samus/board.c +++ b/board/samus/board.c @@ -279,7 +279,7 @@ BUILD_ASSERT(ARRAY_SIZE(als) == ALS_COUNT); */ struct ec_thermal_config thermal_params[] = { /* Only the AP affects the thermal limits and fan speed. */ - {{C_TO_K(100), C_TO_K(114), C_TO_K(110)}, C_TO_K(60), C_TO_K(90)}, + {{C_TO_K(95), C_TO_K(97), C_TO_K(99)}, C_TO_K(55), C_TO_K(85)}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0},