diff options
author | Kornel Dulęba <korneld@google.com> | 2023-04-14 13:23:57 +0000 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-21 22:00:10 +0000 |
commit | efc5344e54472f87fb3b16ba891f2a683dd57bcc (patch) | |
tree | 5ff7a72b61492021316c349dbc53462ed2098ae6 | |
parent | e9b75e1760a796676e2425c7694c0eb617542ba1 (diff) | |
download | chrome-ec-efc5344e54472f87fb3b16ba891f2a683dd57bcc.tar.gz |
whiterun: Adjust CPU fan thermal parameters
The difference between temp_fan_off and temp_fan_max was set to 6°C.
Consequently, a one-degree difference in CPU temperature would cause
a significant change in fan RPM. This results in noticeable noise
variations when the CPU is subjected to a specific load.
To alleviate this issue, decrease the temp_fan_off by 4°C, which,
in turn, reduces the steepness of the temperature-to-RPM relationship.
BUG=b:276577324
BRANCH=none
TEST=Run webGL aquarium with various number of fish
Change-Id: I72ec924e61a68e61a9c7f9721b7df15d18aaa2ee
Signed-off-by: Kornel Dulęba <korneld@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4427797
Reviewed-by: Diana Z <dzigterman@chromium.org>
Reviewed-by: Tim Van Patten <timvp@google.com>
Reviewed-by: Radoslaw Biernacki <biernacki@google.com>
(cherry picked from commit 0852c76440bade57d203bc0f985de56a548c530c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4456123
Commit-Queue: Chao Gui <chaogui@google.com>
-rw-r--r-- | zephyr/program/skyrim/winterhold/src/thermal.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/zephyr/program/skyrim/winterhold/src/thermal.c b/zephyr/program/skyrim/winterhold/src/thermal.c index 82b802f2ee..0ccd9dd9e2 100644 --- a/zephyr/program/skyrim/winterhold/src/thermal.c +++ b/zephyr/program/skyrim/winterhold/src/thermal.c @@ -138,7 +138,7 @@ __maybe_unused static const struct ec_thermal_config fan_soc_laptop = */ #define FAN_CPU_DESKTOP_LID_OPEN \ { \ - .temp_fan_off = C_TO_K(76), .temp_fan_max = C_TO_K(82), \ + .temp_fan_off = C_TO_K(72), .temp_fan_max = C_TO_K(82), \ } __maybe_unused static const struct ec_thermal_config fan_cpu_desktop_lid_open = FAN_CPU_DESKTOP_LID_OPEN; @@ -148,7 +148,7 @@ __maybe_unused static const struct ec_thermal_config fan_cpu_desktop_lid_open = */ #define FAN_CPU_DESKTOP_LID_CLOSE \ { \ - .temp_fan_off = C_TO_K(76), .temp_fan_max = C_TO_K(82), \ + .temp_fan_off = C_TO_K(72), .temp_fan_max = C_TO_K(82), \ } __maybe_unused static const struct ec_thermal_config fan_cpu_desktop_lid_close = FAN_CPU_DESKTOP_LID_CLOSE; @@ -158,7 +158,7 @@ __maybe_unused static const struct ec_thermal_config fan_cpu_desktop_lid_close = */ #define FAN_CPU_LAPTOP \ { \ - .temp_fan_off = C_TO_K(72), .temp_fan_max = C_TO_K(78), \ + .temp_fan_off = C_TO_K(68), .temp_fan_max = C_TO_K(78), \ } __maybe_unused static const struct ec_thermal_config fan_cpu_laptop = FAN_CPU_LAPTOP; |