From efc5344e54472f87fb3b16ba891f2a683dd57bcc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kornel=20Dul=C4=99ba?= Date: Fri, 14 Apr 2023 13:23:57 +0000 Subject: whiterun: Adjust CPU fan thermal parameters MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4427797 Reviewed-by: Diana Z Reviewed-by: Tim Van Patten Reviewed-by: Radoslaw Biernacki (cherry picked from commit 0852c76440bade57d203bc0f985de56a548c530c) Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4456123 Commit-Queue: Chao Gui --- zephyr/program/skyrim/winterhold/src/thermal.c | 6 +++--- 1 file 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; -- cgit v1.2.1