forked from Minki/linux
hwmon: (pc87360) Fix: do not use assignment in if condition
Fix checkpatch issue: ERROR: do not use assignment in if condition Replace repeated calls to device_create_file() with calls to sysfs_create_group. Cc: Jean Delvare <khali@linux-fr.org> Cc: Jim Cromie <jim.cromie@gmail.com> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
This commit is contained in:
Guenter Roeck
Guenter Roeck
parent
449a7a0720
commit
bce2778df9
@ -346,11 +346,13 @@ static struct sensor_device_attribute fan_min[] = {
|
||||
SENSOR_ATTR(fan3_min, S_IWUSR | S_IRUGO, show_fan_min, set_fan_min, 2),
|
||||
};
|
||||
|
||||
#define FAN_UNIT_ATTRS(X) \
|
||||
&fan_input[X].dev_attr.attr, \
|
||||
#define FAN_UNIT_ATTRS(X) \
|
||||
{ &fan_input[X].dev_attr.attr, \
|
||||
&fan_status[X].dev_attr.attr, \
|
||||
&fan_div[X].dev_attr.attr, \
|
||||
&fan_min[X].dev_attr.attr
|
||||
&fan_min[X].dev_attr.attr, \
|
||||
NULL \
|
||||
}
|
||||
|
||||
static ssize_t show_pwm(struct device *dev, struct device_attribute *devattr,
|
||||
char *buf)
|
||||
@ -389,17 +391,16 @@ static struct sensor_device_attribute pwm[] = {
|
||||
SENSOR_ATTR(pwm3, S_IWUSR | S_IRUGO, show_pwm, set_pwm, 2),
|
||||
};
|
||||
|
||||
static struct attribute *pc8736x_fan_attr_array[] = {
|
||||
static struct attribute *pc8736x_fan_attr[][5] = {
|
||||
FAN_UNIT_ATTRS(0),
|
||||
FAN_UNIT_ATTRS(1),
|
||||
FAN_UNIT_ATTRS(2),
|
||||
&pwm[0].dev_attr.attr,
|
||||
&pwm[1].dev_attr.attr,
|
||||
&pwm[2].dev_attr.attr,
|
||||
NULL
|
||||
FAN_UNIT_ATTRS(2)
|
||||
};
|
||||
static const struct attribute_group pc8736x_fan_group = {
|
||||
.attrs = pc8736x_fan_attr_array,
|
||||
|
||||
static const struct attribute_group pc8736x_fan_attr_group[] = {
|
||||
{ .attrs = pc8736x_fan_attr[0], },
|
||||
{ .attrs = pc8736x_fan_attr[1], },
|
||||
{ .attrs = pc8736x_fan_attr[2], },
|
||||
};
|
||||
|
||||
static ssize_t show_in_input(struct device *dev,
|
||||
@ -1078,28 +1079,29 @@ static struct sensor_device_attribute temp_fault[] = {
|
||||
SENSOR_ATTR(temp3_fault, S_IRUGO, show_temp_fault, NULL, 2),
|
||||
};
|
||||
|
||||
#define TEMP_UNIT_ATTRS(X) \
|
||||
&temp_input[X].dev_attr.attr, \
|
||||
&temp_status[X].dev_attr.attr, \
|
||||
&temp_min[X].dev_attr.attr, \
|
||||
&temp_max[X].dev_attr.attr, \
|
||||
&temp_crit[X].dev_attr.attr, \
|
||||
&temp_min_alarm[X].dev_attr.attr, \
|
||||
&temp_max_alarm[X].dev_attr.attr, \
|
||||
&temp_crit_alarm[X].dev_attr.attr, \
|
||||
&temp_fault[X].dev_attr.attr
|
||||
#define TEMP_UNIT_ATTRS(X) \
|
||||
{ &temp_input[X].dev_attr.attr, \
|
||||
&temp_status[X].dev_attr.attr, \
|
||||
&temp_min[X].dev_attr.attr, \
|
||||
&temp_max[X].dev_attr.attr, \
|
||||
&temp_crit[X].dev_attr.attr, \
|
||||
&temp_min_alarm[X].dev_attr.attr, \
|
||||
&temp_max_alarm[X].dev_attr.attr, \
|
||||
&temp_crit_alarm[X].dev_attr.attr, \
|
||||
&temp_fault[X].dev_attr.attr, \
|
||||
NULL \
|
||||
}
|
||||
|
||||
static struct attribute *pc8736x_temp_attr_array[] = {
|
||||
static struct attribute *pc8736x_temp_attr[][10] = {
|
||||
TEMP_UNIT_ATTRS(0),
|
||||
TEMP_UNIT_ATTRS(1),
|
||||
TEMP_UNIT_ATTRS(2),
|
||||
/* include the few miscellaneous atts here */
|
||||
&dev_attr_alarms_temp.attr,
|
||||
NULL
|
||||
TEMP_UNIT_ATTRS(2)
|
||||
};
|
||||
|
||||
static const struct attribute_group pc8736x_temp_group = {
|
||||
.attrs = pc8736x_temp_attr_array,
|
||||
static const struct attribute_group pc8736x_temp_attr_group[] = {
|
||||
{ .attrs = pc8736x_temp_attr[0] },
|
||||
{ .attrs = pc8736x_temp_attr[1] },
|
||||
{ .attrs = pc8736x_temp_attr[2] }
|
||||
};
|
||||
|
||||
static ssize_t show_name(struct device *dev,
|
||||
@ -1203,6 +1205,22 @@ static int __init pc87360_find(int sioaddr, u8 *devid,
|
||||
return 0;
|
||||
}
|
||||
|
||||
static void pc87360_remove_files(struct device *dev)
|
||||
{
|
||||
int i;
|
||||
|
||||
device_remove_file(dev, &dev_attr_name);
|
||||
device_remove_file(dev, &dev_attr_alarms_temp);
|
||||
for (i = 0; i < ARRAY_SIZE(pc8736x_temp_attr_group); i++)
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_temp_attr_group[i]);
|
||||
for (i = 0; i < ARRAY_SIZE(pc8736x_fan_attr_group); i++) {
|
||||
sysfs_remove_group(&pdev->dev.kobj, &pc8736x_fan_attr_group[i]);
|
||||
device_remove_file(dev, &pwm[i].dev_attr);
|
||||
}
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_therm_group);
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_vin_group);
|
||||
}
|
||||
|
||||
static int __devinit pc87360_probe(struct platform_device *pdev)
|
||||
{
|
||||
int i;
|
||||
@ -1320,24 +1338,9 @@ static int __devinit pc87360_probe(struct platform_device *pdev)
|
||||
|
||||
if (data->tempnr) {
|
||||
for (i = 0; i < data->tempnr; i++) {
|
||||
if ((err = device_create_file(dev,
|
||||
&temp_input[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_min[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_max[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_crit[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_status[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_min_alarm[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_max_alarm[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_crit_alarm[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&temp_fault[i].dev_attr)))
|
||||
err = sysfs_create_group(&dev->kobj,
|
||||
&pc8736x_temp_attr_group[i]);
|
||||
if (err)
|
||||
goto ERROR3;
|
||||
}
|
||||
err = device_create_file(dev, &dev_attr_alarms_temp);
|
||||
@ -1346,17 +1349,12 @@ static int __devinit pc87360_probe(struct platform_device *pdev)
|
||||
}
|
||||
|
||||
for (i = 0; i < data->fannr; i++) {
|
||||
if (FAN_CONFIG_MONITOR(data->fan_conf, i)
|
||||
&& ((err = device_create_file(dev,
|
||||
&fan_input[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&fan_min[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&fan_div[i].dev_attr))
|
||||
|| (err = device_create_file(dev,
|
||||
&fan_status[i].dev_attr))))
|
||||
goto ERROR3;
|
||||
|
||||
if (FAN_CONFIG_MONITOR(data->fan_conf, i)) {
|
||||
err = sysfs_create_group(&dev->kobj,
|
||||
&pc8736x_fan_attr_group[i]);
|
||||
if (err)
|
||||
goto ERROR3;
|
||||
}
|
||||
if (FAN_CONFIG_CONTROL(data->fan_conf, i)) {
|
||||
err = device_create_file(dev, &pwm[i].dev_attr);
|
||||
if (err)
|
||||
@ -1376,12 +1374,7 @@ static int __devinit pc87360_probe(struct platform_device *pdev)
|
||||
return 0;
|
||||
|
||||
ERROR3:
|
||||
device_remove_file(dev, &dev_attr_name);
|
||||
/* can still remove groups whose members were added individually */
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_temp_group);
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_fan_group);
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_therm_group);
|
||||
sysfs_remove_group(&dev->kobj, &pc8736x_vin_group);
|
||||
pc87360_remove_files(dev);
|
||||
for (i = 0; i < 3; i++) {
|
||||
if (data->address[i])
|
||||
release_region(data->address[i], PC87360_EXTENT);
|
||||
@ -1397,13 +1390,7 @@ static int __devexit pc87360_remove(struct platform_device *pdev)
|
||||
int i;
|
||||
|
||||
hwmon_device_unregister(data->hwmon_dev);
|
||||
|
||||
device_remove_file(&pdev->dev, &dev_attr_name);
|
||||
sysfs_remove_group(&pdev->dev.kobj, &pc8736x_temp_group);
|
||||
sysfs_remove_group(&pdev->dev.kobj, &pc8736x_fan_group);
|
||||
sysfs_remove_group(&pdev->dev.kobj, &pc8736x_therm_group);
|
||||
sysfs_remove_group(&pdev->dev.kobj, &pc8736x_vin_group);
|
||||
|
||||
pc87360_remove_files(&pdev->dev);
|
||||
for (i = 0; i < 3; i++) {
|
||||
if (data->address[i])
|
||||
release_region(data->address[i], PC87360_EXTENT);
|
||||
|
||||
Loading…
Reference in New Issue
Block a user