Commit 48f680c0 authored by Phil Reid's avatar Phil Reid Committed by Sebastian Reichel

power: supply: bq24735: remove incorrect le16_to_cpu calls

i2c_smbus commands handle the correct byte order for smbus transactions
internally. This will currently result in incorrect operation on big
endian systems.
Signed-off-by: default avatarPhil Reid <preid@electromag.com.au>
Signed-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.co.uk>
parent a1bbec72
...@@ -81,14 +81,12 @@ static int bq24735_charger_property_is_writeable(struct power_supply *psy, ...@@ -81,14 +81,12 @@ static int bq24735_charger_property_is_writeable(struct power_supply *psy,
static inline int bq24735_write_word(struct i2c_client *client, u8 reg, static inline int bq24735_write_word(struct i2c_client *client, u8 reg,
u16 value) u16 value)
{ {
return i2c_smbus_write_word_data(client, reg, le16_to_cpu(value)); return i2c_smbus_write_word_data(client, reg, value);
} }
static inline int bq24735_read_word(struct i2c_client *client, u8 reg) static inline int bq24735_read_word(struct i2c_client *client, u8 reg)
{ {
s32 ret = i2c_smbus_read_word_data(client, reg); return i2c_smbus_read_word_data(client, reg);
return ret < 0 ? ret : le16_to_cpu(ret);
} }
static int bq24735_update_word(struct i2c_client *client, u8 reg, static int bq24735_update_word(struct i2c_client *client, u8 reg,
......
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