edac: convert sysdev_class to a regular subsystem
authorKay Sievers <kay.sievers@vrfy.org>
Wed, 14 Dec 2011 23:21:07 +0000 (15:21 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 14 Dec 2011 23:21:07 +0000 (15:21 -0800)
After all sysdev classes are ported to regular driver core entities, the
sysdev implementation will be entirely removed from the kernel.

Cc: Doug Thompson <dougthompson@xmission.com>
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Lucas De Marchi <lucas.demarchi@profusion.mobi>
Cc: Borislav Petkov <borislav.petkov@amd.com>
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/edac/edac_core.h
drivers/edac/edac_device.c
drivers/edac/edac_device_sysfs.c
drivers/edac/edac_mc.c
drivers/edac/edac_mc_sysfs.c
drivers/edac/edac_module.h
drivers/edac/edac_pci.c
drivers/edac/edac_pci_sysfs.c
drivers/edac/edac_stub.c
drivers/edac/mce_amd_inj.c
include/linux/edac.h

index fe90cd4a7ebc4192da2d1e354cc72e3436d6ab45..e48ab3108ad80c080cfe8714f8263605bba1221a 100644 (file)
@@ -32,7 +32,6 @@
 #include <linux/completion.h>
 #include <linux/kobject.h>
 #include <linux/platform_device.h>
-#include <linux/sysdev.h>
 #include <linux/workqueue.h>
 #include <linux/edac.h>
 
@@ -243,8 +242,8 @@ struct edac_device_ctl_info {
         */
        struct edac_dev_sysfs_attribute *sysfs_attributes;
 
-       /* pointer to main 'edac' class in sysfs */
-       struct sysdev_class *edac_class;
+       /* pointer to main 'edac' subsys in sysfs */
+       struct bus_type *edac_subsys;
 
        /* the internal state of this controller instance */
        int op_state;
@@ -342,7 +341,7 @@ struct edac_pci_ctl_info {
 
        int pci_idx;
 
-       struct sysdev_class *edac_class;        /* pointer to class */
+       struct bus_type *edac_subsys;   /* pointer to subsystem */
 
        /* the internal state of this controller instance */
        int op_state;
index c3f67437afb666f489cf926d573501fb79c344a4..4b154593343a16977b00ca2e73f0577b9cd74462 100644 (file)
@@ -23,7 +23,6 @@
 #include <linux/jiffies.h>
 #include <linux/spinlock.h>
 #include <linux/list.h>
-#include <linux/sysdev.h>
 #include <linux/ctype.h>
 #include <linux/workqueue.h>
 #include <asm/uaccess.h>
index 86649df002850f0c56620063f550fa47d1ac5dd1..b4ea185ccebf6f42e32536cae7f6c54e9064ecf5 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * file for managing the edac_device class of devices for EDAC
+ * file for managing the edac_device subsystem of devices for EDAC
  *
  * (C) 2007 SoftwareBitMaker 
  *
@@ -230,21 +230,21 @@ static struct kobj_type ktype_device_ctrl = {
  */
 int edac_device_register_sysfs_main_kobj(struct edac_device_ctl_info *edac_dev)
 {
-       struct sysdev_class *edac_class;
+       struct bus_type *edac_subsys;
        int err;
 
        debugf1("%s()\n", __func__);
 
        /* get the /sys/devices/system/edac reference */
-       edac_class = edac_get_sysfs_class();
-       if (edac_class == NULL) {
-               debugf1("%s() no edac_class error\n", __func__);
+       edac_subsys = edac_get_sysfs_subsys();
+       if (edac_subsys == NULL) {
+               debugf1("%s() no edac_subsys error\n", __func__);
                err = -ENODEV;
                goto err_out;
        }
 
-       /* Point to the 'edac_class' this instance 'reports' to */
-       edac_dev->edac_class = edac_class;
+       /* Point to the 'edac_subsys' this instance 'reports' to */
+       edac_dev->edac_subsys = edac_subsys;
 
        /* Init the devices's kobject */
        memset(&edac_dev->kobj, 0, sizeof(struct kobject));
@@ -261,7 +261,7 @@ int edac_device_register_sysfs_main_kobj(struct edac_device_ctl_info *edac_dev)
 
        /* register */
        err = kobject_init_and_add(&edac_dev->kobj, &ktype_device_ctrl,
-                                  &edac_class->kset.kobj,
+                                  &edac_subsys->dev_root->kobj,
                                   "%s", edac_dev->name);
        if (err) {
                debugf1("%s()Failed to register '.../edac/%s'\n",
@@ -284,7 +284,7 @@ err_kobj_reg:
        module_put(edac_dev->owner);
 
 err_mod_get:
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 
 err_out:
        return err;
@@ -308,7 +308,7 @@ void edac_device_unregister_sysfs_main_kobj(struct edac_device_ctl_info *dev)
         *   b) 'kfree' the memory
         */
        kobject_put(&dev->kobj);
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 }
 
 /* edac_dev -> instance information */
index d69144a090435cbfdd5d883e231475e649c666d8..ca6c04d350ee4a613be2df96e0c0100f0518fbfb 100644 (file)
@@ -25,7 +25,6 @@
 #include <linux/jiffies.h>
 #include <linux/spinlock.h>
 #include <linux/list.h>
-#include <linux/sysdev.h>
 #include <linux/ctype.h>
 #include <linux/edac.h>
 #include <asm/uaccess.h>
index 29ffa350bfbe00281376c61c5c5c9c8434098a37..d56e63477d5cd286a99eaa06e20d8bbca36ddb2c 100644 (file)
@@ -1021,19 +1021,19 @@ void edac_remove_sysfs_mci_device(struct mem_ctl_info *mci)
 int edac_sysfs_setup_mc_kset(void)
 {
        int err = -EINVAL;
-       struct sysdev_class *edac_class;
+       struct bus_type *edac_subsys;
 
        debugf1("%s()\n", __func__);
 
-       /* get the /sys/devices/system/edac class reference */
-       edac_class = edac_get_sysfs_class();
-       if (edac_class == NULL) {
-               debugf1("%s() no edac_class error=%d\n", __func__, err);
+       /* get the /sys/devices/system/edac subsys reference */
+       edac_subsys = edac_get_sysfs_subsys();
+       if (edac_subsys == NULL) {
+               debugf1("%s() no edac_subsys error=%d\n", __func__, err);
                goto fail_out;
        }
 
        /* Init the MC's kobject */
-       mc_kset = kset_create_and_add("mc", NULL, &edac_class->kset.kobj);
+       mc_kset = kset_create_and_add("mc", NULL, &edac_subsys->dev_root->kobj);
        if (!mc_kset) {
                err = -ENOMEM;
                debugf1("%s() Failed to register '.../edac/mc'\n", __func__);
@@ -1045,7 +1045,7 @@ int edac_sysfs_setup_mc_kset(void)
        return 0;
 
 fail_kset:
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 
 fail_out:
        return err;
@@ -1059,6 +1059,6 @@ fail_out:
 void edac_sysfs_teardown_mc_kset(void)
 {
        kset_unregister(mc_kset);
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 }
 
index 17aabb7b90ecff1ac09a7344ea9fe17a86fb9c8a..00f81b47a51ffd886e58c4aeaa70d8c0b896246f 100644 (file)
@@ -10,8 +10,6 @@
 #ifndef        __EDAC_MODULE_H__
 #define        __EDAC_MODULE_H__
 
-#include <linux/sysdev.h>
-
 #include "edac_core.h"
 
 /*
index 2b378207d571b6bbd2220ac4b7deb263537b34b4..63af1c5673d1bbc790d5a98228dd050fc9de36b5 100644 (file)
@@ -19,7 +19,6 @@
 #include <linux/slab.h>
 #include <linux/spinlock.h>
 #include <linux/list.h>
-#include <linux/sysdev.h>
 #include <linux/ctype.h>
 #include <linux/workqueue.h>
 #include <asm/uaccess.h>
index 495198ad059c2c4b9c452d0ecce3c95ecc45c510..97f5064e39924deb504fc54922fec0941aa043ca 100644 (file)
@@ -338,12 +338,12 @@ static struct kobj_type ktype_edac_pci_main_kobj = {
  * edac_pci_main_kobj_setup()
  *
  *     setup the sysfs for EDAC PCI attributes
- *     assumes edac_class has already been initialized
+ *     assumes edac_subsys has already been initialized
  */
 static int edac_pci_main_kobj_setup(void)
 {
        int err;
-       struct sysdev_class *edac_class;
+       struct bus_type *edac_subsys;
 
        debugf0("%s()\n", __func__);
 
@@ -354,9 +354,9 @@ static int edac_pci_main_kobj_setup(void)
        /* First time, so create the main kobject and its
         * controls and attributes
         */
-       edac_class = edac_get_sysfs_class();
-       if (edac_class == NULL) {
-               debugf1("%s() no edac_class\n", __func__);
+       edac_subsys = edac_get_sysfs_subsys();
+       if (edac_subsys == NULL) {
+               debugf1("%s() no edac_subsys\n", __func__);
                err = -ENODEV;
                goto decrement_count_fail;
        }
@@ -381,7 +381,7 @@ static int edac_pci_main_kobj_setup(void)
        /* Instanstiate the pci object */
        err = kobject_init_and_add(edac_pci_top_main_kobj,
                                   &ktype_edac_pci_main_kobj,
-                                  &edac_class->kset.kobj, "pci");
+                                  &edac_subsys->dev_root->kobj, "pci");
        if (err) {
                debugf1("Failed to register '.../edac/pci'\n");
                goto kobject_init_and_add_fail;
@@ -404,7 +404,7 @@ kzalloc_fail:
        module_put(THIS_MODULE);
 
 mod_get_fail:
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 
 decrement_count_fail:
        /* if are on this error exit, nothing to tear down */
@@ -432,7 +432,7 @@ static void edac_pci_main_kobj_teardown(void)
                        __func__);
                kobject_put(edac_pci_top_main_kobj);
        }
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 }
 
 /*
index 86ad2eee1201d78d95bc84369da772d2cf07f5a3..670c4481453bc2ba7dc690bb505097cfdfa1dca4 100644 (file)
@@ -26,7 +26,7 @@ EXPORT_SYMBOL_GPL(edac_handlers);
 int edac_err_assert = 0;
 EXPORT_SYMBOL_GPL(edac_err_assert);
 
-static atomic_t edac_class_valid = ATOMIC_INIT(0);
+static atomic_t edac_subsys_valid = ATOMIC_INIT(0);
 
 /*
  * called to determine if there is an EDAC driver interested in
@@ -54,36 +54,37 @@ EXPORT_SYMBOL_GPL(edac_atomic_assert_error);
  * sysfs object: /sys/devices/system/edac
  *     need to export to other files
  */
-struct sysdev_class edac_class = {
+struct bus_type edac_subsys = {
        .name = "edac",
+       .dev_name = "edac",
 };
-EXPORT_SYMBOL_GPL(edac_class);
+EXPORT_SYMBOL_GPL(edac_subsys);
 
 /* return pointer to the 'edac' node in sysfs */
-struct sysdev_class *edac_get_sysfs_class(void)
+struct bus_type *edac_get_sysfs_subsys(void)
 {
        int err = 0;
 
-       if (atomic_read(&edac_class_valid))
+       if (atomic_read(&edac_subsys_valid))
                goto out;
 
        /* create the /sys/devices/system/edac directory */
-       err = sysdev_class_register(&edac_class);
+       err = subsys_system_register(&edac_subsys, NULL);
        if (err) {
                printk(KERN_ERR "Error registering toplevel EDAC sysfs dir\n");
                return NULL;
        }
 
 out:
-       atomic_inc(&edac_class_valid);
-       return &edac_class;
+       atomic_inc(&edac_subsys_valid);
+       return &edac_subsys;
 }
-EXPORT_SYMBOL_GPL(edac_get_sysfs_class);
+EXPORT_SYMBOL_GPL(edac_get_sysfs_subsys);
 
-void edac_put_sysfs_class(void)
+void edac_put_sysfs_subsys(void)
 {
        /* last user unregisters it */
-       if (atomic_dec_and_test(&edac_class_valid))
-               sysdev_class_unregister(&edac_class);
+       if (atomic_dec_and_test(&edac_subsys_valid))
+               bus_unregister(&edac_subsys);
 }
-EXPORT_SYMBOL_GPL(edac_put_sysfs_class);
+EXPORT_SYMBOL_GPL(edac_put_sysfs_subsys);
index 73c3e26a0bceb61803feced841fdcc6b7029500c..885e8ad8fdcf94679c3631db3c319d0fff7ac618 100644 (file)
@@ -11,7 +11,6 @@
  */
 
 #include <linux/kobject.h>
-#include <linux/sysdev.h>
 #include <linux/edac.h>
 #include <linux/module.h>
 #include <asm/mce.h>
@@ -116,14 +115,14 @@ static struct edac_mce_attr *sysfs_attrs[] = { &mce_attr_status, &mce_attr_misc,
 
 static int __init edac_init_mce_inject(void)
 {
-       struct sysdev_class *edac_class = NULL;
+       struct bus_type *edac_subsys = NULL;
        int i, err = 0;
 
-       edac_class = edac_get_sysfs_class();
-       if (!edac_class)
+       edac_subsys = edac_get_sysfs_subsys();
+       if (!edac_subsys)
                return -EINVAL;
 
-       mce_kobj = kobject_create_and_add("mce", &edac_class->kset.kobj);
+       mce_kobj = kobject_create_and_add("mce", &edac_subsys->dev_root->kobj);
        if (!mce_kobj) {
                printk(KERN_ERR "Error creating a mce kset.\n");
                err = -ENOMEM;
@@ -147,7 +146,7 @@ err_sysfs_create:
        kobject_del(mce_kobj);
 
 err_mce_kobj:
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 
        return err;
 }
@@ -161,7 +160,7 @@ static void __exit edac_exit_mce_inject(void)
 
        kobject_del(mce_kobj);
 
-       edac_put_sysfs_class();
+       edac_put_sysfs_subsys();
 }
 
 module_init(edac_init_mce_inject);
index 055b248bdd5328dfbac8e59836563771eb8fcd58..1cd3947987e571489b2e0a6a2b5df88480d72e50 100644 (file)
@@ -13,7 +13,7 @@
 #define _LINUX_EDAC_H_
 
 #include <linux/atomic.h>
-#include <linux/sysdev.h>
+#include <linux/device.h>
 
 #define EDAC_OPSTATE_INVAL     -1
 #define EDAC_OPSTATE_POLL      0
 extern int edac_op_state;
 extern int edac_err_assert;
 extern atomic_t edac_handlers;
-extern struct sysdev_class edac_class;
+extern struct bus_type edac_subsys;
 
 extern int edac_handler_set(void);
 extern void edac_atomic_assert_error(void);
-extern struct sysdev_class *edac_get_sysfs_class(void);
-extern void edac_put_sysfs_class(void);
+extern struct bus_type *edac_get_sysfs_subsys(void);
+extern void edac_put_sysfs_subsys(void);
 
 static inline void opstate_init(void)
 {