diff options
author | Kay Sievers <kay.sievers@vrfy.org> | 2007-12-19 20:09:39 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-01-24 23:40:40 -0500 |
commit | af5ca3f4ec5cc4432a42a73b050dd8898ce8fd00 (patch) | |
tree | 3e5a3081b2802547f10da72c0026b4929d0e287b /drivers | |
parent | 528a4bf1d5ffed310d26fc1d82d45c02949f71cf (diff) |
Driver core: change sysdev classes to use dynamic kobject names
All kobjects require a dynamically allocated name now. We no longer
need to keep track if the name is statically assigned, we can just
unconditionally free() all kobject names on cleanup.
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/pci_link.c | 2 | ||||
-rw-r--r-- | drivers/base/class.c | 2 | ||||
-rw-r--r-- | drivers/base/cpu.c | 2 | ||||
-rw-r--r-- | drivers/base/memory.c | 2 | ||||
-rw-r--r-- | drivers/base/node.c | 2 | ||||
-rw-r--r-- | drivers/base/sys.c | 1 | ||||
-rw-r--r-- | drivers/edac/edac_module.c | 2 | ||||
-rw-r--r-- | drivers/kvm/kvm_main.c | 2 | ||||
-rw-r--r-- | drivers/macintosh/via-pmu.c | 2 | ||||
-rw-r--r-- | drivers/scsi/libsas/sas_scsi_host.c | 2 |
10 files changed, 10 insertions, 9 deletions
diff --git a/drivers/acpi/pci_link.c b/drivers/acpi/pci_link.c index c9f526e55392..5400ea173f6f 100644 --- a/drivers/acpi/pci_link.c +++ b/drivers/acpi/pci_link.c | |||
@@ -911,7 +911,7 @@ __setup("acpi_irq_balance", acpi_irq_balance_set); | |||
911 | 911 | ||
912 | /* FIXME: we will remove this interface after all drivers call pci_disable_device */ | 912 | /* FIXME: we will remove this interface after all drivers call pci_disable_device */ |
913 | static struct sysdev_class irqrouter_sysdev_class = { | 913 | static struct sysdev_class irqrouter_sysdev_class = { |
914 | set_kset_name("irqrouter"), | 914 | .name = "irqrouter", |
915 | .resume = irqrouter_resume, | 915 | .resume = irqrouter_resume, |
916 | }; | 916 | }; |
917 | 917 | ||
diff --git a/drivers/base/class.c b/drivers/base/class.c index 61fd26cc9f0e..b962a76875d2 100644 --- a/drivers/base/class.c +++ b/drivers/base/class.c | |||
@@ -466,7 +466,6 @@ static struct kset_uevent_ops class_uevent_ops = { | |||
466 | * entirely soon. | 466 | * entirely soon. |
467 | */ | 467 | */ |
468 | static struct kset class_obj_subsys = { | 468 | static struct kset class_obj_subsys = { |
469 | .kobj = { .k_name = "class_obj", }, | ||
470 | .uevent_ops = &class_uevent_ops, | 469 | .uevent_ops = &class_uevent_ops, |
471 | }; | 470 | }; |
472 | 471 | ||
@@ -872,6 +871,7 @@ int __init classes_init(void) | |||
872 | /* ick, this is ugly, the things we go through to keep from showing up | 871 | /* ick, this is ugly, the things we go through to keep from showing up |
873 | * in sysfs... */ | 872 | * in sysfs... */ |
874 | kset_init(&class_obj_subsys); | 873 | kset_init(&class_obj_subsys); |
874 | kobject_set_name(&class_obj_subsys.kobj, "class_obj"); | ||
875 | if (!class_obj_subsys.kobj.parent) | 875 | if (!class_obj_subsys.kobj.parent) |
876 | class_obj_subsys.kobj.parent = &class_obj_subsys.kobj; | 876 | class_obj_subsys.kobj.parent = &class_obj_subsys.kobj; |
877 | return 0; | 877 | return 0; |
diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c index 40545071e3c9..c5885f5ce0ac 100644 --- a/drivers/base/cpu.c +++ b/drivers/base/cpu.c | |||
@@ -14,7 +14,7 @@ | |||
14 | #include "base.h" | 14 | #include "base.h" |
15 | 15 | ||
16 | struct sysdev_class cpu_sysdev_class = { | 16 | struct sysdev_class cpu_sysdev_class = { |
17 | set_kset_name("cpu"), | 17 | .name = "cpu", |
18 | }; | 18 | }; |
19 | EXPORT_SYMBOL(cpu_sysdev_class); | 19 | EXPORT_SYMBOL(cpu_sysdev_class); |
20 | 20 | ||
diff --git a/drivers/base/memory.c b/drivers/base/memory.c index 7868707c7eda..7ae413fdd5fc 100644 --- a/drivers/base/memory.c +++ b/drivers/base/memory.c | |||
@@ -26,7 +26,7 @@ | |||
26 | #define MEMORY_CLASS_NAME "memory" | 26 | #define MEMORY_CLASS_NAME "memory" |
27 | 27 | ||
28 | static struct sysdev_class memory_sysdev_class = { | 28 | static struct sysdev_class memory_sysdev_class = { |
29 | set_kset_name(MEMORY_CLASS_NAME), | 29 | .name = MEMORY_CLASS_NAME, |
30 | }; | 30 | }; |
31 | 31 | ||
32 | static const char *memory_uevent_name(struct kset *kset, struct kobject *kobj) | 32 | static const char *memory_uevent_name(struct kset *kset, struct kobject *kobj) |
diff --git a/drivers/base/node.c b/drivers/base/node.c index 88eeed72b5d6..e59861f18ce5 100644 --- a/drivers/base/node.c +++ b/drivers/base/node.c | |||
@@ -15,7 +15,7 @@ | |||
15 | #include <linux/device.h> | 15 | #include <linux/device.h> |
16 | 16 | ||
17 | static struct sysdev_class node_class = { | 17 | static struct sysdev_class node_class = { |
18 | set_kset_name("node"), | 18 | .name = "node", |
19 | }; | 19 | }; |
20 | 20 | ||
21 | 21 | ||
diff --git a/drivers/base/sys.c b/drivers/base/sys.c index e666441dd76b..2f79c55acdcc 100644 --- a/drivers/base/sys.c +++ b/drivers/base/sys.c | |||
@@ -136,6 +136,7 @@ int sysdev_class_register(struct sysdev_class * cls) | |||
136 | cls->kset.kobj.parent = &system_kset->kobj; | 136 | cls->kset.kobj.parent = &system_kset->kobj; |
137 | cls->kset.kobj.ktype = &ktype_sysdev_class; | 137 | cls->kset.kobj.ktype = &ktype_sysdev_class; |
138 | cls->kset.kobj.kset = system_kset; | 138 | cls->kset.kobj.kset = system_kset; |
139 | kobject_set_name(&cls->kset.kobj, cls->name); | ||
139 | return kset_register(&cls->kset); | 140 | return kset_register(&cls->kset); |
140 | } | 141 | } |
141 | 142 | ||
diff --git a/drivers/edac/edac_module.c b/drivers/edac/edac_module.c index e0c4a4086055..7e1374afd967 100644 --- a/drivers/edac/edac_module.c +++ b/drivers/edac/edac_module.c | |||
@@ -31,7 +31,7 @@ struct workqueue_struct *edac_workqueue; | |||
31 | * need to export to other files in this modules | 31 | * need to export to other files in this modules |
32 | */ | 32 | */ |
33 | static struct sysdev_class edac_class = { | 33 | static struct sysdev_class edac_class = { |
34 | set_kset_name("edac"), | 34 | .name = "edac", |
35 | }; | 35 | }; |
36 | static int edac_class_valid; | 36 | static int edac_class_valid; |
37 | 37 | ||
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c index 47c10b8f89b3..c0f372f1d761 100644 --- a/drivers/kvm/kvm_main.c +++ b/drivers/kvm/kvm_main.c | |||
@@ -3451,7 +3451,7 @@ static int kvm_resume(struct sys_device *dev) | |||
3451 | } | 3451 | } |
3452 | 3452 | ||
3453 | static struct sysdev_class kvm_sysdev_class = { | 3453 | static struct sysdev_class kvm_sysdev_class = { |
3454 | set_kset_name("kvm"), | 3454 | .name = "kvm", |
3455 | .suspend = kvm_suspend, | 3455 | .suspend = kvm_suspend, |
3456 | .resume = kvm_resume, | 3456 | .resume = kvm_resume, |
3457 | }; | 3457 | }; |
diff --git a/drivers/macintosh/via-pmu.c b/drivers/macintosh/via-pmu.c index 6123c70153d3..ac420b17e16f 100644 --- a/drivers/macintosh/via-pmu.c +++ b/drivers/macintosh/via-pmu.c | |||
@@ -2796,7 +2796,7 @@ static int pmu_sys_resume(struct sys_device *sysdev) | |||
2796 | #endif /* CONFIG_PM_SLEEP && CONFIG_PPC32 */ | 2796 | #endif /* CONFIG_PM_SLEEP && CONFIG_PPC32 */ |
2797 | 2797 | ||
2798 | static struct sysdev_class pmu_sysclass = { | 2798 | static struct sysdev_class pmu_sysclass = { |
2799 | set_kset_name("pmu"), | 2799 | .name = "pmu", |
2800 | }; | 2800 | }; |
2801 | 2801 | ||
2802 | static struct sys_device device_pmu = { | 2802 | static struct sys_device device_pmu = { |
diff --git a/drivers/scsi/libsas/sas_scsi_host.c b/drivers/scsi/libsas/sas_scsi_host.c index 7663841eb4cf..a3fdc57e2673 100644 --- a/drivers/scsi/libsas/sas_scsi_host.c +++ b/drivers/scsi/libsas/sas_scsi_host.c | |||
@@ -464,7 +464,7 @@ int sas_eh_bus_reset_handler(struct scsi_cmnd *cmd) | |||
464 | res = sas_phy_reset(phy, 1); | 464 | res = sas_phy_reset(phy, 1); |
465 | if (res) | 465 | if (res) |
466 | SAS_DPRINTK("Bus reset of %s failed 0x%x\n", | 466 | SAS_DPRINTK("Bus reset of %s failed 0x%x\n", |
467 | phy->dev.kobj.k_name, | 467 | kobject_name(&phy->dev.kobj), |
468 | res); | 468 | res); |
469 | if (res == TMF_RESP_FUNC_SUCC || res == TMF_RESP_FUNC_COMPLETE) | 469 | if (res == TMF_RESP_FUNC_SUCC || res == TMF_RESP_FUNC_COMPLETE) |
470 | return SUCCESS; | 470 | return SUCCESS; |