Commit 90d1a336 authored by Mark M. Hoffman's avatar Mark M. Hoffman Committed by Greg Kroah-Hartman

[PATCH] I2C: sysfs interface update for w83627hf

This patch updates the sysfs names of the w83627hf driver
to match the new standard.  The patch applies on top of
one recently applied to your tree [1].  I have tested it
using a w83627thf & the latest lm_sensors CVS.  Please apply.

[1] http://archives.andrew.net.au/lm-sensors/msg06746.html
parent 293ff9af
...@@ -294,10 +294,10 @@ struct w83627hf_data { ...@@ -294,10 +294,10 @@ struct w83627hf_data {
u8 fan_min[3]; /* Register value */ u8 fan_min[3]; /* Register value */
u8 temp; u8 temp;
u8 temp_max; /* Register value */ u8 temp_max; /* Register value */
u8 temp_hyst; /* Register value */ u8 temp_max_hyst; /* Register value */
u16 temp_add[2]; /* Register value */ u16 temp_add[2]; /* Register value */
u16 temp_max_add[2]; /* Register value */ u16 temp_max_add[2]; /* Register value */
u16 temp_hyst_add[2]; /* Register value */ u16 temp_max_hyst_add[2]; /* Register value */
u8 fan_div[3]; /* Register encoding, shifted right */ u8 fan_div[3]; /* Register encoding, shifted right */
u8 vid; /* Register encoding, combined */ u8 vid; /* Register encoding, combined */
u32 alarms; /* Register encoding, combined */ u32 alarms; /* Register encoding, combined */
...@@ -373,7 +373,7 @@ show_regs_in_##offset (struct device *dev, char *buf) \ ...@@ -373,7 +373,7 @@ show_regs_in_##offset (struct device *dev, char *buf) \
{ \ { \
return show_in(dev, buf, 0x##offset); \ return show_in(dev, buf, 0x##offset); \
} \ } \
static DEVICE_ATTR(in_input##offset, S_IRUGO, show_regs_in_##offset, NULL) static DEVICE_ATTR(in##offset##_input, S_IRUGO, show_regs_in_##offset, NULL)
#define sysfs_in_reg_offset(reg, offset) \ #define sysfs_in_reg_offset(reg, offset) \
static ssize_t show_regs_in_##reg##offset (struct device *dev, char *buf) \ static ssize_t show_regs_in_##reg##offset (struct device *dev, char *buf) \
...@@ -386,7 +386,7 @@ store_regs_in_##reg##offset (struct device *dev, \ ...@@ -386,7 +386,7 @@ store_regs_in_##reg##offset (struct device *dev, \
{ \ { \
return store_in_##reg (dev, buf, count, 0x##offset); \ return store_in_##reg (dev, buf, count, 0x##offset); \
} \ } \
static DEVICE_ATTR(in_##reg##offset, S_IRUGO| S_IWUSR, \ static DEVICE_ATTR(in##offset##_##reg, S_IRUGO| S_IWUSR, \
show_regs_in_##reg##offset, store_regs_in_##reg##offset) show_regs_in_##reg##offset, store_regs_in_##reg##offset)
#define sysfs_in_offsets(offset) \ #define sysfs_in_offsets(offset) \
...@@ -406,9 +406,9 @@ sysfs_in_offsets(8) ...@@ -406,9 +406,9 @@ sysfs_in_offsets(8)
#define device_create_file_in(client, offset) \ #define device_create_file_in(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_in_input##offset); \ device_create_file(&client->dev, &dev_attr_in##offset##_input); \
device_create_file(&client->dev, &dev_attr_in_min##offset); \ device_create_file(&client->dev, &dev_attr_in##offset##_min); \
device_create_file(&client->dev, &dev_attr_in_max##offset); \ device_create_file(&client->dev, &dev_attr_in##offset##_max); \
} while (0) } while (0)
#define show_fan_reg(reg) \ #define show_fan_reg(reg) \
...@@ -447,7 +447,7 @@ static ssize_t show_regs_fan_##offset (struct device *dev, char *buf) \ ...@@ -447,7 +447,7 @@ static ssize_t show_regs_fan_##offset (struct device *dev, char *buf) \
{ \ { \
return show_fan(dev, buf, 0x##offset); \ return show_fan(dev, buf, 0x##offset); \
} \ } \
static DEVICE_ATTR(fan_input##offset, S_IRUGO, show_regs_fan_##offset, NULL) static DEVICE_ATTR(fan##offset##_input, S_IRUGO, show_regs_fan_##offset, NULL)
#define sysfs_fan_min_offset(offset) \ #define sysfs_fan_min_offset(offset) \
static ssize_t show_regs_fan_min##offset (struct device *dev, char *buf) \ static ssize_t show_regs_fan_min##offset (struct device *dev, char *buf) \
...@@ -459,7 +459,7 @@ store_regs_fan_min##offset (struct device *dev, const char *buf, size_t count) \ ...@@ -459,7 +459,7 @@ store_regs_fan_min##offset (struct device *dev, const char *buf, size_t count) \
{ \ { \
return store_fan_min(dev, buf, count, 0x##offset); \ return store_fan_min(dev, buf, count, 0x##offset); \
} \ } \
static DEVICE_ATTR(fan_min##offset, S_IRUGO | S_IWUSR, \ static DEVICE_ATTR(fan##offset##_min, S_IRUGO | S_IWUSR, \
show_regs_fan_min##offset, store_regs_fan_min##offset) show_regs_fan_min##offset, store_regs_fan_min##offset)
sysfs_fan_offset(1) sysfs_fan_offset(1)
...@@ -471,8 +471,8 @@ sysfs_fan_min_offset(3) ...@@ -471,8 +471,8 @@ sysfs_fan_min_offset(3)
#define device_create_file_fan(client, offset) \ #define device_create_file_fan(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_fan_input##offset); \ device_create_file(&client->dev, &dev_attr_fan##offset##_input); \
device_create_file(&client->dev, &dev_attr_fan_min##offset); \ device_create_file(&client->dev, &dev_attr_fan##offset##_min); \
} while (0) } while (0)
#define show_temp_reg(reg) \ #define show_temp_reg(reg) \
...@@ -492,7 +492,7 @@ static ssize_t show_##reg (struct device *dev, char *buf, int nr) \ ...@@ -492,7 +492,7 @@ static ssize_t show_##reg (struct device *dev, char *buf, int nr) \
} }
show_temp_reg(temp) show_temp_reg(temp)
show_temp_reg(temp_max) show_temp_reg(temp_max)
show_temp_reg(temp_hyst) show_temp_reg(temp_max_hyst)
#define store_temp_reg(REG, reg) \ #define store_temp_reg(REG, reg) \
static ssize_t \ static ssize_t \
...@@ -517,7 +517,7 @@ store_temp_##reg (struct device *dev, const char *buf, size_t count, int nr) \ ...@@ -517,7 +517,7 @@ store_temp_##reg (struct device *dev, const char *buf, size_t count, int nr) \
return count; \ return count; \
} }
store_temp_reg(OVER, max) store_temp_reg(OVER, max)
store_temp_reg(HYST, hyst) store_temp_reg(HYST, max_hyst)
#define sysfs_temp_offset(offset) \ #define sysfs_temp_offset(offset) \
static ssize_t \ static ssize_t \
...@@ -525,7 +525,7 @@ show_regs_temp_##offset (struct device *dev, char *buf) \ ...@@ -525,7 +525,7 @@ show_regs_temp_##offset (struct device *dev, char *buf) \
{ \ { \
return show_temp(dev, buf, 0x##offset); \ return show_temp(dev, buf, 0x##offset); \
} \ } \
static DEVICE_ATTR(temp_input##offset, S_IRUGO, show_regs_temp_##offset, NULL) static DEVICE_ATTR(temp##offset##_input, S_IRUGO, show_regs_temp_##offset, NULL)
#define sysfs_temp_reg_offset(reg, offset) \ #define sysfs_temp_reg_offset(reg, offset) \
static ssize_t show_regs_temp_##reg##offset (struct device *dev, char *buf) \ static ssize_t show_regs_temp_##reg##offset (struct device *dev, char *buf) \
...@@ -538,13 +538,13 @@ store_regs_temp_##reg##offset (struct device *dev, \ ...@@ -538,13 +538,13 @@ store_regs_temp_##reg##offset (struct device *dev, \
{ \ { \
return store_temp_##reg (dev, buf, count, 0x##offset); \ return store_temp_##reg (dev, buf, count, 0x##offset); \
} \ } \
static DEVICE_ATTR(temp_##reg##offset, S_IRUGO| S_IWUSR, \ static DEVICE_ATTR(temp##offset##_##reg, S_IRUGO| S_IWUSR, \
show_regs_temp_##reg##offset, store_regs_temp_##reg##offset) show_regs_temp_##reg##offset, store_regs_temp_##reg##offset)
#define sysfs_temp_offsets(offset) \ #define sysfs_temp_offsets(offset) \
sysfs_temp_offset(offset) \ sysfs_temp_offset(offset) \
sysfs_temp_reg_offset(max, offset) \ sysfs_temp_reg_offset(max, offset) \
sysfs_temp_reg_offset(hyst, offset) sysfs_temp_reg_offset(max_hyst, offset)
sysfs_temp_offsets(1) sysfs_temp_offsets(1)
sysfs_temp_offsets(2) sysfs_temp_offsets(2)
...@@ -552,9 +552,9 @@ sysfs_temp_offsets(3) ...@@ -552,9 +552,9 @@ sysfs_temp_offsets(3)
#define device_create_file_temp(client, offset) \ #define device_create_file_temp(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_temp_input##offset); \ device_create_file(&client->dev, &dev_attr_temp##offset##_input); \
device_create_file(&client->dev, &dev_attr_temp_max##offset); \ device_create_file(&client->dev, &dev_attr_temp##offset##_max); \
device_create_file(&client->dev, &dev_attr_temp_hyst##offset); \ device_create_file(&client->dev, &dev_attr_temp##offset##_max_hyst); \
} while (0) } while (0)
static ssize_t static ssize_t
...@@ -567,9 +567,9 @@ show_vid_reg(struct device *dev, char *buf) ...@@ -567,9 +567,9 @@ show_vid_reg(struct device *dev, char *buf)
return sprintf(buf, "%ld\n", (long) vid_from_reg(data->vid, data->vrm)); return sprintf(buf, "%ld\n", (long) vid_from_reg(data->vid, data->vrm));
} }
static DEVICE_ATTR(vid, S_IRUGO, show_vid_reg, NULL) static DEVICE_ATTR(in0_ref, S_IRUGO, show_vid_reg, NULL)
#define device_create_file_vid(client) \ #define device_create_file_vid(client) \
device_create_file(&client->dev, &dev_attr_vid) device_create_file(&client->dev, &dev_attr_in0_ref)
static ssize_t static ssize_t
show_vrm_reg(struct device *dev, char *buf) show_vrm_reg(struct device *dev, char *buf)
...@@ -734,7 +734,7 @@ store_regs_fan_div_##offset (struct device *dev, \ ...@@ -734,7 +734,7 @@ store_regs_fan_div_##offset (struct device *dev, \
{ \ { \
return store_fan_div_reg(dev, buf, count, offset); \ return store_fan_div_reg(dev, buf, count, offset); \
} \ } \
static DEVICE_ATTR(fan_div##offset, S_IRUGO | S_IWUSR, \ static DEVICE_ATTR(fan##offset##_div, S_IRUGO | S_IWUSR, \
show_regs_fan_div_##offset, store_regs_fan_div_##offset) show_regs_fan_div_##offset, store_regs_fan_div_##offset)
sysfs_fan_div(1) sysfs_fan_div(1)
...@@ -743,7 +743,7 @@ sysfs_fan_div(3) ...@@ -743,7 +743,7 @@ sysfs_fan_div(3)
#define device_create_file_fan_div(client, offset) \ #define device_create_file_fan_div(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_fan_div##offset); \ device_create_file(&client->dev, &dev_attr_fan##offset##_div); \
} while (0) } while (0)
static ssize_t static ssize_t
...@@ -794,7 +794,7 @@ store_regs_pwm_##offset (struct device *dev, const char *buf, size_t count) \ ...@@ -794,7 +794,7 @@ store_regs_pwm_##offset (struct device *dev, const char *buf, size_t count) \
{ \ { \
return store_pwm_reg(dev, buf, count, offset); \ return store_pwm_reg(dev, buf, count, offset); \
} \ } \
static DEVICE_ATTR(pwm##offset, S_IRUGO | S_IWUSR, \ static DEVICE_ATTR(fan##offset##_pwm, S_IRUGO | S_IWUSR, \
show_regs_pwm_##offset, store_regs_pwm_##offset) show_regs_pwm_##offset, store_regs_pwm_##offset)
sysfs_pwm(1) sysfs_pwm(1)
...@@ -803,7 +803,7 @@ sysfs_pwm(3) ...@@ -803,7 +803,7 @@ sysfs_pwm(3)
#define device_create_file_pwm(client, offset) \ #define device_create_file_pwm(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_pwm##offset); \ device_create_file(&client->dev, &dev_attr_fan##offset##_pwm); \
} while (0) } while (0)
static ssize_t static ssize_t
...@@ -871,7 +871,7 @@ store_regs_sensor_##offset (struct device *dev, const char *buf, size_t count) \ ...@@ -871,7 +871,7 @@ store_regs_sensor_##offset (struct device *dev, const char *buf, size_t count) \
{ \ { \
return store_sensor_reg(dev, buf, count, offset); \ return store_sensor_reg(dev, buf, count, offset); \
} \ } \
static DEVICE_ATTR(sensor##offset, S_IRUGO | S_IWUSR, \ static DEVICE_ATTR(temp##offset##_type, S_IRUGO | S_IWUSR, \
show_regs_sensor_##offset, store_regs_sensor_##offset) show_regs_sensor_##offset, store_regs_sensor_##offset)
sysfs_sensor(1) sysfs_sensor(1)
...@@ -880,7 +880,7 @@ sysfs_sensor(3) ...@@ -880,7 +880,7 @@ sysfs_sensor(3)
#define device_create_file_sensor(client, offset) \ #define device_create_file_sensor(client, offset) \
do { \ do { \
device_create_file(&client->dev, &dev_attr_sensor##offset); \ device_create_file(&client->dev, &dev_attr_temp##offset##_type); \
} while (0) } while (0)
...@@ -1319,20 +1319,20 @@ static void w83627hf_update_client(struct i2c_client *client) ...@@ -1319,20 +1319,20 @@ static void w83627hf_update_client(struct i2c_client *client)
data->temp = w83627hf_read_value(client, W83781D_REG_TEMP(1)); data->temp = w83627hf_read_value(client, W83781D_REG_TEMP(1));
data->temp_max = data->temp_max =
w83627hf_read_value(client, W83781D_REG_TEMP_OVER(1)); w83627hf_read_value(client, W83781D_REG_TEMP_OVER(1));
data->temp_hyst = data->temp_max_hyst =
w83627hf_read_value(client, W83781D_REG_TEMP_HYST(1)); w83627hf_read_value(client, W83781D_REG_TEMP_HYST(1));
data->temp_add[0] = data->temp_add[0] =
w83627hf_read_value(client, W83781D_REG_TEMP(2)); w83627hf_read_value(client, W83781D_REG_TEMP(2));
data->temp_max_add[0] = data->temp_max_add[0] =
w83627hf_read_value(client, W83781D_REG_TEMP_OVER(2)); w83627hf_read_value(client, W83781D_REG_TEMP_OVER(2));
data->temp_hyst_add[0] = data->temp_max_hyst_add[0] =
w83627hf_read_value(client, W83781D_REG_TEMP_HYST(2)); w83627hf_read_value(client, W83781D_REG_TEMP_HYST(2));
if (data->type != w83697hf) { if (data->type != w83697hf) {
data->temp_add[1] = data->temp_add[1] =
w83627hf_read_value(client, W83781D_REG_TEMP(3)); w83627hf_read_value(client, W83781D_REG_TEMP(3));
data->temp_max_add[1] = data->temp_max_add[1] =
w83627hf_read_value(client, W83781D_REG_TEMP_OVER(3)); w83627hf_read_value(client, W83781D_REG_TEMP_OVER(3));
data->temp_hyst_add[1] = data->temp_max_hyst_add[1] =
w83627hf_read_value(client, W83781D_REG_TEMP_HYST(3)); w83627hf_read_value(client, W83781D_REG_TEMP_HYST(3));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment