summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2008-02-19 12:42:58 +0100
committerMark M. Hoffman <mhoffman@lightlink.com>2008-04-27 09:23:12 -0400
commitccd6befceb9a9b02114a93ff4cfa29adbdf60b6d (patch)
treef3cfe156da6c8bead8ffc81da87648916912eb18
parent93c75a4ac2d95834e7202965d853d3cd23aadb40 (diff)
downloadop-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.c5
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)
OpenPOWER on IntegriCloud