Skip to content

K
kernel_linux

OpenHarmony / kernel_linux
上一次同步 4 年多

通知 15
Star 8
Fork 2
K
kernel_linux
“ee893e9adccd7dc22a6cfc5d4f49e7dd0baead0c”上不存在“drivers/media/git@gitcode.net:openharmony/kernel_linux.git”
提交 2185696d 编写于 作者: G Guenter Roeck 提交者： Guenter Roeck
浏览文件
操作

hwmon: (vt1211) 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: Juerg Haefliger <juergh@gmail.com>
Signed-off-by: NGuenter Roeck <linux@roeck-us.net>
上级 b162c033
隐藏空白更改
内联 并排
Showing with 94 addition and 133 deletion
drivers/hwmon/vt1211.c
浏览文件 @ 2185696d
...@@ -909,112 +909,99 @@ static ssize_t show_alarms(struct device *dev, ...@@ -909,112 +909,99 @@ static ssize_t show_alarms(struct device *dev,
* Device attribute structs * Device attribute structs
* --------------------------------------------------------------------- */ * --------------------------------------------------------------------- */
#define SENSOR_ATTR_IN_INPUT(ix) \ #define SENSOR_ATTR_IN(ix) \
SENSOR_ATTR_2(in##ix##_input, S_IRUGO, \ { SENSOR_ATTR_2(in##ix##_input, S_IRUGO, \
show_in, NULL, SHOW_IN_INPUT, ix) show_in, NULL, SHOW_IN_INPUT, ix), \
static struct sensor_device_attribute_2 vt1211_sysfs_in_input[] = {
SENSOR_ATTR_IN_INPUT(0),
SENSOR_ATTR_IN_INPUT(1),
SENSOR_ATTR_IN_INPUT(2),
SENSOR_ATTR_IN_INPUT(3),
SENSOR_ATTR_IN_INPUT(4),
SENSOR_ATTR_IN_INPUT(5),
};
#define SENSOR_ATTR_IN_MIN(ix) \
SENSOR_ATTR_2(in##ix##_min, S_IRUGO | S_IWUSR, \ SENSOR_ATTR_2(in##ix##_min, S_IRUGO | S_IWUSR, \
show_in, set_in, SHOW_SET_IN_MIN, ix) show_in, set_in, SHOW_SET_IN_MIN, ix), \
static struct sensor_device_attribute_2 vt1211_sysfs_in_min[] = {
SENSOR_ATTR_IN_MIN(0),
SENSOR_ATTR_IN_MIN(1),
SENSOR_ATTR_IN_MIN(2),
SENSOR_ATTR_IN_MIN(3),
SENSOR_ATTR_IN_MIN(4),
SENSOR_ATTR_IN_MIN(5),
};
#define SENSOR_ATTR_IN_MAX(ix) \
SENSOR_ATTR_2(in##ix##_max, S_IRUGO | S_IWUSR, \ SENSOR_ATTR_2(in##ix##_max, S_IRUGO | S_IWUSR, \
show_in, set_in, SHOW_SET_IN_MAX, ix) show_in, set_in, SHOW_SET_IN_MAX, ix), \
SENSOR_ATTR_2(in##ix##_alarm, S_IRUGO, \
static struct sensor_device_attribute_2 vt1211_sysfs_in_max[] = { show_in, NULL, SHOW_IN_ALARM, ix) \
SENSOR_ATTR_IN_MAX(0), }
SENSOR_ATTR_IN_MAX(1),
SENSOR_ATTR_IN_MAX(2), static struct sensor_device_attribute_2 vt1211_sysfs_in[][4] = {
SENSOR_ATTR_IN_MAX(3), SENSOR_ATTR_IN(0),
SENSOR_ATTR_IN_MAX(4), SENSOR_ATTR_IN(1),
SENSOR_ATTR_IN_MAX(5), SENSOR_ATTR_IN(2),
SENSOR_ATTR_IN(3),
SENSOR_ATTR_IN(4),
SENSOR_ATTR_IN(5)
}; };
#define SENSOR_ATTR_IN_ALARM(ix) \ #define IN_UNIT_ATTRS(X) \
SENSOR_ATTR_2(in##ix##_alarm, S_IRUGO, \ { &vt1211_sysfs_in[X][0].dev_attr.attr, \
show_in, NULL, SHOW_IN_ALARM, ix) &vt1211_sysfs_in[X][1].dev_attr.attr, \
&vt1211_sysfs_in[X][2].dev_attr.attr, \
static struct sensor_device_attribute_2 vt1211_sysfs_in_alarm[] = { &vt1211_sysfs_in[X][3].dev_attr.attr, \
SENSOR_ATTR_IN_ALARM(0), NULL \
SENSOR_ATTR_IN_ALARM(1), }
SENSOR_ATTR_IN_ALARM(2),
SENSOR_ATTR_IN_ALARM(3), static struct attribute *vt1211_in_attr[][5] = {
SENSOR_ATTR_IN_ALARM(4), IN_UNIT_ATTRS(0),
SENSOR_ATTR_IN_ALARM(5), IN_UNIT_ATTRS(1),
IN_UNIT_ATTRS(2),
IN_UNIT_ATTRS(3),
IN_UNIT_ATTRS(4),
IN_UNIT_ATTRS(5)
}; };
#define SENSOR_ATTR_TEMP_INPUT(ix) \ static const struct attribute_group vt1211_in_attr_group[] = {
SENSOR_ATTR_2(temp##ix##_input, S_IRUGO, \ { .attrs = vt1211_in_attr[0] },
show_temp, NULL, SHOW_TEMP_INPUT, ix-1) { .attrs = vt1211_in_attr[1] },
{ .attrs = vt1211_in_attr[2] },
static struct sensor_device_attribute_2 vt1211_sysfs_temp_input[] = { { .attrs = vt1211_in_attr[3] },
SENSOR_ATTR_TEMP_INPUT(1), { .attrs = vt1211_in_attr[4] },
SENSOR_ATTR_TEMP_INPUT(2), { .attrs = vt1211_in_attr[5] }
SENSOR_ATTR_TEMP_INPUT(3),
SENSOR_ATTR_TEMP_INPUT(4),
SENSOR_ATTR_TEMP_INPUT(5),
SENSOR_ATTR_TEMP_INPUT(6),
SENSOR_ATTR_TEMP_INPUT(7),
}; };
#define SENSOR_ATTR_TEMP_MAX(ix) \ #define SENSOR_ATTR_TEMP(ix) \
{ SENSOR_ATTR_2(temp##ix##_input, S_IRUGO, \
show_temp, NULL, SHOW_TEMP_INPUT, ix-1), \
SENSOR_ATTR_2(temp##ix##_max, S_IRUGO | S_IWUSR, \ SENSOR_ATTR_2(temp##ix##_max, S_IRUGO | S_IWUSR, \
show_temp, set_temp, SHOW_SET_TEMP_MAX, ix-1) show_temp, set_temp, SHOW_SET_TEMP_MAX, ix-1), \
SENSOR_ATTR_2(temp##ix##_max_hyst, S_IRUGO | S_IWUSR, \
static struct sensor_device_attribute_2 vt1211_sysfs_temp_max[] = { show_temp, set_temp, SHOW_SET_TEMP_MAX_HYST, ix-1), \
SENSOR_ATTR_TEMP_MAX(1), SENSOR_ATTR_2(temp##ix##_alarm, S_IRUGO, \
SENSOR_ATTR_TEMP_MAX(2), show_temp, NULL, SHOW_TEMP_ALARM, ix-1) \
SENSOR_ATTR_TEMP_MAX(3), }
SENSOR_ATTR_TEMP_MAX(4),
SENSOR_ATTR_TEMP_MAX(5), static struct sensor_device_attribute_2 vt1211_sysfs_temp[][4] = {
SENSOR_ATTR_TEMP_MAX(6), SENSOR_ATTR_TEMP(1),
SENSOR_ATTR_TEMP_MAX(7), SENSOR_ATTR_TEMP(2),
SENSOR_ATTR_TEMP(3),
SENSOR_ATTR_TEMP(4),
SENSOR_ATTR_TEMP(5),
SENSOR_ATTR_TEMP(6),
SENSOR_ATTR_TEMP(7),
}; };
#define SENSOR_ATTR_TEMP_MAX_HYST(ix) \ #define TEMP_UNIT_ATTRS(X) \
SENSOR_ATTR_2(temp##ix##_max_hyst, S_IRUGO | S_IWUSR, \ { &vt1211_sysfs_temp[X][0].dev_attr.attr, \
show_temp, set_temp, SHOW_SET_TEMP_MAX_HYST, ix-1) &vt1211_sysfs_temp[X][1].dev_attr.attr, \
&vt1211_sysfs_temp[X][2].dev_attr.attr, \
static struct sensor_device_attribute_2 vt1211_sysfs_temp_max_hyst[] = { &vt1211_sysfs_temp[X][3].dev_attr.attr, \
SENSOR_ATTR_TEMP_MAX_HYST(1), NULL \
SENSOR_ATTR_TEMP_MAX_HYST(2), }
SENSOR_ATTR_TEMP_MAX_HYST(3),
SENSOR_ATTR_TEMP_MAX_HYST(4), static struct attribute *vt1211_temp_attr[][5] = {
SENSOR_ATTR_TEMP_MAX_HYST(5), TEMP_UNIT_ATTRS(0),
SENSOR_ATTR_TEMP_MAX_HYST(6), TEMP_UNIT_ATTRS(1),
SENSOR_ATTR_TEMP_MAX_HYST(7), TEMP_UNIT_ATTRS(2),
TEMP_UNIT_ATTRS(3),
TEMP_UNIT_ATTRS(4),
TEMP_UNIT_ATTRS(5),
TEMP_UNIT_ATTRS(6)
}; };
#define SENSOR_ATTR_TEMP_ALARM(ix) \ static const struct attribute_group vt1211_temp_attr_group[] = {
SENSOR_ATTR_2(temp##ix##_alarm, S_IRUGO, \ { .attrs = vt1211_temp_attr[0] },
show_temp, NULL, SHOW_TEMP_ALARM, ix-1) { .attrs = vt1211_temp_attr[1] },
{ .attrs = vt1211_temp_attr[2] },
static struct sensor_device_attribute_2 vt1211_sysfs_temp_alarm[] = { { .attrs = vt1211_temp_attr[3] },
SENSOR_ATTR_TEMP_ALARM(1), { .attrs = vt1211_temp_attr[4] },
SENSOR_ATTR_TEMP_ALARM(2), { .attrs = vt1211_temp_attr[5] },
SENSOR_ATTR_TEMP_ALARM(3), { .attrs = vt1211_temp_attr[6] }
SENSOR_ATTR_TEMP_ALARM(4),
SENSOR_ATTR_TEMP_ALARM(5),
SENSOR_ATTR_TEMP_ALARM(6),
SENSOR_ATTR_TEMP_ALARM(7),
}; };
#define SENSOR_ATTR_FAN(ix) \ #define SENSOR_ATTR_FAN(ix) \
...@@ -1140,26 +1127,12 @@ static void vt1211_remove_sysfs(struct platform_device *pdev) ...@@ -1140,26 +1127,12 @@ static void vt1211_remove_sysfs(struct platform_device *pdev)
struct device *dev = &pdev->dev; struct device *dev = &pdev->dev;
int i; int i;
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_in_input); i++) { for (i = 0; i < ARRAY_SIZE(vt1211_in_attr_group); i++)
device_remove_file(dev, sysfs_remove_group(&dev->kobj, &vt1211_in_attr_group[i]);
&vt1211_sysfs_in_input[i].dev_attr);
device_remove_file(dev, for (i = 0; i < ARRAY_SIZE(vt1211_temp_attr_group); i++)
&vt1211_sysfs_in_min[i].dev_attr); sysfs_remove_group(&dev->kobj, &vt1211_temp_attr_group[i]);
device_remove_file(dev,
&vt1211_sysfs_in_max[i].dev_attr);
device_remove_file(dev,
&vt1211_sysfs_in_alarm[i].dev_attr);
}
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_temp_input); i++) {
device_remove_file(dev,
&vt1211_sysfs_temp_input[i].dev_attr);
device_remove_file(dev,
&vt1211_sysfs_temp_max[i].dev_attr);
device_remove_file(dev,
&vt1211_sysfs_temp_max_hyst[i].dev_attr);
device_remove_file(dev,
&vt1211_sysfs_temp_alarm[i].dev_attr);
}
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_fan_pwm); i++) { for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_fan_pwm); i++) {
device_remove_file(dev, device_remove_file(dev,
&vt1211_sysfs_fan_pwm[i].dev_attr); &vt1211_sysfs_fan_pwm[i].dev_attr);
...@@ -1199,32 +1172,20 @@ static int __devinit vt1211_probe(struct platform_device *pdev) ...@@ -1199,32 +1172,20 @@ static int __devinit vt1211_probe(struct platform_device *pdev)
vt1211_init_device(data); vt1211_init_device(data);
/* Create sysfs interface files */ /* Create sysfs interface files */
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_in_input); i++) { for (i = 0; i < ARRAY_SIZE(vt1211_in_attr_group); i++) {
if (ISVOLT(i, data->uch_config)) { if (ISVOLT(i, data->uch_config)) {
if ((err = device_create_file(dev, err = sysfs_create_group(&dev->kobj,
&vt1211_sysfs_in_input[i].dev_attr)) || &vt1211_in_attr_group[i]);
(err = device_create_file(dev, if (err)
&vt1211_sysfs_in_min[i].dev_attr)) ||
(err = device_create_file(dev,
&vt1211_sysfs_in_max[i].dev_attr)) ||
(err = device_create_file(dev,
&vt1211_sysfs_in_alarm[i].dev_attr))) {
goto EXIT_DEV_REMOVE; goto EXIT_DEV_REMOVE;
}
} }
} }
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_temp_input); i++) { for (i = 0; i < ARRAY_SIZE(vt1211_temp_attr_group); i++) {
if (ISTEMP(i, data->uch_config)) { if (ISTEMP(i, data->uch_config)) {
if ((err = device_create_file(dev, err = sysfs_create_group(&dev->kobj,
&vt1211_sysfs_temp_input[i].dev_attr)) || &vt1211_temp_attr_group[i]);
(err = device_create_file(dev, if (err)
&vt1211_sysfs_temp_max[i].dev_attr)) ||
(err = device_create_file(dev,
&vt1211_sysfs_temp_max_hyst[i].dev_attr)) ||
(err = device_create_file(dev,
&vt1211_sysfs_temp_alarm[i].dev_attr))) {
goto EXIT_DEV_REMOVE; goto EXIT_DEV_REMOVE;
}
} }
} }
for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_fan_pwm); i++) { for (i = 0; i < ARRAY_SIZE(vt1211_sysfs_fan_pwm); i++) {
......
Markdown is supported
0% 或 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑！
想要评论请 注册 或