driver core: class: make class_register() take a const *
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 2 Apr 2023 17:58:47 +0000 (19:58 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 3 Apr 2023 19:42:46 +0000 (21:42 +0200)
Now that the class code is cleaned up to not modify the class pointer
registered with it, change class_register() to take a const * to allow
the structure to be placed into read-only memory.

Acked-by: Rafael J. Wysocki <rafael@kernel.org>
Link: https://lore.kernel.org/r/2023040248-customary-release-4aec@gregkh
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/base/base.h
drivers/base/class.c
include/linux/device/class.h

index e96f334..eb4c0ac 100644 (file)
@@ -54,7 +54,7 @@ struct subsys_private {
        struct device *dev_root;
 
        struct kset glue_dirs;
-       struct class *class;
+       const struct class *class;
 
        struct lock_class_key lock_key;
 };
index 53fc705..05bce79 100644 (file)
@@ -93,7 +93,7 @@ static ssize_t class_attr_store(struct kobject *kobj, struct attribute *attr,
 static void class_release(struct kobject *kobj)
 {
        struct subsys_private *cp = to_subsys_private(kobj);
-       struct class *class = cp->class;
+       const struct class *class = cp->class;
 
        pr_debug("class '%s': release.\n", class->name);
 
@@ -110,7 +110,7 @@ static void class_release(struct kobject *kobj)
 static const struct kobj_ns_type_operations *class_child_ns_type(const struct kobject *kobj)
 {
        const struct subsys_private *cp = to_subsys_private(kobj);
-       struct class *class = cp->class;
+       const struct class *class = cp->class;
 
        return class->ns_type;
 }
@@ -175,7 +175,7 @@ static void klist_class_dev_put(struct klist_node *n)
        put_device(dev);
 }
 
-int class_register(struct class *cls)
+int class_register(const struct class *cls)
 {
        struct subsys_private *cp;
        struct lock_class_key *key;
index f3c418f..4bf46f9 100644 (file)
@@ -76,7 +76,7 @@ struct class_dev_iter {
        const struct device_type        *type;
 };
 
-int __must_check class_register(struct class *class);
+int __must_check class_register(const struct class *class);
 void class_unregister(const struct class *class);
 bool class_is_registered(const struct class *class);