Commit bc87d2fe authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

kobject: convert efivars to use kobject_create

Using a kset for this simple directory is an overkill.

Cc: Kay Sievers <kay.sievers@vrfy.org>
Cc: Matt Domsch <Matt_Domsch@dell.com>
Cc: Matt Tolentino <matthew.e.tolentino@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 6e90aa97
...@@ -573,7 +573,7 @@ static struct attribute_group efi_subsys_attr_group = { ...@@ -573,7 +573,7 @@ static struct attribute_group efi_subsys_attr_group = {
static struct kset *vars_kset; static struct kset *vars_kset;
static struct kset *efi_kset; static struct kobject *efi_kobj;
/* /*
* efivar_create_sysfs_entry() * efivar_create_sysfs_entry()
...@@ -665,17 +665,15 @@ efivars_init(void) ...@@ -665,17 +665,15 @@ efivars_init(void)
printk(KERN_INFO "EFI Variables Facility v%s %s\n", EFIVARS_VERSION, printk(KERN_INFO "EFI Variables Facility v%s %s\n", EFIVARS_VERSION,
EFIVARS_DATE); EFIVARS_DATE);
/* /* For now we'll register the efi directory at /sys/firmware/efi */
* For now we'll register the efi subsys within this driver efi_kobj = kobject_create_and_add("efi", firmware_kobj);
*/ if (!efi_kobj) {
efi_kset = kset_create_and_add("efi", NULL, firmware_kobj);
if (!efi_kset) {
printk(KERN_ERR "efivars: Firmware registration failed.\n"); printk(KERN_ERR "efivars: Firmware registration failed.\n");
error = -ENOMEM; error = -ENOMEM;
goto out_free; goto out_free;
} }
vars_kset = kset_create_and_add("vars", NULL, &efi_kset->kobj); vars_kset = kset_create_and_add("vars", NULL, efi_kobj);
if (!vars_kset) { if (!vars_kset) {
printk(KERN_ERR "efivars: Subsystem registration failed.\n"); printk(KERN_ERR "efivars: Subsystem registration failed.\n");
error = -ENOMEM; error = -ENOMEM;
...@@ -725,7 +723,7 @@ efivars_init(void) ...@@ -725,7 +723,7 @@ efivars_init(void)
" due to error %d\n", error); " due to error %d\n", error);
/* Don't forget the systab entry */ /* Don't forget the systab entry */
error = sysfs_create_group(&efi_kset->kobj, &efi_subsys_attr_group); error = sysfs_create_group(efi_kobj, &efi_subsys_attr_group);
if (error) if (error)
printk(KERN_ERR "efivars: Sysfs attribute export failed with error %d.\n", error); printk(KERN_ERR "efivars: Sysfs attribute export failed with error %d.\n", error);
else else
...@@ -734,7 +732,7 @@ efivars_init(void) ...@@ -734,7 +732,7 @@ efivars_init(void)
kset_unregister(vars_kset); kset_unregister(vars_kset);
out_firmware_unregister: out_firmware_unregister:
kset_unregister(efi_kset); kobject_unregister(efi_kobj);
out_free: out_free:
kfree(variable_name); kfree(variable_name);
...@@ -755,7 +753,7 @@ efivars_exit(void) ...@@ -755,7 +753,7 @@ efivars_exit(void)
} }
kset_unregister(vars_kset); kset_unregister(vars_kset);
kset_unregister(efi_kset); kobject_unregister(efi_kobj);
} }
module_init(efivars_init); module_init(efivars_init);
......
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