summaryrefslogtreecommitdiffstats
path: root/drivers/hwmon
diff options
context:
space:
mode:
authorEugene Shalygin <eugene.shalygin@gmail.com>2022-02-04 17:30:45 +0100
committerGuenter Roeck <linux@roeck-us.net>2022-02-27 17:03:18 -0800
commit339f8a998f58e8693f73429e7597f2ffc9b93ec5 (patch)
tree4b3982dcd0cf2d86108b7fb3c6b10f7dd9f789ba /drivers/hwmon
parent88846ff7422089f576e29b68153034babbc68ce6 (diff)
downloadlinux-339f8a998f58e8693f73429e7597f2ffc9b93ec5.tar.bz2
hwmon: (asus-ec-sensors) read sensors as signed ints
Temperature sensor readings are signed, which is hinted by their blank value (oxd8, 216 as unsigned and -40 as signed). T_Sensor, Crosshair VIII Hero, and a freezer were used to confirm that. Here we read fan sensors as signed too, because with their typical values and 2-byte width, I can't tell a difference between signed and unsigned, as I don't have a high speed chipset fan. Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com> Link: https://lore.kernel.org/r/20220204163045.576903-1-eugene.shalygin@gmail.com Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r--drivers/hwmon/asus-ec-sensors.c16
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/hwmon/asus-ec-sensors.c b/drivers/hwmon/asus-ec-sensors.c
index 7330cce46b14..dd7b207d062f 100644
--- a/drivers/hwmon/asus-ec-sensors.c
+++ b/drivers/hwmon/asus-ec-sensors.c
@@ -221,7 +221,7 @@ static const struct dmi_system_id asus_ec_dmi_table[] __initconst = {
struct ec_sensor {
unsigned int info_index;
- u32 cached_value;
+ s32 cached_value;
};
struct ec_sensors_data {
@@ -408,15 +408,15 @@ static int asus_ec_block_read(const struct device *dev,
return status;
}
-static inline u32 get_sensor_value(const struct ec_sensor_info *si, u8 *data)
+static inline s32 get_sensor_value(const struct ec_sensor_info *si, u8 *data)
{
switch (si->addr.components.size) {
case 1:
- return *data;
+ return (s8)*data;
case 2:
- return get_unaligned_be16(data);
+ return (s16)get_unaligned_be16(data);
case 4:
- return get_unaligned_be32(data);
+ return (s32)get_unaligned_be32(data);
default:
return 0;
}
@@ -462,7 +462,7 @@ cleanup:
return status;
}
-static int scale_sensor_value(u32 value, int data_type)
+static long scale_sensor_value(s32 value, int data_type)
{
switch (data_type) {
case hwmon_curr:
@@ -476,7 +476,7 @@ static int scale_sensor_value(u32 value, int data_type)
static int get_cached_value_or_update(const struct device *dev,
int sensor_index,
- struct ec_sensors_data *state, u32 *value)
+ struct ec_sensors_data *state, s32 *value)
{
if (time_after(jiffies, state->last_updated + HZ)) {
if (update_ec_sensors(dev, state)) {
@@ -499,7 +499,7 @@ static int asus_ec_hwmon_read(struct device *dev, enum hwmon_sensor_types type,
u32 attr, int channel, long *val)
{
int ret;
- u32 value = 0;
+ s32 value = 0;
struct ec_sensors_data *state = dev_get_drvdata(dev);
int sidx = find_ec_sensor_index(state, type, channel);