diff options
| author | Ruben Rodriguez Buchillon <coconutruben@chromium.org> | 2018-03-15 17:09:40 +0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-03-16 15:56:23 -0700 |
| commit | 43d0769918a0c674423227bb9e81226a0dba6274 (patch) | |
| tree | 208e1aa1ef24f07a89ddb7c870d195a656842ecb | |
| parent | 171916a4a37c369013849b1ebc597538504254f6 (diff) | |
| download | chrome-ec-43d0769918a0c674423227bb9e81226a0dba6274.tar.gz | |
temp_metrics: use fanduty 0 to enable fan
If the fan is never enabled, temp_metrics itself has no code-path to
enable the fan. This fixes this by calling fanduty 0 in the beginning
of temp_metrics, since fanduty does explicitly enable the fan.
Note: This is a hack to avoid having to flash a new EC image. See
crrev.com/c/964037 for a more fundamental fix to the same issue.
BRANCH=link
BUG=chromium:808764
TEST=couldn't reproduce issue with this version of temp_metrics.
Change-Id: I8a9b258ba7b50cf5180497d318f8d94454dab434
Signed-off-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/964069
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Sameer Nanda <snanda@chromium.org>
| -rw-r--r-- | util/temp_metrics.conf | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/util/temp_metrics.conf b/util/temp_metrics.conf index bf10ee0907..f28127aa0b 100644 --- a/util/temp_metrics.conf +++ b/util/temp_metrics.conf @@ -336,6 +336,9 @@ script echo -n 'disabled' > /sys/class/thermal/thermal_zone1/mode fi + # Enable the fan in case no other code has enabled it. + ectool fanduty 0 + # Get list of sensors to monitor. sensor_list=$(get_sensor_list) |