diff options
author | Rob Barnes <robbarnes@google.com> | 2021-06-11 12:17:09 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-06-11 21:53:18 +0000 |
commit | b09c7368862c4b09c31ffd7999c44a91ff9c8792 (patch) | |
tree | e8cee0be6906589ad29a14543fbb357cfa2cd82b /baseboard | |
parent | 5030ac7c16fef646cf767ea7cae7f962a64c909c (diff) | |
download | chrome-ec-b09c7368862c4b09c31ffd7999c44a91ff9c8792.tar.gz |
guybrush: Add fan thresholds to CPU temp sensor
If the SOC temp sensor fails for any reason, the CPU temp sensor should
be used as a backup to trigger the fan. Mostly guessing on thresholds
here. These values should be adjusted during thremal tuning.
BUG=b:190831599
TEST=Disable SOC sensor, see fan trigger when CPU temp reached 60
BRANCH=None
Signed-off-by: Rob Barnes <robbarnes@google.com>
Change-Id: I6f95dccd07bc588e629b8447be3149edaf2611e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2956830
Reviewed-by: Diana Z <dzigterman@chromium.org>
Diffstat (limited to 'baseboard')
-rw-r--r-- | baseboard/guybrush/baseboard.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/baseboard/guybrush/baseboard.c b/baseboard/guybrush/baseboard.c index b034b6f633..1eb6207013 100644 --- a/baseboard/guybrush/baseboard.c +++ b/baseboard/guybrush/baseboard.c @@ -274,8 +274,12 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = { .temp_host_release = { [EC_TEMP_THRESH_HIGH] = C_TO_K(80), }, - .temp_fan_off = 0, - .temp_fan_max = 0, + /* + * CPU temp sensor fan thresholds are high because they are a + * backup for the SOC temp sensor fan thresholds. + */ + .temp_fan_off = C_TO_K(60), + .temp_fan_max = C_TO_K(90), }, /* * Note: Leave ambient entries at 0, both as it does not represent a |