diff options
| author | Bill Richardson <wfrichar@chromium.org> | 2014-12-09 14:26:14 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-12-11 23:40:10 +0000 |
| commit | 99ec74d7d414b4acd6e210798469f7699fedea01 (patch) | |
| tree | f4a34225674c1cdb4a5b6afab287de054730e809 | |
| parent | 73949da5d907689d8e3b04f09fe8d200d3b223cd (diff) | |
| download | chrome-ec-99ec74d7d414b4acd6e210798469f7699fedea01.tar.gz | |
samus: Start fans at 1000 RPM
After receiving more info from the manufacturer, it appears that
since we're using closed-loop feedback to drive the fan we can
turn it at whatever speed seems to work.
While we're bikeshedding over the startup noise, let's put the
start/min speed back to 1000RPM to help distinguish the startup
chirp from the fan-is-running-now noise.
BUG=chrome-os-partner:32757
BRANCH=ToT,Samus
TEST=make buildall -j
Watch fan speeds while doing things. It still makes noise, but it's
quieter.
Change-Id: I5c21bf9021e4110f31c6dded78852347c4eb6119
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/234755
Reviewed-by: Sameer Nanda <snanda@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
| -rw-r--r-- | board/samus/board.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/board/samus/board.c b/board/samus/board.c index d68b2bb2e6..bff47a13a4 100644 --- a/board/samus/board.c +++ b/board/samus/board.c @@ -107,16 +107,16 @@ BUILD_ASSERT(ARRAY_SIZE(pwm_channels) == PWM_CH_COUNT); /* Physical fans. These are logically separate from pwm_channels. */ const struct fan_t fans[] = { {.flags = FAN_USE_RPM_MODE, - .rpm_min = 2286, - .rpm_start = 3090, + .rpm_min = 1000, + .rpm_start = 1000, .rpm_max = 6350, .ch = 2, .pgood_gpio = -1, .enable_gpio = -1, }, {.flags = FAN_USE_RPM_MODE, - .rpm_min = 2286, - .rpm_start = 3090, + .rpm_min = 1000, + .rpm_start = 1000, .rpm_max = 6350, .ch = 3, .pgood_gpio = -1, @@ -188,7 +188,7 @@ BUILD_ASSERT(ARRAY_SIZE(als) == ALS_COUNT); */ struct ec_thermal_config thermal_params[] = { /* Only the AP affects the thermal limits and fan speed. */ - {{C_TO_K(95), C_TO_K(97), C_TO_K(99)}, C_TO_K(43), C_TO_K(75)}, + {{C_TO_K(95), C_TO_K(97), C_TO_K(99)}, C_TO_K(43), C_TO_K(85)}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, {{0, 0, 0}, 0, 0}, |