Commit 083a93b0 authored by Luis R. Rodriguez's avatar Luis R. Rodriguez Committed by Greg Kroah-Hartman

test_firmware: use device attribute groups

This simplifies init and exit.
Signed-off-by: default avatarLuis R. Rodriguez <mcgrof@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 67fd553c
...@@ -126,10 +126,21 @@ static ssize_t trigger_async_request_store(struct device *dev, ...@@ -126,10 +126,21 @@ static ssize_t trigger_async_request_store(struct device *dev,
} }
static DEVICE_ATTR_WO(trigger_async_request); static DEVICE_ATTR_WO(trigger_async_request);
#define TEST_FW_DEV_ATTR(name) &dev_attr_##name.attr
static struct attribute *test_dev_attrs[] = {
TEST_FW_DEV_ATTR(trigger_request),
TEST_FW_DEV_ATTR(trigger_async_request),
NULL,
};
ATTRIBUTE_GROUPS(test_dev);
static struct miscdevice test_fw_misc_device = { static struct miscdevice test_fw_misc_device = {
.minor = MISC_DYNAMIC_MINOR, .minor = MISC_DYNAMIC_MINOR,
.name = "test_firmware", .name = "test_firmware",
.fops = &test_fw_fops, .fops = &test_fw_fops,
.groups = test_dev_groups,
}; };
static int __init test_firmware_init(void) static int __init test_firmware_init(void)
...@@ -141,30 +152,10 @@ static int __init test_firmware_init(void) ...@@ -141,30 +152,10 @@ static int __init test_firmware_init(void)
pr_err("could not register misc device: %d\n", rc); pr_err("could not register misc device: %d\n", rc);
return rc; return rc;
} }
rc = device_create_file(test_fw_misc_device.this_device,
&dev_attr_trigger_request);
if (rc) {
pr_err("could not create sysfs interface: %d\n", rc);
goto dereg;
}
rc = device_create_file(test_fw_misc_device.this_device,
&dev_attr_trigger_async_request);
if (rc) {
pr_err("could not create async sysfs interface: %d\n", rc);
goto remove_file;
}
pr_warn("interface ready\n"); pr_warn("interface ready\n");
return 0; return 0;
remove_file:
device_remove_file(test_fw_misc_device.this_device,
&dev_attr_trigger_async_request);
dereg:
misc_deregister(&test_fw_misc_device);
return rc;
} }
module_init(test_firmware_init); module_init(test_firmware_init);
...@@ -172,10 +163,6 @@ module_init(test_firmware_init); ...@@ -172,10 +163,6 @@ module_init(test_firmware_init);
static void __exit test_firmware_exit(void) static void __exit test_firmware_exit(void)
{ {
release_firmware(test_firmware); release_firmware(test_firmware);
device_remove_file(test_fw_misc_device.this_device,
&dev_attr_trigger_async_request);
device_remove_file(test_fw_misc_device.this_device,
&dev_attr_trigger_request);
misc_deregister(&test_fw_misc_device); misc_deregister(&test_fw_misc_device);
pr_warn("removed interface\n"); pr_warn("removed interface\n");
} }
......
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