提交 c10b3ee8 编写于 作者: G Guenter Roeck 提交者: Jean Delvare

hwmon: (w83795) Fix multi-line comments

Signed-off-by: NGuenter Roeck <linux@roeck-us.net>
Signed-off-by: NJean Delvare <khali@linux-fr.org>
上级 72fea694
...@@ -72,8 +72,10 @@ MODULE_PARM_DESC(reset, "Set to 1 to reset chip, not recommended"); ...@@ -72,8 +72,10 @@ MODULE_PARM_DESC(reset, "Set to 1 to reset chip, not recommended");
#define TEMP_CRIT_HYST 2 #define TEMP_CRIT_HYST 2
#define TEMP_WARN 3 #define TEMP_WARN 3
#define TEMP_WARN_HYST 4 #define TEMP_WARN_HYST 4
/* only crit and crit_hyst affect real-time alarm status /*
* current crit crit_hyst warn warn_hyst */ * only crit and crit_hyst affect real-time alarm status
* current crit crit_hyst warn warn_hyst
*/
static const u16 W83795_REG_TEMP[][5] = { static const u16 W83795_REG_TEMP[][5] = {
{0x21, 0x96, 0x97, 0x98, 0x99}, /* TD1/TR1 */ {0x21, 0x96, 0x97, 0x98, 0x99}, /* TD1/TR1 */
{0x22, 0x9a, 0x9b, 0x9c, 0x9d}, /* TD2/TR2 */ {0x22, 0x9a, 0x9b, 0x9c, 0x9d}, /* TD2/TR2 */
...@@ -354,26 +356,34 @@ struct w83795_data { ...@@ -354,26 +356,34 @@ struct w83795_data {
u8 temp_mode; /* Bit vector, 0 = TR, 1 = TD */ u8 temp_mode; /* Bit vector, 0 = TR, 1 = TD */
u8 temp_src[3]; /* Register value */ u8 temp_src[3]; /* Register value */
u8 enable_dts; /* Enable PECI and SB-TSI, u8 enable_dts; /*
* Enable PECI and SB-TSI,
* bit 0: =1 enable, =0 disable, * bit 0: =1 enable, =0 disable,
* bit 1: =1 AMD SB-TSI, =0 Intel PECI */ * bit 1: =1 AMD SB-TSI, =0 Intel PECI
*/
u8 has_dts; /* Enable monitor DTS temp */ u8 has_dts; /* Enable monitor DTS temp */
s8 dts[8]; /* Register value */ s8 dts[8]; /* Register value */
u8 dts_read_vrlsb[8]; /* Register value */ u8 dts_read_vrlsb[8]; /* Register value */
s8 dts_ext[4]; /* Register value */ s8 dts_ext[4]; /* Register value */
u8 has_pwm; /* 795g supports 8 pwm, 795adg only supports 2, u8 has_pwm; /*
* 795g supports 8 pwm, 795adg only supports 2,
* no config register, only affected by chip * no config register, only affected by chip
* type */ * type
u8 pwm[8][5]; /* Register value, output, freq, start, */
* non stop, stop time */ u8 pwm[8][5]; /*
* Register value, output, freq, start,
* non stop, stop time
*/
u16 clkin; /* CLKIN frequency in kHz */ u16 clkin; /* CLKIN frequency in kHz */
u8 pwm_fcms[2]; /* Register value */ u8 pwm_fcms[2]; /* Register value */
u8 pwm_tfmr[6]; /* Register value */ u8 pwm_tfmr[6]; /* Register value */
u8 pwm_fomc; /* Register value */ u8 pwm_fomc; /* Register value */
u16 target_speed[8]; /* Register value, target speed for speed u16 target_speed[8]; /*
* cruise */ * Register value, target speed for speed
* cruise
*/
u8 tol_speed; /* tolerance of target speed */ u8 tol_speed; /* tolerance of target speed */
u8 pwm_temp[6][4]; /* TTTI, CTFS, HCT, HOT */ u8 pwm_temp[6][4]; /* TTTI, CTFS, HCT, HOT */
u8 sf4_reg[6][2][7]; /* 6 temp, temp/dcpwm, 7 registers */ u8 sf4_reg[6][2][7]; /* 6 temp, temp/dcpwm, 7 registers */
...@@ -482,8 +492,10 @@ static void w83795_update_limits(struct i2c_client *client) ...@@ -482,8 +492,10 @@ static void w83795_update_limits(struct i2c_client *client)
/* Read the fan limits */ /* Read the fan limits */
lsb = 0; /* Silent false gcc warning */ lsb = 0; /* Silent false gcc warning */
for (i = 0; i < ARRAY_SIZE(data->fan); i++) { for (i = 0; i < ARRAY_SIZE(data->fan); i++) {
/* Each register contains LSB for 2 fans, but we want to /*
* read it only once to save time */ * Each register contains LSB for 2 fans, but we want to
* read it only once to save time
*/
if ((i & 1) == 0 && (data->has_fan & (3 << i))) if ((i & 1) == 0 && (data->has_fan & (3 << i)))
lsb = w83795_read(client, W83795_REG_FAN_MIN_LSB(i)); lsb = w83795_read(client, W83795_REG_FAN_MIN_LSB(i));
...@@ -665,9 +677,11 @@ static struct w83795_data *w83795_update_device(struct device *dev) ...@@ -665,9 +677,11 @@ static struct w83795_data *w83795_update_device(struct device *dev)
w83795_read(client, W83795_REG_PWM(i, PWM_OUTPUT)); w83795_read(client, W83795_REG_PWM(i, PWM_OUTPUT));
} }
/* Update intrusion and alarms /*
* Update intrusion and alarms
* It is important to read intrusion first, because reading from * It is important to read intrusion first, because reading from
* register SMI STS6 clears the interrupt status temporarily. */ * register SMI STS6 clears the interrupt status temporarily.
*/
tmp = w83795_read(client, W83795_REG_ALARM_CTRL); tmp = w83795_read(client, W83795_REG_ALARM_CTRL);
/* Switch to interrupt status for intrusion if needed */ /* Switch to interrupt status for intrusion if needed */
if (tmp & ALARM_CTRL_RTSACS) if (tmp & ALARM_CTRL_RTSACS)
...@@ -1603,8 +1617,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr, ...@@ -1603,8 +1617,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr,
#define NOT_USED -1 #define NOT_USED -1
/* Don't change the attribute order, _max, _min and _beep are accessed by index /*
* somewhere else in the code */ * Don't change the attribute order, _max, _min and _beep are accessed by index
* somewhere else in the code
*/
#define SENSOR_ATTR_IN(index) { \ #define SENSOR_ATTR_IN(index) { \
SENSOR_ATTR_2(in##index##_input, S_IRUGO, show_in, NULL, \ SENSOR_ATTR_2(in##index##_input, S_IRUGO, show_in, NULL, \
IN_READ, index), \ IN_READ, index), \
...@@ -1618,8 +1634,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr, ...@@ -1618,8 +1634,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr,
show_alarm_beep, store_beep, BEEP_ENABLE, \ show_alarm_beep, store_beep, BEEP_ENABLE, \
index + ((index > 14) ? 1 : 0)) } index + ((index > 14) ? 1 : 0)) }
/* Don't change the attribute order, _beep is accessed by index /*
* somewhere else in the code */ * Don't change the attribute order, _beep is accessed by index
* somewhere else in the code
*/
#define SENSOR_ATTR_FAN(index) { \ #define SENSOR_ATTR_FAN(index) { \
SENSOR_ATTR_2(fan##index##_input, S_IRUGO, show_fan, \ SENSOR_ATTR_2(fan##index##_input, S_IRUGO, show_fan, \
NULL, FAN_INPUT, index - 1), \ NULL, FAN_INPUT, index - 1), \
...@@ -1648,8 +1666,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr, ...@@ -1648,8 +1666,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr,
SENSOR_ATTR_2(fan##index##_target, S_IWUSR | S_IRUGO, \ SENSOR_ATTR_2(fan##index##_target, S_IWUSR | S_IRUGO, \
show_fanin, store_fanin, FANIN_TARGET, index - 1) } show_fanin, store_fanin, FANIN_TARGET, index - 1) }
/* Don't change the attribute order, _beep is accessed by index /*
* somewhere else in the code */ * Don't change the attribute order, _beep is accessed by index
* somewhere else in the code
*/
#define SENSOR_ATTR_DTS(index) { \ #define SENSOR_ATTR_DTS(index) { \
SENSOR_ATTR_2(temp##index##_type, S_IRUGO , \ SENSOR_ATTR_2(temp##index##_type, S_IRUGO , \
show_dts_mode, NULL, NOT_USED, index - 7), \ show_dts_mode, NULL, NOT_USED, index - 7), \
...@@ -1668,8 +1688,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr, ...@@ -1668,8 +1688,10 @@ store_sf_setup(struct device *dev, struct device_attribute *attr,
SENSOR_ATTR_2(temp##index##_beep, S_IWUSR | S_IRUGO, \ SENSOR_ATTR_2(temp##index##_beep, S_IWUSR | S_IRUGO, \
show_alarm_beep, store_beep, BEEP_ENABLE, index + 17) } show_alarm_beep, store_beep, BEEP_ENABLE, index + 17) }
/* Don't change the attribute order, _beep is accessed by index /*
* somewhere else in the code */ * Don't change the attribute order, _beep is accessed by index
* somewhere else in the code
*/
#define SENSOR_ATTR_TEMP(index) { \ #define SENSOR_ATTR_TEMP(index) { \
SENSOR_ATTR_2(temp##index##_type, S_IRUGO | (index < 4 ? S_IWUSR : 0), \ SENSOR_ATTR_2(temp##index##_type, S_IRUGO | (index < 4 ? S_IWUSR : 0), \
show_temp_mode, store_temp_mode, NOT_USED, index - 1), \ show_temp_mode, store_temp_mode, NOT_USED, index - 1), \
...@@ -1875,8 +1897,10 @@ static int w83795_get_device_id(struct i2c_client *client) ...@@ -1875,8 +1897,10 @@ static int w83795_get_device_id(struct i2c_client *client)
device_id = i2c_smbus_read_byte_data(client, W83795_REG_DEVICEID); device_id = i2c_smbus_read_byte_data(client, W83795_REG_DEVICEID);
/* Special case for rev. A chips; can't be checked first because later /*
revisions emulate this for compatibility */ * Special case for rev. A chips; can't be checked first because later
* revisions emulate this for compatibility
*/
if (device_id < 0 || (device_id & 0xf0) != 0x50) { if (device_id < 0 || (device_id & 0xf0) != 0x50) {
int alt_id; int alt_id;
...@@ -1928,8 +1952,10 @@ static int w83795_detect(struct i2c_client *client, ...@@ -1928,8 +1952,10 @@ static int w83795_detect(struct i2c_client *client,
return -ENODEV; return -ENODEV;
} }
/* If Nuvoton chip, address of chip and W83795_REG_I2C_ADDR /*
should match */ * If Nuvoton chip, address of chip and W83795_REG_I2C_ADDR
* should match
*/
if ((bank & 0x07) == 0) { if ((bank & 0x07) == 0) {
i2c_addr = i2c_smbus_read_byte_data(client, i2c_addr = i2c_smbus_read_byte_data(client,
W83795_REG_I2C_ADDR); W83795_REG_I2C_ADDR);
...@@ -1941,10 +1967,12 @@ static int w83795_detect(struct i2c_client *client, ...@@ -1941,10 +1967,12 @@ static int w83795_detect(struct i2c_client *client,
} }
} }
/* Check 795 chip type: 795G or 795ADG /*
Usually we don't write to chips during detection, but here we don't * Check 795 chip type: 795G or 795ADG
quite have the choice; hopefully it's OK, we are about to return * Usually we don't write to chips during detection, but here we don't
success anyway */ * quite have the choice; hopefully it's OK, we are about to return
* success anyway
*/
if ((bank & 0x07) != 0) if ((bank & 0x07) != 0)
i2c_smbus_write_byte_data(client, W83795_REG_BANKSEL, i2c_smbus_write_byte_data(client, W83795_REG_BANKSEL,
bank & ~0x07); bank & ~0x07);
...@@ -2193,8 +2221,10 @@ static int w83795_probe(struct i2c_client *client, ...@@ -2193,8 +2221,10 @@ static int w83795_probe(struct i2c_client *client,
/* The W83795G has a dedicated BEEP pin */ /* The W83795G has a dedicated BEEP pin */
data->enable_beep = 1; data->enable_beep = 1;
} else { } else {
/* The W83795ADG has a shared pin for OVT# and BEEP, so you /*
* can't have both */ * The W83795ADG has a shared pin for OVT# and BEEP, so you
* can't have both
*/
tmp = w83795_read(client, W83795_REG_OVT_CFG); tmp = w83795_read(client, W83795_REG_OVT_CFG);
if ((tmp & OVT_CFG_SEL) == 0) if ((tmp & OVT_CFG_SEL) == 0)
data->enable_beep = 1; data->enable_beep = 1;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册