diff options
author | Devin Lu <devin.lu@quantatw.com> | 2019-10-18 17:33:27 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-10-21 03:28:28 +0000 |
commit | d88e38674b103108d0491638b506360f2204af28 (patch) | |
tree | 30103246fbd005e789df6d7a21f9be105940c08b /board/dratini | |
parent | ddc3ecad7999e77256f5915d7fb36f42bd725d1a (diff) | |
download | chrome-ec-d88e38674b103108d0491638b506360f2204af28.tar.gz |
dratini: chagne thermal configration
split thermal configration table for temp sensor 1/temp sensor 2, because
sensor location is different, they have their own table.
BUG=none
BRANCH=none
TEST=manually remove dptf control then do follwoing:
make sure system shutdown when over 85 degree with temp sensor 1.
make sure system shutdown when over 75 degree with temp sensor 2.
make sure fan works when over 25 degree with temp sensor 1.
make sure fan fully on when over 70 degree with temp sensor 1.
make sure fan works when over 25 degree with temp sensor 2.
make sure fan fully on when over 50 degree with temp sensor 2.
Change-Id: I1139f9ee8a8ef9fc67c3ba6f5cbc2b6cf265ede9
Signed-off-by: Devin Lu <Devin.Lu@quantatw.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1869171
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Diffstat (limited to 'board/dratini')
-rw-r--r-- | board/dratini/board.c | 26 |
1 files changed, 18 insertions, 8 deletions
diff --git a/board/dratini/board.c b/board/dratini/board.c index d967784222..f86436f856 100644 --- a/board/dratini/board.c +++ b/board/dratini/board.c @@ -319,17 +319,27 @@ const struct temp_sensor_t temp_sensors[] = { BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT); -/* Hatch Temperature sensors */ -/* - * TODO(b/124316213): These setting need to be reviewed and set appropriately - * for Hatch. They matter when the EC is controlling the fan as opposed to DPTF - * control. - */ +/* Dratini Temperature sensors */ const static struct ec_thermal_config thermal_a = { .temp_host = { [EC_TEMP_THRESH_WARN] = 0, [EC_TEMP_THRESH_HIGH] = C_TO_K(75), - [EC_TEMP_THRESH_HALT] = C_TO_K(80), + [EC_TEMP_THRESH_HALT] = C_TO_K(85), + }, + .temp_host_release = { + [EC_TEMP_THRESH_WARN] = 0, + [EC_TEMP_THRESH_HIGH] = C_TO_K(65), + [EC_TEMP_THRESH_HALT] = 0, + }, + .temp_fan_off = C_TO_K(25), + .temp_fan_max = C_TO_K(70), +}; + +const static struct ec_thermal_config thermal_b = { + .temp_host = { + [EC_TEMP_THRESH_WARN] = 0, + [EC_TEMP_THRESH_HIGH] = C_TO_K(75), + [EC_TEMP_THRESH_HALT] = C_TO_K(75), }, .temp_host_release = { [EC_TEMP_THRESH_WARN] = 0, @@ -345,7 +355,7 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT]; static void setup_fans(void) { thermal_params[TEMP_SENSOR_1] = thermal_a; - thermal_params[TEMP_SENSOR_2] = thermal_a; + thermal_params[TEMP_SENSOR_2] = thermal_b; } /* |