diff options
author | Joe Schaack <jschaack@xes-inc.com> | 2017-04-18 08:15:50 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2017-04-21 15:30:09 -0400 |
commit | bdaf9c40b2d1d67617b259c59ce1858eda54b4ce (patch) | |
tree | 2126071904a9252f1d2bea87cfc61e3631786ff2 | |
parent | a02c24a321583e7970393488834501e15a69b656 (diff) |
hwmon: (ina209) Handled signed registers
The shunt voltage and current registers are signed 16-bit values so
handle them as such.
Signed-off-by: Joe Schaack <jschaack@xes-inc.com>
Reviewed-by: Aaron Sierra <asierra@xes-inc.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
-rw-r--r-- | drivers/hwmon/ina209.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hwmon/ina209.c b/drivers/hwmon/ina209.c index c5f20f03e0be..aa0768ce8aea 100644 --- a/drivers/hwmon/ina209.c +++ b/drivers/hwmon/ina209.c | |||
@@ -117,7 +117,7 @@ static long ina209_from_reg(const u8 reg, const u16 val) | |||
117 | case INA209_SHUNT_VOLTAGE_POS_WARN: | 117 | case INA209_SHUNT_VOLTAGE_POS_WARN: |
118 | case INA209_SHUNT_VOLTAGE_NEG_WARN: | 118 | case INA209_SHUNT_VOLTAGE_NEG_WARN: |
119 | /* LSB=10 uV. Convert to mV. */ | 119 | /* LSB=10 uV. Convert to mV. */ |
120 | return DIV_ROUND_CLOSEST(val, 100); | 120 | return DIV_ROUND_CLOSEST((s16)val, 100); |
121 | 121 | ||
122 | case INA209_BUS_VOLTAGE: | 122 | case INA209_BUS_VOLTAGE: |
123 | case INA209_BUS_VOLTAGE_MAX_PEAK: | 123 | case INA209_BUS_VOLTAGE_MAX_PEAK: |
@@ -146,7 +146,7 @@ static long ina209_from_reg(const u8 reg, const u16 val) | |||
146 | 146 | ||
147 | case INA209_CURRENT: | 147 | case INA209_CURRENT: |
148 | /* LSB=1 mA (selected). Is in mA */ | 148 | /* LSB=1 mA (selected). Is in mA */ |
149 | return val; | 149 | return (s16)val; |
150 | } | 150 | } |
151 | 151 | ||
152 | /* programmer goofed */ | 152 | /* programmer goofed */ |