Kobject: rename kobject_init_ng() to kobject_init()
[pandora-kernel.git] / drivers / base / core.c
index 22fdf32..675a719 100644 (file)
@@ -18,7 +18,7 @@
 #include <linux/string.h>
 #include <linux/kdev_t.h>
 #include <linux/notifier.h>
-
+#include <linux/genhd.h>
 #include <asm/semaphore.h>
 
 #include "base.h"
@@ -525,7 +525,7 @@ static void klist_children_put(struct klist_node *n)
 void device_initialize(struct device *dev)
 {
        dev->kobj.kset = devices_kset;
-       kobject_init_ng(&dev->kobj, &device_ktype);
+       kobject_init(&dev->kobj, &device_ktype);
        klist_init(&dev->klist_children, klist_children_get,
                   klist_children_put);
        INIT_LIST_HEAD(&dev->dma_pools);
@@ -538,22 +538,20 @@ void device_initialize(struct device *dev)
 }
 
 #ifdef CONFIG_SYSFS_DEPRECATED
-static struct kobject * get_device_parent(struct device *dev,
-                                         struct device *parent)
+static struct kobject *get_device_parent(struct device *dev,
+                                        struct device *parent)
 {
-       /*
-        * Set the parent to the class, not the parent device
-        * for topmost devices in class hierarchy.
-        * This keeps sysfs from having a symlink to make old
-        * udevs happy
-        */
+       /* class devices without a parent live in /sys/class/<classname>/ */
        if (dev->class && (!parent || parent->class != dev->class))
                return &dev->class->subsys.kobj;
+       /* all other devices keep their parent */
        else if (parent)
                return &parent->kobj;
 
        return NULL;
 }
+
+static inline void cleanup_device_parent(struct device *dev) {}
 #else
 static struct kobject *virtual_device_parent(struct device *dev)
 {
@@ -566,8 +564,8 @@ static struct kobject *virtual_device_parent(struct device *dev)
        return virtual_dir;
 }
 
