diff options
author | Jean Delvare <khali@linux-fr.org> | 2008-02-19 06:42:58 -0500 |
---|---|---|
committer | Mark M. Hoffman <mhoffman@lightlink.com> | 2008-04-27 09:23:12 -0400 |
commit | ccd6befceb9a9b02114a93ff4cfa29adbdf60b6d (patch) | |
tree | f3cfe156da6c8bead8ffc81da87648916912eb18 /drivers/hwmon | |
parent | 93c75a4ac2d95834e7202965d853d3cd23aadb40 (diff) |
hwmon: (lm75) Fix an incorrect comment
High-byte first is not opposite to the usual practice - that's what
almost all hardware monitoring drivers do. It is opposite to the SMBus
standard though.
Also delete a duplicate comment.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/lm75.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c index 115f4090b98e..fa7696905154 100644 --- a/drivers/hwmon/lm75.c +++ b/drivers/hwmon/lm75.c | |||
@@ -248,7 +248,7 @@ static int lm75_detach_client(struct i2c_client *client) | |||
248 | 248 | ||
249 | /* All registers are word-sized, except for the configuration register. | 249 | /* All registers are word-sized, except for the configuration register. |
250 | LM75 uses a high-byte first convention, which is exactly opposite to | 250 | LM75 uses a high-byte first convention, which is exactly opposite to |
251 | the usual practice. */ | 251 | the SMBus standard. */ |
252 | static int lm75_read_value(struct i2c_client *client, u8 reg) | 252 | static int lm75_read_value(struct i2c_client *client, u8 reg) |
253 | { | 253 | { |
254 | if (reg == LM75_REG_CONF) | 254 | if (reg == LM75_REG_CONF) |
@@ -257,9 +257,6 @@ static int lm75_read_value(struct i2c_client *client, u8 reg) | |||
257 | return swab16(i2c_smbus_read_word_data(client, reg)); | 257 | return swab16(i2c_smbus_read_word_data(client, reg)); |
258 | } | 258 | } |
259 | 259 | ||
260 | /* All registers are word-sized, except for the configuration register. | ||
261 | LM75 uses a high-byte first convention, which is exactly opposite to | ||
262 | the usual practice. */ | ||
263 | static int lm75_write_value(struct i2c_client *client, u8 reg, u16 value) | 260 | static int lm75_write_value(struct i2c_client *client, u8 reg, u16 value) |
264 | { | 261 | { |
265 | if (reg == LM75_REG_CONF) | 262 | if (reg == LM75_REG_CONF) |