Commit cb929eae authored by Jean Delvare's avatar Jean Delvare Committed by Greg Kroah-Hartman

[PATCH] hwmon: Minor w83l785ts optimization

Using s8 instead of u8 to store temperature register values saves a
few instructions on sysfs file read. The very same was done for
several other drivers a while ago (lm63, lm83, lm90...)
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>

 drivers/hwmon/w83l785ts.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
parent 709439a2
...@@ -74,7 +74,7 @@ I2C_CLIENT_INSMOD_1(w83l785ts); ...@@ -74,7 +74,7 @@ I2C_CLIENT_INSMOD_1(w83l785ts);
* The W83L785TS-S uses signed 8-bit values. * The W83L785TS-S uses signed 8-bit values.
*/ */
#define TEMP_FROM_REG(val) ((val & 0x80 ? val-0x100 : val) * 1000) #define TEMP_FROM_REG(val) ((val) * 1000)
/* /*
* Functions declaration * Functions declaration
...@@ -112,7 +112,7 @@ struct w83l785ts_data { ...@@ -112,7 +112,7 @@ struct w83l785ts_data {
unsigned long last_updated; /* in jiffies */ unsigned long last_updated; /* in jiffies */
/* registers values */ /* registers values */
u8 temp[2]; /* 0: input s8 temp[2]; /* 0: input
1: critical limit */ 1: critical limit */
}; };
......
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