From 88503ab4ec8f68c19d36ec4b6d3516848d71404a Mon Sep 17 00:00:00 2001 From: Bill Richardson Date: Fri, 1 Nov 2013 11:11:09 -0700 Subject: Provide multiple fan support within the EC itself This adds explicit "int fan" args to the exported functions from common/fan.c: fan_set_percent_needed() and fan_percent_to_rpm(). Within that file, multiple fans are handled independently. This is not complete, though. Host commands and sysjump support still only handle a single fan, so at the moment multiple fans are treated identically in those cases. BUG=chrome-os-partner:23530 BRANCH=none TEST=manual All boards build, "make runtests" passes. On a multi-fan system, the EC command "faninfo" displays multiple results: > faninfo Fan 0 Actual: 0 rpm Fan 0 Target: 0 rpm Fan 0 Duty: 0% Fan 0 Status: 0 (not spinning) Fan 0 Mode: rpm Fan 0 Auto: yes Fan 0 Enable: yes Fan 1 Actual: 0 rpm Fan 1 Target: 0 rpm Fan 1 Duty: 0% Fan 1 Status: 0 (not spinning) Fan 1 Mode: rpm Fan 1 Auto: no Fan 1 Enable: no > and the "fanduty", "fanset", and "fanauto" all require the fan number as the first arg: > fanduty 0 30 Setting fan 0 duty cycle to 30% > fanset 1 2000 Setting fan 1 rpm target to 2000 > fanauto 0 > fanauto 1 On single-fan systems, there is no visible change. Change-Id: Idb8b818122e157960d56779b2a86e5ba433bee1b Signed-off-by: Bill Richardson Reviewed-on: https://chromium-review.googlesource.com/175368 Reviewed-by: Randall Spangler --- test/thermal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test/thermal.c') diff --git a/test/thermal.c b/test/thermal.c index deaac2f631..10662ffd7d 100644 --- a/test/thermal.c +++ b/test/thermal.c @@ -62,7 +62,7 @@ void host_throttle_cpu(int throttled) host_throttled = throttled; } -void fan_set_percent_needed(int pct) +void fan_set_percent_needed(int fan, int pct) { fan_pct = pct; } -- cgit v1.2.1