* [PATCH v2 0/1] hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() @ 2019-07-21 22:55 Grant McEwan 2019-07-21 22:55 ` [PATCH v2 1/1] " Grant McEwan 0 siblings, 1 reply; 3+ messages in thread From: Grant McEwan @ 2019-07-21 22:55 UTC (permalink / raw) To: jdelvare, linux; +Cc: linux-hwmon, linux-kernel, Grant McEwan V2: Addressing maintainer feedback, - Increased attribute array from 8 to 9, to accommodate NULL terminator. - Removed #define ATTR_GROUP_COUNT. - Replaced hwmon_device_register_with_groups() with devm_hwmon_device_register_with_groups() api. - Removed the adt7475_remove_files() as no longer necessary with new API. - Replaced goto eremove calls, with simple returns. - Removed .remove function. Grant McEwan (1): hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() drivers/hwmon/adt7475.c | 146 ++++++++++++++-------------------------- 1 file changed, 50 insertions(+), 96 deletions(-) -- 2.22.0 ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2 1/1] hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() 2019-07-21 22:55 [PATCH v2 0/1] hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() Grant McEwan @ 2019-07-21 22:55 ` Grant McEwan 2019-07-22 18:35 ` Guenter Roeck 0 siblings, 1 reply; 3+ messages in thread From: Grant McEwan @ 2019-07-21 22:55 UTC (permalink / raw) To: jdelvare, linux; +Cc: linux-hwmon, linux-kernel, Grant McEwan hwmon_device_register() is a deprecated function and produces a warning. Converting the driver to use the hwmon_device_register_with_groups() instead. Signed-off-by: Grant McEwan <grant.mcewan@alliedtelesis.co.nz> --- drivers/hwmon/adt7475.c | 146 ++++++++++++++-------------------------- 1 file changed, 50 insertions(+), 96 deletions(-) diff --git a/drivers/hwmon/adt7475.c b/drivers/hwmon/adt7475.c index c3c6031a7285..6c64d50c9aae 100644 --- a/drivers/hwmon/adt7475.c +++ b/drivers/hwmon/adt7475.c @@ -187,7 +187,7 @@ static const struct of_device_id __maybe_unused adt7475_of_match[] = { MODULE_DEVICE_TABLE(of, adt7475_of_match); struct adt7475_data { - struct device *hwmon_dev; + struct i2c_client *client; struct mutex lock; unsigned long measure_updated; @@ -212,6 +212,7 @@ struct adt7475_data { u8 vid; u8 vrm; + const struct attribute_group *groups[9]; }; static struct i2c_driver adt7475_driver; @@ -346,8 +347,8 @@ static ssize_t voltage_store(struct device *dev, { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; unsigned char reg; long val; @@ -440,8 +441,8 @@ static ssize_t temp_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; unsigned char reg = 0; u8 out; int temp; @@ -542,8 +543,7 @@ static ssize_t temp_st_show(struct device *dev, struct device_attribute *attr, char *buf) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); long val; switch (sattr->index) { @@ -570,8 +570,8 @@ static ssize_t temp_st_store(struct device *dev, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; unsigned char reg; int shift, idx; ulong val; @@ -647,8 +647,8 @@ static ssize_t point2_show(struct device *dev, struct device_attribute *attr, static ssize_t point2_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) { - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); int temp; long val; @@ -710,8 +710,8 @@ static ssize_t tach_store(struct device *dev, struct device_attribute *attr, { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; unsigned long val; if (kstrtoul(buf, 10, &val)) @@ -769,8 +769,8 @@ static ssize_t pwm_store(struct device *dev, struct device_attribute *attr, { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; unsigned char reg = 0; long val; @@ -818,8 +818,8 @@ static ssize_t stall_disable_show(struct device *dev, struct device_attribute *attr, char *buf) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + u8 mask = BIT(5 + sattr->index); return sprintf(buf, "%d\n", !!(data->enh_acoustics[0] & mask)); @@ -830,8 +830,8 @@ static ssize_t stall_disable_store(struct device *dev, const char *buf, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; long val; u8 mask = BIT(5 + sattr->index); @@ -914,8 +914,8 @@ static ssize_t pwmchan_store(struct device *dev, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; int r; long val; @@ -938,8 +938,8 @@ static ssize_t pwmctrl_store(struct device *dev, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; int r; long val; @@ -982,8 +982,8 @@ static ssize_t pwmfreq_store(struct device *dev, size_t count) { struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; int out; long val; @@ -1022,8 +1022,8 @@ static ssize_t pwm_use_point2_pwm_at_crit_store(struct device *dev, struct device_attribute *devattr, const char *buf, size_t count) { - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; long val; if (kstrtol(buf, 10, &val)) @@ -1342,26 +1342,6 @@ static int adt7475_detect(struct i2c_client *client, return 0; } -static void adt7475_remove_files(struct i2c_client *client, - struct adt7475_data *data) -{ - sysfs_remove_group(&client->dev.kobj, &adt7475_attr_group); - if (data->has_fan4) - sysfs_remove_group(&client->dev.kobj, &fan4_attr_group); - if (data->has_pwm2) - sysfs_remove_group(&client->dev.kobj, &pwm2_attr_group); - if (data->has_voltage & (1 << 0)) - sysfs_remove_group(&client->dev.kobj, &in0_attr_group); - if (data->has_voltage & (1 << 3)) - sysfs_remove_group(&client->dev.kobj, &in3_attr_group); - if (data->has_voltage & (1 << 4)) - sysfs_remove_group(&client->dev.kobj, &in4_attr_group); - if (data->has_voltage & (1 << 5)) - sysfs_remove_group(&client->dev.kobj, &in5_attr_group); - if (data->has_vid) - sysfs_remove_group(&client->dev.kobj, &vid_attr_group); -} - static int adt7475_update_limits(struct i2c_client *client) { struct adt7475_data *data = i2c_get_clientdata(client); @@ -1489,7 +1469,8 @@ static int adt7475_probe(struct i2c_client *client, }; struct adt7475_data *data; - int i, ret = 0, revision; + struct device *hwmon_dev; + int i, ret = 0, revision, group_num = 0; u8 config2, config3; data = devm_kzalloc(&client->dev, sizeof(*data), GFP_KERNEL); @@ -1497,6 +1478,7 @@ static int adt7475_probe(struct i2c_client *client, return -ENOMEM; mutex_init(&data->lock); + data->client = client; i2c_set_clientdata(client, data); if (client->dev.of_node) @@ -1590,52 +1572,40 @@ static int adt7475_probe(struct i2c_client *client, break; } - ret = sysfs_create_group(&client->dev.kobj, &adt7475_attr_group); - if (ret) - return ret; + data->groups[group_num++] = &adt7475_attr_group; /* Features that can be disabled individually */ if (data->has_fan4) { - ret = sysfs_create_group(&client->dev.kobj, &fan4_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &fan4_attr_group; } if (data->has_pwm2) { - ret = sysfs_create_group(&client->dev.kobj, &pwm2_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &pwm2_attr_group; } if (data->has_voltage & (1 << 0)) { - ret = sysfs_create_group(&client->dev.kobj, &in0_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &in0_attr_group; } if (data->has_voltage & (1 << 3)) { - ret = sysfs_create_group(&client->dev.kobj, &in3_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &in3_attr_group; } if (data->has_voltage & (1 << 4)) { - ret = sysfs_create_group(&client->dev.kobj, &in4_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &in4_attr_group; } if (data->has_voltage & (1 << 5)) { - ret = sysfs_create_group(&client->dev.kobj, &in5_attr_group); - if (ret) - goto eremove; + data->groups[group_num++] = &in5_attr_group; } if (data->has_vid) { data->vrm = vid_which_vrm(); - ret = sysfs_create_group(&client->dev.kobj, &vid_attr_group); - if (ret) - goto eremove; + data->groups[group_num] = &vid_attr_group; } - data->hwmon_dev = hwmon_device_register(&client->dev); - if (IS_ERR(data->hwmon_dev)) { - ret = PTR_ERR(data->hwmon_dev); - goto eremove; + /* register device with all the acquired attributes */ + hwmon_dev = devm_hwmon_device_register_with_groups(&client->dev, + client->name, data, + data->groups); + + if (IS_ERR(hwmon_dev)) { + ret = PTR_ERR(hwmon_dev); + return ret; } dev_info(&client->dev, "%s device, revision %d\n", @@ -1657,21 +1627,7 @@ static int adt7475_probe(struct i2c_client *client, /* Limits and settings, should never change update more than once */ ret = adt7475_update_limits(client); if (ret) - goto eremove; - - return 0; - -eremove: - adt7475_remove_files(client, data); - return ret; -} - -static int adt7475_remove(struct i2c_client *client) -{ - struct adt7475_data *data = i2c_get_clientdata(client); - - hwmon_device_unregister(data->hwmon_dev); - adt7475_remove_files(client, data); + return ret; return 0; } @@ -1683,7 +1639,6 @@ static struct i2c_driver adt7475_driver = { .of_match_table = of_match_ptr(adt7475_of_match), }, .probe = adt7475_probe, - .remove = adt7475_remove, .id_table = adt7475_id, .detect = adt7475_detect, .address_list = normal_i2c, @@ -1757,8 +1712,8 @@ static void adt7475_read_pwm(struct i2c_client *client, int index) static int adt7475_update_measure(struct device *dev) { - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); + struct i2c_client *client = data->client; u16 ext; int i; int ret; @@ -1854,8 +1809,7 @@ static int adt7475_update_measure(struct device *dev) static struct adt7475_data *adt7475_update_device(struct device *dev) { - struct i2c_client *client = to_i2c_client(dev); - struct adt7475_data *data = i2c_get_clientdata(client); + struct adt7475_data *data = dev_get_drvdata(dev); int ret; mutex_lock(&data->lock); -- 2.22.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2 1/1] hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() 2019-07-21 22:55 ` [PATCH v2 1/1] " Grant McEwan @ 2019-07-22 18:35 ` Guenter Roeck 0 siblings, 0 replies; 3+ messages in thread From: Guenter Roeck @ 2019-07-22 18:35 UTC (permalink / raw) To: Grant McEwan; +Cc: jdelvare, linux-hwmon, linux-kernel On Mon, Jul 22, 2019 at 10:55:30AM +1200, Grant McEwan wrote: > hwmon_device_register() is a deprecated function and produces a warning. > > Converting the driver to use the hwmon_device_register_with_groups() > instead. > > Signed-off-by: Grant McEwan <grant.mcewan@alliedtelesis.co.nz> Applied to hwmon-next. Thanks, Guenter > --- > drivers/hwmon/adt7475.c | 146 ++++++++++++++-------------------------- > 1 file changed, 50 insertions(+), 96 deletions(-) > > diff --git a/drivers/hwmon/adt7475.c b/drivers/hwmon/adt7475.c > index c3c6031a7285..6c64d50c9aae 100644 > --- a/drivers/hwmon/adt7475.c > +++ b/drivers/hwmon/adt7475.c > @@ -187,7 +187,7 @@ static const struct of_device_id __maybe_unused adt7475_of_match[] = { > MODULE_DEVICE_TABLE(of, adt7475_of_match); > > struct adt7475_data { > - struct device *hwmon_dev; > + struct i2c_client *client; > struct mutex lock; > > unsigned long measure_updated; > @@ -212,6 +212,7 @@ struct adt7475_data { > > u8 vid; > u8 vrm; > + const struct attribute_group *groups[9]; > }; > > static struct i2c_driver adt7475_driver; > @@ -346,8 +347,8 @@ static ssize_t voltage_store(struct device *dev, > { > > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > unsigned char reg; > long val; > > @@ -440,8 +441,8 @@ static ssize_t temp_store(struct device *dev, struct device_attribute *attr, > const char *buf, size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > unsigned char reg = 0; > u8 out; > int temp; > @@ -542,8 +543,7 @@ static ssize_t temp_st_show(struct device *dev, struct device_attribute *attr, > char *buf) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > long val; > > switch (sattr->index) { > @@ -570,8 +570,8 @@ static ssize_t temp_st_store(struct device *dev, > size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > unsigned char reg; > int shift, idx; > ulong val; > @@ -647,8 +647,8 @@ static ssize_t point2_show(struct device *dev, struct device_attribute *attr, > static ssize_t point2_store(struct device *dev, struct device_attribute *attr, > const char *buf, size_t count) > { > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > int temp; > long val; > @@ -710,8 +710,8 @@ static ssize_t tach_store(struct device *dev, struct device_attribute *attr, > { > > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > unsigned long val; > > if (kstrtoul(buf, 10, &val)) > @@ -769,8 +769,8 @@ static ssize_t pwm_store(struct device *dev, struct device_attribute *attr, > { > > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > unsigned char reg = 0; > long val; > > @@ -818,8 +818,8 @@ static ssize_t stall_disable_show(struct device *dev, > struct device_attribute *attr, char *buf) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + > u8 mask = BIT(5 + sattr->index); > > return sprintf(buf, "%d\n", !!(data->enh_acoustics[0] & mask)); > @@ -830,8 +830,8 @@ static ssize_t stall_disable_store(struct device *dev, > const char *buf, size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > long val; > u8 mask = BIT(5 + sattr->index); > > @@ -914,8 +914,8 @@ static ssize_t pwmchan_store(struct device *dev, > size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > int r; > long val; > > @@ -938,8 +938,8 @@ static ssize_t pwmctrl_store(struct device *dev, > size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > int r; > long val; > > @@ -982,8 +982,8 @@ static ssize_t pwmfreq_store(struct device *dev, > size_t count) > { > struct sensor_device_attribute_2 *sattr = to_sensor_dev_attr_2(attr); > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > int out; > long val; > > @@ -1022,8 +1022,8 @@ static ssize_t pwm_use_point2_pwm_at_crit_store(struct device *dev, > struct device_attribute *devattr, > const char *buf, size_t count) > { > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > long val; > > if (kstrtol(buf, 10, &val)) > @@ -1342,26 +1342,6 @@ static int adt7475_detect(struct i2c_client *client, > return 0; > } > > -static void adt7475_remove_files(struct i2c_client *client, > - struct adt7475_data *data) > -{ > - sysfs_remove_group(&client->dev.kobj, &adt7475_attr_group); > - if (data->has_fan4) > - sysfs_remove_group(&client->dev.kobj, &fan4_attr_group); > - if (data->has_pwm2) > - sysfs_remove_group(&client->dev.kobj, &pwm2_attr_group); > - if (data->has_voltage & (1 << 0)) > - sysfs_remove_group(&client->dev.kobj, &in0_attr_group); > - if (data->has_voltage & (1 << 3)) > - sysfs_remove_group(&client->dev.kobj, &in3_attr_group); > - if (data->has_voltage & (1 << 4)) > - sysfs_remove_group(&client->dev.kobj, &in4_attr_group); > - if (data->has_voltage & (1 << 5)) > - sysfs_remove_group(&client->dev.kobj, &in5_attr_group); > - if (data->has_vid) > - sysfs_remove_group(&client->dev.kobj, &vid_attr_group); > -} > - > static int adt7475_update_limits(struct i2c_client *client) > { > struct adt7475_data *data = i2c_get_clientdata(client); > @@ -1489,7 +1469,8 @@ static int adt7475_probe(struct i2c_client *client, > }; > > struct adt7475_data *data; > - int i, ret = 0, revision; > + struct device *hwmon_dev; > + int i, ret = 0, revision, group_num = 0; > u8 config2, config3; > > data = devm_kzalloc(&client->dev, sizeof(*data), GFP_KERNEL); > @@ -1497,6 +1478,7 @@ static int adt7475_probe(struct i2c_client *client, > return -ENOMEM; > > mutex_init(&data->lock); > + data->client = client; > i2c_set_clientdata(client, data); > > if (client->dev.of_node) > @@ -1590,52 +1572,40 @@ static int adt7475_probe(struct i2c_client *client, > break; > } > > - ret = sysfs_create_group(&client->dev.kobj, &adt7475_attr_group); > - if (ret) > - return ret; > + data->groups[group_num++] = &adt7475_attr_group; > > /* Features that can be disabled individually */ > if (data->has_fan4) { > - ret = sysfs_create_group(&client->dev.kobj, &fan4_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &fan4_attr_group; > } > if (data->has_pwm2) { > - ret = sysfs_create_group(&client->dev.kobj, &pwm2_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &pwm2_attr_group; > } > if (data->has_voltage & (1 << 0)) { > - ret = sysfs_create_group(&client->dev.kobj, &in0_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &in0_attr_group; > } > if (data->has_voltage & (1 << 3)) { > - ret = sysfs_create_group(&client->dev.kobj, &in3_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &in3_attr_group; > } > if (data->has_voltage & (1 << 4)) { > - ret = sysfs_create_group(&client->dev.kobj, &in4_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &in4_attr_group; > } > if (data->has_voltage & (1 << 5)) { > - ret = sysfs_create_group(&client->dev.kobj, &in5_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num++] = &in5_attr_group; > } > if (data->has_vid) { > data->vrm = vid_which_vrm(); > - ret = sysfs_create_group(&client->dev.kobj, &vid_attr_group); > - if (ret) > - goto eremove; > + data->groups[group_num] = &vid_attr_group; > } > > - data->hwmon_dev = hwmon_device_register(&client->dev); > - if (IS_ERR(data->hwmon_dev)) { > - ret = PTR_ERR(data->hwmon_dev); > - goto eremove; > + /* register device with all the acquired attributes */ > + hwmon_dev = devm_hwmon_device_register_with_groups(&client->dev, > + client->name, data, > + data->groups); > + > + if (IS_ERR(hwmon_dev)) { > + ret = PTR_ERR(hwmon_dev); > + return ret; > } > > dev_info(&client->dev, "%s device, revision %d\n", > @@ -1657,21 +1627,7 @@ static int adt7475_probe(struct i2c_client *client, > /* Limits and settings, should never change update more than once */ > ret = adt7475_update_limits(client); > if (ret) > - goto eremove; > - > - return 0; > - > -eremove: > - adt7475_remove_files(client, data); > - return ret; > -} > - > -static int adt7475_remove(struct i2c_client *client) > -{ > - struct adt7475_data *data = i2c_get_clientdata(client); > - > - hwmon_device_unregister(data->hwmon_dev); > - adt7475_remove_files(client, data); > + return ret; > > return 0; > } > @@ -1683,7 +1639,6 @@ static struct i2c_driver adt7475_driver = { > .of_match_table = of_match_ptr(adt7475_of_match), > }, > .probe = adt7475_probe, > - .remove = adt7475_remove, > .id_table = adt7475_id, > .detect = adt7475_detect, > .address_list = normal_i2c, > @@ -1757,8 +1712,8 @@ static void adt7475_read_pwm(struct i2c_client *client, int index) > > static int adt7475_update_measure(struct device *dev) > { > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > + struct i2c_client *client = data->client; > u16 ext; > int i; > int ret; > @@ -1854,8 +1809,7 @@ static int adt7475_update_measure(struct device *dev) > > static struct adt7475_data *adt7475_update_device(struct device *dev) > { > - struct i2c_client *client = to_i2c_client(dev); > - struct adt7475_data *data = i2c_get_clientdata(client); > + struct adt7475_data *data = dev_get_drvdata(dev); > int ret; > > mutex_lock(&data->lock); ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-07-22 18:35 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-21 22:55 [PATCH v2 0/1] hwmon: (adt7475) Convert to use hwmon_device_register_with_groups() Grant McEwan 2019-07-21 22:55 ` [PATCH v2 1/1] " Grant McEwan 2019-07-22 18:35 ` Guenter Roeck
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).