提交 c67c0693 编写于 作者: H Hans de Goede 提交者: Sebastian Reichel

power: supply: max17042_battery: Use sign_extend32 instead of DIY code

Use sign_extend32 to sign-extend variables where necessary instead of
DIY code.
Signed-off-by: NHans de Goede <hdegoede@redhat.com>
Reviewed-by: NKrzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.co.uk>
上级 7f232af3
...@@ -106,13 +106,7 @@ static int max17042_get_temperature(struct max17042_chip *chip, int *temp) ...@@ -106,13 +106,7 @@ static int max17042_get_temperature(struct max17042_chip *chip, int *temp)
if (ret < 0) if (ret < 0)
return ret; return ret;
*temp = data; *temp = sign_extend32(data, 15);
/* The value is signed. */
if (*temp & 0x8000) {
*temp = (0x7fff & ~*temp) + 1;
*temp *= -1;
}
/* The value is converted into deci-centigrade scale */ /* The value is converted into deci-centigrade scale */
/* Units of LSB = 1 / 256 degree Celsius */ /* Units of LSB = 1 / 256 degree Celsius */
*temp = *temp * 10 / 256; *temp = *temp * 10 / 256;
...@@ -302,13 +296,7 @@ static int max17042_get_property(struct power_supply *psy, ...@@ -302,13 +296,7 @@ static int max17042_get_property(struct power_supply *psy,
if (ret < 0) if (ret < 0)
return ret; return ret;
val->intval = data; val->intval = sign_extend32(data, 15);
if (val->intval & 0x8000) {
/* Negative */
val->intval = ~val->intval & 0x7fff;
val->intval++;
val->intval *= -1;
}
val->intval *= 1562500 / chip->pdata->r_sns; val->intval *= 1562500 / chip->pdata->r_sns;
} else { } else {
return -EINVAL; return -EINVAL;
...@@ -320,13 +308,7 @@ static int max17042_get_property(struct power_supply *psy, ...@@ -320,13 +308,7 @@ static int max17042_get_property(struct power_supply *psy,
if (ret < 0) if (ret < 0)
return ret; return ret;
val->intval = data; val->intval = sign_extend32(data, 15);
if (val->intval & 0x8000) {
/* Negative */
val->intval = ~val->intval & 0x7fff;
val->intval++;
val->intval *= -1;
}
val->intval *= 1562500 / chip->pdata->r_sns; val->intval *= 1562500 / chip->pdata->r_sns;
} else { } else {
return -EINVAL; return -EINVAL;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册