diff options
| author | Nico Huber <nico.huber@secunet.com> | 2019-08-06 18:05:50 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-09-30 11:52:31 +0000 |
| commit | b56fcfe9b5b49915c1ae00ac06421fca7a7e2308 (patch) | |
| tree | ac1304b95952f50017e0f8ddc0f903a1c0acc676 /src/drivers/i2c/lm96000/chip.h | |
| parent | 9a940bf295337153728e69afdbfee8776170b7bb (diff) | |
drivers/i2c/lm96000: Add more settings for fan control
Allow to set a lower temperature limit, as the currently hard-coded
25C may be to low for a given temperature sensor. Also enable smoo-
thing, currently hard-coded to the maximum interval of 35s, and set
the hysteresis value.
Change-Id: I5fde1cf909e8fbbaf8a345790b00c58a73c19ef8
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35475
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'src/drivers/i2c/lm96000/chip.h')
| -rw-r--r-- | src/drivers/i2c/lm96000/chip.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/drivers/i2c/lm96000/chip.h b/src/drivers/i2c/lm96000/chip.h index ab782dbbb5..cbf8601146 100644 --- a/src/drivers/i2c/lm96000/chip.h +++ b/src/drivers/i2c/lm96000/chip.h @@ -93,6 +93,7 @@ struct lm96000_fan_config { }; struct lm96000_temp_zone { + u8 low_temp; /* temperature for min. duty cycle (in °C) */ u8 target_temp; /* temperature for 100% duty cycle (in °C) */ u8 panic_temp; /* temperature for 100% duty cycle on all fans */ @@ -100,9 +101,12 @@ struct lm96000_temp_zone { with. (Datasheet clearly states the opposite, that this is tied to each PWM output so YMMV.) */ enum { - LM96000_LOW_TEMP_OFF = 0, /* turn fan off below low temp. */ - LM96000_LOW_TEMP_MIN = 1, /* keep PWM at mininum duty cycle */ + /* turn fan off below `low_temp - hysteresis` */ + LM96000_LOW_TEMP_OFF = 0, + /* keep PWM at mininum duty cycle */ + LM96000_LOW_TEMP_MIN = 1, } min_off; + u8 hysteresis; }; /* Implements only those parts currently used by coreboot mainboards. */ |