diff options
author | Jean Delvare <khali@linux-fr.org> | 2008-02-19 12:42:58 +0100 |
---|---|---|
committer | Mark M. Hoffman <mhoffman@lightlink.com> | 2008-04-27 09:23:12 -0400 |
commit | ccd6befceb9a9b02114a93ff4cfa29adbdf60b6d (patch) | |
tree | f3cfe156da6c8bead8ffc81da87648916912eb18 | |
parent | 93c75a4ac2d95834e7202965d853d3cd23aadb40 (diff) | |
download | op-kernel-dev-ccd6befceb9a9b02114a93ff4cfa29adbdf60b6d.zip op-kernel-dev-ccd6befceb9a9b02114a93ff4cfa29adbdf60b6d.tar.gz |
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>
-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 115f409..fa76969 100644 --- a/drivers/hwmon/lm75.c +++ b/drivers/hwmon/lm75.c @@ -248,7 +248,7 @@ static int lm75_detach_client(struct i2c_client *client) /* All registers are word-sized, except for the configuration register. LM75 uses a high-byte first convention, which is exactly opposite to - the usual practice. */ + the SMBus standard. */ static int lm75_read_value(struct i2c_client *client, u8 reg) { if (reg == LM75_REG_CONF) @@ -257,9 +257,6 @@ static int lm75_read_value(struct i2c_client *client, u8 reg) return swab16(i2c_smbus_read_word_data(client, reg)); } -/* All registers are word-sized, except for the configuration register. - LM75 uses a high-byte first convention, which is exactly opposite to - the usual practice. */ static int lm75_write_value(struct i2c_client *client, u8 reg, u16 value) { if (reg == LM75_REG_CONF) |