leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

hwmon: (lm80) Fix multi-line comments

Browse Source 
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Jean Delvare <khali@linux-fr.org>
This commit is contained in:
Guenter Roeck 2012-01-19 11:02:20 -08:00 committed by Guenter Roeck
parent bf0f3a0430
commit 1160631b10
1 changed files with 19 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
drivers/hwmon/lm80.c
View File

@ -1,8 +1,8 @@
/* /*
* lm80.c - From lm_sensors, Linux kernel modules for hardware * lm80.c - From lm_sensors, Linux kernel modules for hardware
* monitoring * monitoring
* Copyright (C) 1998, 1999 Frodo Looijaard <frodol@dds.nl> * Copyright (C) 1998, 1999 Frodo Looijaard <frodol@dds.nl>
* and Philip Edelbrock <phil@netroedge.com> * and Philip Edelbrock <phil@netroedge.com>
* *
* Ported to Linux 2.6 by Tiago Sousa <mirage@kaotik.org> * Ported to Linux 2.6 by Tiago Sousa <mirage@kaotik.org>
* *
@ -61,10 +61,12 @@ static const unsigned short normal_i2c[] = { 0x28, 0x29, 0x2a, 0x2b, 0x2c, 0x2d,
#define LM80_REG_RES 0x06 #define LM80_REG_RES 0x06
/* Conversions. Rounding and limit checking is only done on the TO_REG /*
variants. Note that you should be a bit careful with which arguments * Conversions. Rounding and limit checking is only done on the TO_REG
these macros are called: arguments may be evaluated more than once. * variants. Note that you should be a bit careful with which arguments
Fixing this is just not worth it. */ * these macros are called: arguments may be evaluated more than once.
* Fixing this is just not worth it.
*/
#define IN_TO_REG(val) (SENSORS_LIMIT(((val) + 5) / 10, 0, 255)) #define IN_TO_REG(val) (SENSORS_LIMIT(((val) + 5) / 10, 0, 255))
#define IN_FROM_REG(val) ((val) * 10) #define IN_FROM_REG(val) ((val) * 10)
@ -242,10 +244,12 @@ static ssize_t set_fan_min(struct device *dev, struct device_attribute *attr,
return count; return count;
} }
/* Note: we save and restore the fan minimum here, because its value is /*
determined in part by the fan divisor. This follows the principle of * Note: we save and restore the fan minimum here, because its value is
least surprise; the user doesn't expect the fan minimum to change just * determined in part by the fan divisor. This follows the principle of
because the divisor changed. */ * least surprise; the user doesn't expect the fan minimum to change just
* because the divisor changed.
*/
static ssize_t set_fan_div(struct device *dev, struct device_attribute *attr, static ssize_t set_fan_div(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
{ {
@ -574,9 +578,11 @@ static int lm80_write_value(struct i2c_client *client, u8 reg, u8 value)
/* Called when we have found a new LM80. */ /* Called when we have found a new LM80. */
static void lm80_init_client(struct i2c_client *client) static void lm80_init_client(struct i2c_client *client)
{ {
/* Reset all except Watchdog values and last conversion values /*
This sets fan-divs to 2, among others. This makes most other * Reset all except Watchdog values and last conversion values
initializations unnecessary */ * This sets fan-divs to 2, among others. This makes most other
* initializations unnecessary
*/
lm80_write_value(client, LM80_REG_CONFIG, 0x80); lm80_write_value(client, LM80_REG_CONFIG, 0x80);
/* Set 11-bit temperature resolution */ /* Set 11-bit temperature resolution */
lm80_write_value(client, LM80_REG_RES, 0x08); lm80_write_value(client, LM80_REG_RES, 0x08);