Commit 62a1d05f authored by Guenter Roeck's avatar Guenter Roeck Committed by Guenter Roeck

hwmon: (it87) Convert to use devm_kzalloc and devm_request_region

This makes the code a bit simpler and smaller.
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
Acked-by: default avatarJean Delvare <khali@linux-fr.org>
parent f7c2fe38
...@@ -1965,19 +1965,17 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -1965,19 +1965,17 @@ static int __devinit it87_probe(struct platform_device *pdev)
}; };
res = platform_get_resource(pdev, IORESOURCE_IO, 0); res = platform_get_resource(pdev, IORESOURCE_IO, 0);
if (!request_region(res->start, IT87_EC_EXTENT, DRVNAME)) { if (!devm_request_region(&pdev->dev, res->start, IT87_EC_EXTENT,
DRVNAME)) {
dev_err(dev, "Failed to request region 0x%lx-0x%lx\n", dev_err(dev, "Failed to request region 0x%lx-0x%lx\n",
(unsigned long)res->start, (unsigned long)res->start,
(unsigned long)(res->start + IT87_EC_EXTENT - 1)); (unsigned long)(res->start + IT87_EC_EXTENT - 1));
err = -EBUSY; return -EBUSY;
goto ERROR0;
} }
data = kzalloc(sizeof(struct it87_data), GFP_KERNEL); data = devm_kzalloc(&pdev->dev, sizeof(struct it87_data), GFP_KERNEL);
if (!data) { if (!data)
err = -ENOMEM; return -ENOMEM;
goto ERROR1;
}
data->addr = res->start; data->addr = res->start;
data->type = sio_data->type; data->type = sio_data->type;
...@@ -1986,10 +1984,8 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -1986,10 +1984,8 @@ static int __devinit it87_probe(struct platform_device *pdev)
/* Now, we do the remaining detection. */ /* Now, we do the remaining detection. */
if ((it87_read_value(data, IT87_REG_CONFIG) & 0x80) if ((it87_read_value(data, IT87_REG_CONFIG) & 0x80)
|| it87_read_value(data, IT87_REG_CHIPID) != 0x90) { || it87_read_value(data, IT87_REG_CHIPID) != 0x90)
err = -ENODEV; return -ENODEV;
goto ERROR2;
}
platform_set_drvdata(pdev, data); platform_set_drvdata(pdev, data);
...@@ -2019,26 +2015,26 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -2019,26 +2015,26 @@ static int __devinit it87_probe(struct platform_device *pdev)
/* Register sysfs hooks */ /* Register sysfs hooks */
err = sysfs_create_group(&dev->kobj, &it87_group); err = sysfs_create_group(&dev->kobj, &it87_group);
if (err) if (err)
goto ERROR2; return err;
for (i = 0; i < 9; i++) { for (i = 0; i < 9; i++) {
if (sio_data->skip_in & (1 << i)) if (sio_data->skip_in & (1 << i))
continue; continue;
err = sysfs_create_group(&dev->kobj, &it87_group_in[i]); err = sysfs_create_group(&dev->kobj, &it87_group_in[i]);
if (err) if (err)
goto ERROR4; goto error;
if (sio_data->beep_pin && it87_attributes_in_beep[i]) { if (sio_data->beep_pin && it87_attributes_in_beep[i]) {
err = sysfs_create_file(&dev->kobj, err = sysfs_create_file(&dev->kobj,
it87_attributes_in_beep[i]); it87_attributes_in_beep[i]);
if (err) if (err)
goto ERROR4; goto error;
} }
} }
if (sio_data->beep_pin) { if (sio_data->beep_pin) {
err = sysfs_create_group(&dev->kobj, &it87_group_beep); err = sysfs_create_group(&dev->kobj, &it87_group_beep);
if (err) if (err)
goto ERROR4; goto error;
} }
/* Do not create fan files for disabled fans */ /* Do not create fan files for disabled fans */
...@@ -2049,13 +2045,13 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -2049,13 +2045,13 @@ static int __devinit it87_probe(struct platform_device *pdev)
continue; continue;
err = sysfs_create_group(&dev->kobj, &fan_group[i]); err = sysfs_create_group(&dev->kobj, &fan_group[i]);
if (err) if (err)
goto ERROR4; goto error;
if (sio_data->beep_pin) { if (sio_data->beep_pin) {
err = sysfs_create_file(&dev->kobj, err = sysfs_create_file(&dev->kobj,
it87_attributes_fan_beep[i]); it87_attributes_fan_beep[i]);
if (err) if (err)
goto ERROR4; goto error;
if (!fan_beep_need_rw) if (!fan_beep_need_rw)
continue; continue;
...@@ -2080,14 +2076,14 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -2080,14 +2076,14 @@ static int __devinit it87_probe(struct platform_device *pdev)
err = sysfs_create_group(&dev->kobj, err = sysfs_create_group(&dev->kobj,
&it87_group_pwm[i]); &it87_group_pwm[i]);
if (err) if (err)
goto ERROR4; goto error;
if (!has_old_autopwm(data)) if (!has_old_autopwm(data))
continue; continue;
err = sysfs_create_group(&dev->kobj, err = sysfs_create_group(&dev->kobj,
&it87_group_autopwm[i]); &it87_group_autopwm[i]);
if (err) if (err)
goto ERROR4; goto error;
} }
} }
...@@ -2097,7 +2093,7 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -2097,7 +2093,7 @@ static int __devinit it87_probe(struct platform_device *pdev)
data->vid = sio_data->vid_value; data->vid = sio_data->vid_value;
err = sysfs_create_group(&dev->kobj, &it87_group_vid); err = sysfs_create_group(&dev->kobj, &it87_group_vid);
if (err) if (err)
goto ERROR4; goto error;
} }
/* Export labels for internal sensors */ /* Export labels for internal sensors */
...@@ -2107,25 +2103,19 @@ static int __devinit it87_probe(struct platform_device *pdev) ...@@ -2107,25 +2103,19 @@ static int __devinit it87_probe(struct platform_device *pdev)
err = sysfs_create_file(&dev->kobj, err = sysfs_create_file(&dev->kobj,
it87_attributes_label[i]); it87_attributes_label[i]);
if (err) if (err)
goto ERROR4; goto error;
} }
data->hwmon_dev = hwmon_device_register(dev); data->hwmon_dev = hwmon_device_register(dev);
if (IS_ERR(data->hwmon_dev)) { if (IS_ERR(data->hwmon_dev)) {
err = PTR_ERR(data->hwmon_dev); err = PTR_ERR(data->hwmon_dev);
goto ERROR4; goto error;
} }
return 0; return 0;
ERROR4: error:
it87_remove_files(dev); it87_remove_files(dev);
ERROR2:
platform_set_drvdata(pdev, NULL);
kfree(data);
ERROR1:
release_region(res->start, IT87_EC_EXTENT);
ERROR0:
return err; return err;
} }
...@@ -2136,10 +2126,6 @@ static int __devexit it87_remove(struct platform_device *pdev) ...@@ -2136,10 +2126,6 @@ static int __devexit it87_remove(struct platform_device *pdev)
hwmon_device_unregister(data->hwmon_dev); hwmon_device_unregister(data->hwmon_dev);
it87_remove_files(&pdev->dev); it87_remove_files(&pdev->dev);
release_region(data->addr, IT87_EC_EXTENT);
platform_set_drvdata(pdev, NULL);
kfree(data);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment