diff options
| author | Jared Bents <jared.bents@rockwellcollins.com> | 2016-11-18 23:20:38 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-12 05:39:25 -0500 |
| commit | 29e7b170db90a852b51409a76870cdbe57341a9b (patch) | |
| tree | b435bf57e7d942138a2f480cb27288283ecb0a6a /drivers/hwmon/amc6821.c | |
| parent | af3cd3f0a805ab701f1e56ed93c676c77b7d2169 (diff) | |
hwmon: (amc6821) sign extension temperature
commit 4538bfbf2d9f1fc48c07ac0cc0ee58716fe7fe96 upstream.
Converts the unsigned temperature values from the i2c read
to be sign extended as defined in the datasheet so that
negative temperatures are properly read.
Fixes: 28e6274d8fa67 ("hwmon: (amc6821) Avoid forward declaration")
Signed-off-by: Jared Bents <jared.bents@rockwellcollins.com>
Signed-off-by: Matt Weber <matthew.weber@rockwellcollins.com>
[groeck: Dropped unnecessary continuation line]
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hwmon/amc6821.c')
| -rw-r--r-- | drivers/hwmon/amc6821.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hwmon/amc6821.c b/drivers/hwmon/amc6821.c index 12e851a5af48..46b4e35fd555 100644 --- a/drivers/hwmon/amc6821.c +++ b/drivers/hwmon/amc6821.c | |||
| @@ -188,8 +188,8 @@ static struct amc6821_data *amc6821_update_device(struct device *dev) | |||
| 188 | !data->valid) { | 188 | !data->valid) { |
| 189 | 189 | ||
| 190 | for (i = 0; i < TEMP_IDX_LEN; i++) | 190 | for (i = 0; i < TEMP_IDX_LEN; i++) |
| 191 | data->temp[i] = i2c_smbus_read_byte_data(client, | 191 | data->temp[i] = (int8_t)i2c_smbus_read_byte_data( |
| 192 | temp_reg[i]); | 192 | client, temp_reg[i]); |
| 193 | 193 | ||
| 194 | data->stat1 = i2c_smbus_read_byte_data(client, | 194 | data->stat1 = i2c_smbus_read_byte_data(client, |
| 195 | AMC6821_REG_STAT1); | 195 | AMC6821_REG_STAT1); |