diff options
author | David Hendricks <dhendrix@chromium.org> | 2016-05-13 20:24:56 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2016-08-01 20:01:49 -0700 |
commit | 201211f2dceba0007d9d84cb5cd6696fad0ee611 (patch) | |
tree | 47df1e60ca20767f7c89ca324249a725752106f9 /driver | |
parent | 83b6d69732f782e2b295153f959ec36d4a56c024 (diff) | |
download | chrome-ec-201211f2dceba0007d9d84cb5cd6696fad0ee611.tar.gz |
thermistor: Add generic linear interpolation algorithm
The existing algorithm makes several assumptions for a particular
thermistor circuit. This patch introduces a more generic version
that can be used for multiple thermistors on a single board.
The idea is to approximate a curve produced by solving for voltage
measued by an ADC using the Steinhart-Hart equation. For a straight
line one only needs two data points. For a steady curve data
points can be distributed evenly. For the most part, though, data
points should be provided after a significant change in slope.
More data points give more accuracy at the expense of memory, and
we mostly only care about accuracy in the range between "warm"
and "too hot" so only a few data points should be used.
BUG=chrome-os-partner:54818
BRANCH=none
TEST=added unit test, needs real testing
Change-Id: I046e61dbfd1e8c26c2a533777f222f5413938556
Signed-off-by: David Hendricks <dhendrix@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/344781
Reviewed-by: Shawn N <shawnn@chromium.org>
Diffstat (limited to 'driver')
-rw-r--r-- | driver/temp_sensor/thermistor.h | 38 | ||||
-rw-r--r-- | driver/temp_sensor/thermistor_ncp15wb.c | 51 |
2 files changed, 89 insertions, 0 deletions
diff --git a/driver/temp_sensor/thermistor.h b/driver/temp_sensor/thermistor.h index 3bb5335ddd..0f42742352 100644 --- a/driver/temp_sensor/thermistor.h +++ b/driver/temp_sensor/thermistor.h @@ -8,6 +8,44 @@ #ifndef __CROS_EC_TEMP_SENSOR_THERMISTOR_H #define __CROS_EC_TEMP_SENSOR_THERMISTOR_H +struct thermistor_data_pair { + uint8_t mv; /* Scaled voltage level at ADC (in mV) */ + uint8_t temp; /* Temperature in Celsius */ +}; + +struct thermistor_info { + uint8_t scaling_factor; /* Scaling factor for voltage in data pair. */ + uint8_t num_pairs; /* Number of data pairs. */ + + /* + * Values between given data pairs will be calculated as points on + * a line. Pairs can be derived using the Steinhart-Hart equation. + * + * Guidelines for data sets: + * - Must contain at least two pairs. + * - First and last pairs are the min and max. + * - Pairs must be sorted in ascending order. + * - 5 pairs should provide reasonable accuracy in most cases. Use + * points where the slope changes significantly or to recalibrate + * the algorithm if needed. + */ + const struct thermistor_data_pair *data; +}; + +/** + * @brief Calculate temperature using linear interpolation of data points. + * + * Given a set of datapoints, the algorithm will calculate the "step" in + * between each one in order to interpolate missing entries. + * + * @param mv Value read from ADC (in millivolts). + * @param info Reference data set and info. + * + * @return temperature in C + */ +int thermistor_linear_interpolate(uint16_t mv, + const struct thermistor_info *info); + /** * ncp15wb temperature conversion routine. * diff --git a/driver/temp_sensor/thermistor_ncp15wb.c b/driver/temp_sensor/thermistor_ncp15wb.c index 51e884ed35..64ba9fea40 100644 --- a/driver/temp_sensor/thermistor_ncp15wb.c +++ b/driver/temp_sensor/thermistor_ncp15wb.c @@ -98,3 +98,54 @@ int ncp15wb_calculate_temp(uint16_t adc) return temp; } + +int thermistor_linear_interpolate(uint16_t mv, + const struct thermistor_info *info) +{ + const struct thermistor_data_pair *data = info->data; + int v0, v1, t0, t1, num_steps; + int head, tail, mid; + + /* We need at least two points to form a line. */ + ASSERT(info->num_pairs >= 2); + + /* + * If input value is out of bounds return the lowest or highest + * value in the data sets provided. + */ + if (mv < data[0].mv * info->scaling_factor) + return data[0].temp; + else if (mv > data[info->num_pairs - 1].mv * info->scaling_factor) + return data[info->num_pairs - 1].temp; + + head = 0; + tail = info->num_pairs - 1; + while (head != tail) { + mid = (head + tail) / 2; + v0 = data[mid].mv * info->scaling_factor; + v1 = data[mid + 1].mv * info->scaling_factor; + + if ((mv >= v0) && (mv <= v1)) + break; + else if (mv < v0) + tail = mid; + else if (mv > v1) + head = mid + 1; + } + + t0 = data[mid].temp; + t1 = data[mid + 1].temp; + + /* + * The obvious way of doing this is to figure out how many mV per + * degree are in between the two points (mv_per_deg_c), and then how + * many of those exist between the input voltage and lower voltage (v0): + * 1. mv_per_deg_c = (v1 - v0) / (t1 - t0) + * 2. num_steps = (mv - v0) / mv_per_deg_c + * 3. result = t0 + num_steps + * + * Combine #1 and #2 to mitigate precision loss due to integer division. + */ + num_steps = ((mv - v0) * (t1 - t0)) / (v1 - v0); + return t0 + num_steps; +} |