-static struct kobject * get_device_parent(struct device *dev,
-                                         struct device *parent)
+static struct kobject *get_device_parent(struct device *dev,
+                                        struct device *parent)
 {
        int retval;
 
@@ -604,7 +602,7 @@ static struct kobject * get_device_parent(struct device *dev,
                if (!k)
                        return NULL;
                k->kset = &dev->class->class_dirs;
-               retval = kobject_add_ng(k, parent_kobj, "%s", dev->class->name);
+               retval = kobject_add(k, parent_kobj, "%s", dev->class->name);
                if (retval < 0) {
                        kobject_put(k);
                        return NULL;
@@ -618,6 +616,34 @@ static struct kobject * get_device_parent(struct device *dev,
                return &parent->kobj;
        return NULL;
 }
+
+static void cleanup_device_parent(struct device *dev)
+{
+       struct device *d;
+       int other = 0;
+
+       if (!dev->class)
+               return;
+
+       /* see if we live in a parent class directory */
+       if (dev->kobj.parent->kset != &dev->class->class_dirs)
+               return;
+
+       /* if we are the last child of our class, delete the directory */
+       down(&dev->class->sem);
+       list_for_each_entry(d, &dev->class->devices, node) {
+               if (d == dev)
+                       continue;
+               if (d->kobj.parent == dev->kobj.parent) {
+                       other = 1;
+                       break;
+               }
+       }
+       if (!other)
+               kobject_del(dev->kobj.parent);
+       kobject_put(dev->kobj.parent);
+       up(&dev->class->sem);
+}
 #endif
 
 static int setup_parent(struct device *dev, struct device *parent)
@@ -637,65 +663,76 @@ static int device_add_class_symlinks(struct device *dev)
 
        if (!dev->class)
                return 0;
+
        error = sysfs_create_link(&dev->kobj, &dev->class->subsys.kobj,
                                  "subsystem");
        if (error)
                goto out;
-       /*
-        * If this is not a "fake" compatible device, then create the
-        * symlink from the class to the device.
-        */
-       if (dev->kobj.parent != &dev->class->subsys.kobj) {
+
+#ifdef CONFIG_SYSFS_DEPRECATED
+       /* stacked class devices need a symlink in the class directory */
+       if (dev->kobj.parent != &dev->class->subsys.kobj &&
+           dev->type != &part_type) {
                error = sysfs_create_link(&dev->class->subsys.kobj, &dev->kobj,
                                          dev->bus_id);
                if (error)
                        goto out_subsys;
        }
-       if (dev->parent) {
-#ifdef CONFIG_SYSFS_DEPRECATED
-               {
-                       struct device *parent = dev->parent;
-                       char *class_name;
-
-                       /*
-                        * In old sysfs stacked class devices had 'device'
-                        * link pointing to real device instead of parent
-                        */
-                       while (parent->class && !parent->bus && parent->parent)
-                               parent = parent->parent;
-
-                       error = sysfs_create_link(&dev->kobj,
-                                                 &parent->kobj,
-                                                 "device");
-                       if (error)
-                               goto out_busid;
 
-                       class_name = make_class_name(dev->class->name,
-                                                       &dev->kobj);
-                       if (class_name)
-                               error = sysfs_create_link(&dev->parent->kobj,
-                                                       &dev->kobj, class_name);
-                       kfree(class_name);
-                       if (error)
-                               goto out_device;
-               }
-#else
-               error = sysfs_create_link(&dev->kobj, &dev->parent->kobj,
+       if (dev->parent && dev->type != &part_type) {
+               struct device *parent = dev->parent;
+               char *class_name;
+
+               /*
+                * stacked class devices have the 'device' link
+                * pointing to the bus device instead of the parent
+                */
+               while (parent->class && !parent->bus && parent->parent)
+                       parent = parent->parent;
+
+               error = sysfs_create_link(&dev->kobj,
+                                         &parent->kobj,
                                          "device");
                if (error)
                        goto out_busid;
-#endif
+
+               class_name = make_class_name(dev->class->name,
+                                               &dev->kobj);
+               if (class_name)
+                       error = sysfs_create_link(&dev->parent->kobj,
+                                               &dev->kobj, class_name);
+               kfree(class_name);
+               if (error)
+                       goto out_device;
        }
        return 0;
 
-#ifdef CONFIG_SYSFS_DEPRECATED
 out_device:
-       if (dev->parent)
+       if (dev->parent && dev->type != &part_type)
                sysfs_remove_link(&dev->kobj, "device");
-#endif
 out_busid:
-       if (dev->kobj.parent != &dev->class->subsys.kobj)
+       if (dev->kobj.parent != &dev->class->subsys.kobj &&
+           dev->type != &part_type)
                sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
+#else
+       /* link in the class directory pointing to the device */
+       error = sysfs_create_link(&dev->class->subsys.kobj, &dev->kobj,
+                                 dev->bus_id);
+       if (error)
+               goto out_subsys;
+
+       if (dev->parent && dev->type != &part_type) {
+               error = sysfs_create_link(&dev->kobj, &dev->parent->kobj,
+                                         "device");
+               if (error)
+                       goto out_busid;
+       }
+       return 0;
+
+out_busid:
+       sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
+#endif
+
 out_subsys:
        sysfs_remove_link(&dev->kobj, "subsystem");
 out:
@@ -706,8 +743,9 @@ static void device_remove_class_symlinks(struct device *dev)
 {
        if (!dev->class)
                return;
-       if (dev->parent) {
+
 #ifdef CONFIG_SYSFS_DEPRECATED
+       if (dev->parent && dev->type != &part_type) {
                char *class_name;
 
                class_name = make_class_name(dev->class->name, &dev->kobj);
@@ -715,11 +753,19 @@ static void device_remove_class_symlinks(struct device *dev)
                        sysfs_remove_link(&dev->parent->kobj, class_name);
                        kfree(class_name);
                }
-#endif
                sysfs_remove_link(&dev->kobj, "device");
        }
-       if (dev->kobj.parent != &dev->class->subsys.kobj)
+
+       if (dev->kobj.parent != &dev->class->subsys.kobj &&
+           dev->type != &part_type)
                sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
+#else
+       if (dev->parent && dev->type != &part_type)
+               sysfs_remove_link(&dev->kobj, "device");
+
+       sysfs_remove_link(&dev->class->subsys.kobj, dev->bus_id);
+#endif
+
        sysfs_remove_link(&dev->kobj, "subsystem");
 }
 
@@ -730,7 +776,7 @@ static void device_remove_class_symlinks(struct device *dev)
  *     This is part 2 of device_register(), though may be called
  *     separately _iff_ device_initialize() has been called separately.
  *
- *     This adds it to the kobject hierarchy via kobject_add_ng(), adds it
+ *     This adds it to the kobject hierarchy via kobject_add(), adds it
  *     to the global and sibling lists for the device, then
  *     adds it to the other relevant subsystems of the driver model.
  */
@@ -761,7 +807,7 @@ int device_add(struct device *dev)
                goto Error;
 
        /* first, register with generic layer. */
-       error = kobject_add_ng(&dev->kobj, dev->kobj.parent, "%s", dev->bus_id);
+       error = kobject_add(&dev->kobj, dev->kobj.parent, "%s", dev->bus_id);
        if (error)
                goto Error;
 
@@ -830,26 +876,6 @@ int device_add(struct device *dev)
  SymlinkError:
        if (MAJOR(dev->devt))
                device_remove_file(dev, &devt_attr);
-
-       if (dev->class) {
-               sysfs_remove_link(&dev->kobj, "subsystem");
-               /* If this is not a "fake" compatible device, remove the
-                * symlink from the class to the device. */
-               if (dev->kobj.parent != &dev->class->subsys.kobj)
-                       sysfs_remove_link(&dev->class->subsys.kobj,
-                                         dev->bus_id);
-               if (parent) {
-#ifdef CONFIG_SYSFS_DEPRECATED
-                       char *class_name = make_class_name(dev->class->name,
-                                                          &dev->kobj);
-                       if (class_name)
-                               sysfs_remove_link(&dev->parent->kobj,
-                                                 class_name);
-                       kfree(class_name);
-#endif
-                       sysfs_remove_link(&dev->kobj, "device");
-               }
-       }
  ueventattrError:
        device_remove_file(dev, &uevent_attr);
  attrError:
@@ -902,6 +928,7 @@ struct device * get_device(struct device * dev)
  */
 void put_device(struct device * dev)
 {
+       /* might_sleep(); */
        if (dev)
                kobject_put(&dev->kobj);
 }
@@ -932,23 +959,7 @@ void device_del(struct device * dev)
        if (MAJOR(dev->devt))
                device_remove_file(dev, &devt_attr);
        if (dev->class) {
-               sysfs_remove_link(&dev->kobj, "subsystem");
-               /* If this is not a "fake" compatible device, remove the
-                * symlink from the class to the device. */
-               if (dev->kobj.parent != &dev->class->subsys.kobj)
-                       sysfs_remove_link(&dev->class->subsys.kobj,
-                                         dev->bus_id);
-               if (parent) {
-#ifdef CONFIG_SYSFS_DEPRECATED
-                       char *class_name = make_class_name(dev->class->name,
-                                                          &dev->kobj);
-                       if (class_name)
-                               sysfs_remove_link(&dev->parent->kobj,
-                                                 class_name);
-                       kfree(class_name);
-#endif
-                       sysfs_remove_link(&dev->kobj, "device");
-               }
+               device_remove_class_symlinks(dev);
 
                down(&dev->class->sem);
                /* notify any interfaces that the device is now gone */
@@ -958,31 +969,6 @@ void device_del(struct device * dev)
                /* remove the device from the class list */
                list_del_init(&dev->node);
                up(&dev->class->sem);
-
-               /* If we live in a parent class-directory, unreference it */
-               if (dev->kobj.parent->kset == &dev->class->class_dirs) {
-                       struct device *d;
-                       int other = 0;
-
-                       /*
-                        * if we are the last child of our class, delete
-                        * our class-directory at this parent
-                        */
-                       down(&dev->class->sem);
-                       list_for_each_entry(d, &dev->class->devices, node) {
-                               if (d == dev)
-                                       continue;
-                               if (d->kobj.parent == dev->kobj.parent) {
-                                       other = 1;
-                                       break;
-                               }
-                       }
-                       if (!other)
-                               kobject_del(dev->kobj.parent);
-
-                       kobject_put(dev->kobj.parent);
-                       up(&dev->class->sem);
-               }
        }
        device_remove_file(dev, &uevent_attr);
        device_remove_attrs(dev);
@@ -1004,9 +990,9 @@ void device_del(struct device * dev)
                blocking_notifier_call_chain(&dev->bus->p->bus_notifier,
                                             BUS_NOTIFY_DEL_DEVICE, dev);
        kobject_uevent(&dev->kobj, KOBJ_REMOVE);
+       cleanup_device_parent(dev);
        kobject_del(&dev->kobj);
-       if (parent)
-               put_device(parent);
+       put_device(parent);
 }
 
 /**