summaryrefslogtreecommitdiff
path: root/board/genesis/board.h
diff options
context:
space:
mode:
authorRichard Yeh <rcy@google.com>2022-11-28 04:08:24 +0000
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2023-01-31 04:00:20 +0000
commit41497123d484b33620f4adc9387f65722797b0f0 (patch)
tree35112f6342b30cc0d157f8d7cf44baee0e65fb2c /board/genesis/board.h
parentda770c44e6481758085af8d17efe731622a92c66 (diff)
downloadchrome-ec-firmware-puff-13324.B-master.tar.gz
{ambassador,genesis,moonbuggy,scout}: Increase fan to reduce throttling.firmware-puff-13324.B-master
Chromebox for Meetings (CfM) devices are videoconferencing appliances. Discovering that a significant fraction of our fleet was throttling the CPU due to temperature, resulting in bad video quality, we have decided that these devices should never throttle. The present change reduces the temperature thresholds at which we increase fan speeds. For most of the time, when the CfM is idle, we expect no or little change in fan speeds. During calls, the CfM CPU can exceed 60 C, and the sensors controlling the fan are typically 10-20 degrees C cooler than the CPU, so we set the maximum-fan temperature to 50-55 C in an attempt to arrest any temperature increase before the CPU reaches throttling temperatures. We add fan tables to enable hysteresis in the fan speed setting. Fan RPMs are chosen to span the min-to-max range, while avoiding 50 Hz and 60 Hz line frequencies. fizz and kalista boards are shared with non-CfM versions, while the other boards are CfM-only. Since those boards are several years old, we plan to build these firmware changes only into CfM-branch releases, not all fizz and kalista devices. Additional details at go/cfm-overheating . Temperature traces with load testing on both top-of-tree and firmware with this change are shown at go/cfm-overheating#heading=h.ykpzxotgnu2j BRANCH=update-fan-tables BUG=b:252966838,b:191187610,chromium:1383859 TEST=Confirmed validity of settings with `ectool thermalset`. Built firmware, flashed to endeavour and excelsior (fizz), verified that fan turns on earlier. This does not completely prevent throttling from 47.25 W to 15.00 W on Kaby Lake boards (endeavour, fizz, kalista), but it does prevent dropping below 15.00 W. Comet Lake boards (ambassador, genesis, moonbuggy, scout) start at 15.00 W and rarely throttle anyway. Signed-off-by: Richard Yeh <rcy@google.com> Change-Id: I7f23583ca1b85d3b8b9da43918880f68b41538c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4060610 Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com> Reviewed-by: Abe Levkoy <alevkoy@chromium.org> Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org> (cherry picked from commit 5e2e4a52a1b2d5231285a7ff5081f291b8adad1a) Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4200717 Reviewed-by: Kyle Williams <kdgwill@chromium.org>
Diffstat (limited to 'board/genesis/board.h')
-rw-r--r--board/genesis/board.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/board/genesis/board.h b/board/genesis/board.h
index eba2fd52e4..1688152739 100644
--- a/board/genesis/board.h
+++ b/board/genesis/board.h
@@ -106,6 +106,7 @@
#define CONFIG_FANS 1
#undef CONFIG_FAN_INIT_SPEED
#define CONFIG_FAN_INIT_SPEED 0
+#define CONFIG_FAN_RPM_CUSTOM
#define CONFIG_TEMP_SENSOR
#define CONFIG_TEMP_SENSOR_POWER_GPIO GPIO_EN_ROA_RAILS
#define CONFIG_THERMISTOR