From 43d0769918a0c674423227bb9e81226a0dba6274 Mon Sep 17 00:00:00 2001 From: Ruben Rodriguez Buchillon Date: Thu, 15 Mar 2018 17:09:40 +0800 Subject: 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 Reviewed-on: https://chromium-review.googlesource.com/964069 Commit-Ready: ChromeOS CL Exonerator Bot Reviewed-by: Sameer Nanda --- util/temp_metrics.conf | 3 +++ 1 file changed, 3 insertions(+) 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) -- cgit v1.2.1