diff options
author | Jean Delvare <khali@linux-fr.org> | 2010-10-28 20:31:46 +0200 |
---|---|---|
committer | Jean Delvare <khali@endymion.delvare> | 2010-10-28 20:31:46 +0200 |
commit | b2cc528e5838d744892b30c5104bd872a439088f (patch) | |
tree | 27f3a0718da764b23addbc99e06487dc0918ac83 | |
parent | b5f6a90a0bbedc07a0b0b82b83aa716c7203fcd2 (diff) |
hwmon: (w83795) Only create fan[1-8]_target files when needed
Only create fan[1-8]_target files when the fan in question can be
controlled (PWM output is present.) Also name these files according
to the standard.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
-rw-r--r-- | drivers/hwmon/w83795.c | 27 |
1 files changed, 4 insertions, 23 deletions
diff --git a/drivers/hwmon/w83795.c b/drivers/hwmon/w83795.c index 76e89c7bc46..00d64b0c200 100644 --- a/drivers/hwmon/w83795.c +++ b/drivers/hwmon/w83795.c @@ -1482,11 +1482,9 @@ store_sf_setup(struct device *dev, struct device_attribute *attr, SENSOR_ATTR_2(fan##index##_div, S_IWUSR | S_IRUGO, \ show_pwm, store_pwm, PWM_DIV, index - 1), \ SENSOR_ATTR_2(pwm##index##_enable, S_IWUSR | S_IRUGO, \ - show_pwm_enable, store_pwm_enable, NOT_USED, index - 1) } - -#define SENSOR_ATTR_FANIN_TARGET(index) \ - SENSOR_ATTR_2(speed_cruise##index##_target, S_IWUSR | S_IRUGO, \ - show_fanin, store_fanin, FANIN_TARGET, index - 1) + show_pwm_enable, store_pwm_enable, NOT_USED, index - 1), \ + SENSOR_ATTR_2(fan##index##_target, S_IWUSR | S_IRUGO, \ + show_fanin, store_fanin, FANIN_TARGET, index - 1) } #define SENSOR_ATTR_DTS(index) { \ SENSOR_ATTR_2(temp##index##_type, S_IRUGO , \ @@ -1632,18 +1630,7 @@ static struct sensor_device_attribute_2 w83795_dts[][8] = { SENSOR_ATTR_DTS(14), }; -static struct sensor_device_attribute_2 w83795_static[] = { - SENSOR_ATTR_FANIN_TARGET(1), - SENSOR_ATTR_FANIN_TARGET(2), - SENSOR_ATTR_FANIN_TARGET(3), - SENSOR_ATTR_FANIN_TARGET(4), - SENSOR_ATTR_FANIN_TARGET(5), - SENSOR_ATTR_FANIN_TARGET(6), - SENSOR_ATTR_FANIN_TARGET(7), - SENSOR_ATTR_FANIN_TARGET(8), -}; - -static struct sensor_device_attribute_2 w83795_pwm[][6] = { +static struct sensor_device_attribute_2 w83795_pwm[][7] = { SENSOR_ATTR_PWM(1), SENSOR_ATTR_PWM(2), SENSOR_ATTR_PWM(3), @@ -1837,12 +1824,6 @@ static int w83795_handle_files(struct device *dev, int (*fn)(struct device *, } } - for (i = 0; i < ARRAY_SIZE(w83795_static); i++) { - err = fn(dev, &w83795_static[i].dev_attr); - if (err) - return err; - } - return 0; } |