aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoe Schaack <jschaack@xes-inc.com>2017-04-18 08:15:50 -0400
committerGuenter Roeck <linux@roeck-us.net>2017-04-21 15:30:09 -0400
commitbdaf9c40b2d1d67617b259c59ce1858eda54b4ce (patch)
tree2126071904a9252f1d2bea87cfc61e3631786ff2
parenta02c24a321583e7970393488834501e15a69b656 (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.c4
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 */