On Wed, 2009-07-22 at 21:07 +0200, Jean Delvare wrote:
> > > * Do we need an actually struct class for each fake class, or just a
> > > class name?
> >
> > We will need to create a kobject for the compat class directory, we
> > will not need a "struct class" for it and can just use a simple
> > pointer to the registered kobject. If we use a string, we would need
> > to find the registered kobject with every call to create a link there,
> > not necessarily bad, but an explicitely registered object might be
> > easier.
>
> Any progress on this? I have just committed the patches to
> sensors-detect and libsensors, and the kernel patch is ready to go, but
> without the compatibility links it doesn't make any sense to push it
> upstream
Something like this? Please change it as you need. I did only a very
quick test.
The only important part is that the kobject of the class directly is not
exposed, so nobody else can do weird things with it.
Thanks,
Kay
--- a/drivers/base/class.c
+++ b/drivers/base/class.c
@@ -488,6 +488,45 @@ void class_interface_unregister(struct c
class_put(parent);
}
+struct class_compat {
+ struct kobject *kobj;
+};
+
+struct class_compat *class_compat_register(const char *name)
+{
+ struct class_compat *cls;
+
+ cls = kmalloc(sizeof(struct class_compat), GFP_KERNEL);
+ if (!cls)
+ return NULL;
+ cls->kobj = kobject_create_and_add(name, &class_kset->kobj);
+ if (!cls->kobj) {
+ kfree(cls);
+ return NULL;
+ }
+ return cls;
+}
+EXPORT_SYMBOL_GPL(class_compat_register);
+
+void class_compat_unregister(struct class_compat *cls)
+{
+ kobject_put(cls->kobj);
+ kfree(cls);
+}
+EXPORT_SYMBOL_GPL(class_compat_unregister);
+
+int class_compat_create_link(struct class_compat *cls, struct device *dev)
+{
+ return sysfs_create_link(cls->kobj, &dev->kobj, dev_name(dev));
+}
+EXPORT_SYMBOL_GPL(class_compat_create_link);
+
+void class_compat_remove_link(struct class_compat *cls, struct device *dev)
+{
+ return sysfs_remove_link(cls->kobj, dev_name(dev));
+}
+EXPORT_SYMBOL_GPL(class_compat_remove_link);
+
int __init classes_init(void)
{
class_kset = kset_create_and_add("class", NULL, NULL);
--- a/include/linux/device.h
+++ b/include/linux/device.h
@@ -223,6 +223,12 @@ extern void class_unregister(struct clas
__class_register(class, &__key); \
})
+struct class_compat;
+struct class_compat *class_compat_register(const char *name);
+void class_compat_unregister(struct class_compat *cls);
+int class_compat_create_link(struct class_compat *cls, struct device *dev);
+void class_compat_remove_link(struct class_compat *cls, struct device *dev);
+
extern void class_dev_iter_init(struct class_dev_iter *iter,
struct class *class,
struct device *start,
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
