* [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).