Commit f196dec6 authored by Dan Carpenter's avatar Dan Carpenter Committed by Guenter Roeck

hwmon: (adt7475) Make adt7475_read_word() return errors

The adt7475_read_word() function was meant to return negative error
codes on failure.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarTokunori Ikegami <ikegami@allied-telesis.co.jp>
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent 9d19371d
...@@ -302,14 +302,18 @@ static inline u16 volt2reg(int channel, long volt, u8 bypass_attn) ...@@ -302,14 +302,18 @@ static inline u16 volt2reg(int channel, long volt, u8 bypass_attn)
return clamp_val(reg, 0, 1023) & (0xff << 2); return clamp_val(reg, 0, 1023) & (0xff << 2);
} }
static u16 adt7475_read_word(struct i2c_client *client, int reg) static int adt7475_read_word(struct i2c_client *client, int reg)
{ {
u16 val; int val1, val2;
val = i2c_smbus_read_byte_data(client, reg); val1 = i2c_smbus_read_byte_data(client, reg);
val |= (i2c_smbus_read_byte_data(client, reg + 1) << 8); if (val1 < 0)
return val1;
val2 = i2c_smbus_read_byte_data(client, reg + 1);
if (val2 < 0)
return val2;
return val; return val1 | (val2 << 8);
} }
static void adt7475_write_word(struct i2c_client *client, int reg, u16 val) static void adt7475_write_word(struct i2c_client *client, int reg, u16 val)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment