Commit e11a7d2c authored by Ivan Orlov's avatar Ivan Orlov Committed by Oded Gabbay

accel: make accel_class a static const structure

Now that the driver core allows for struct class to be in read-only
memory, move the accel_class structure to be declared at build time
placing it into read-only memory, instead of having to be dynamically
allocated at boot time.

Cc: dri-devel@lists.freedesktop.org
Suggested-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarIvan Orlov <ivan.orlov0322@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: default avatarTomer Tayar <ttayar@habana.ai>
Signed-off-by: default avatarOded Gabbay <ogabbay@kernel.org>
parent d33c3d05
...@@ -21,7 +21,6 @@ static DEFINE_SPINLOCK(accel_minor_lock); ...@@ -21,7 +21,6 @@ static DEFINE_SPINLOCK(accel_minor_lock);
static struct idr accel_minors_idr; static struct idr accel_minors_idr;
static struct dentry *accel_debugfs_root; static struct dentry *accel_debugfs_root;
static struct class *accel_class;
static struct device_type accel_sysfs_device_minor = { static struct device_type accel_sysfs_device_minor = {
.name = "accel_minor" .name = "accel_minor"
...@@ -32,23 +31,19 @@ static char *accel_devnode(const struct device *dev, umode_t *mode) ...@@ -32,23 +31,19 @@ static char *accel_devnode(const struct device *dev, umode_t *mode)
return kasprintf(GFP_KERNEL, "accel/%s", dev_name(dev)); return kasprintf(GFP_KERNEL, "accel/%s", dev_name(dev));
} }
static const struct class accel_class = {
.name = "accel",
.devnode = accel_devnode,
};
static int accel_sysfs_init(void) static int accel_sysfs_init(void)
{ {
accel_class = class_create("accel"); return class_register(&accel_class);
if (IS_ERR(accel_class))
return PTR_ERR(accel_class);
accel_class->devnode = accel_devnode;
return 0;
} }
static void accel_sysfs_destroy(void) static void accel_sysfs_destroy(void)
{ {
if (IS_ERR_OR_NULL(accel_class)) class_unregister(&accel_class);
return;
class_destroy(accel_class);
accel_class = NULL;
} }
static int accel_name_info(struct seq_file *m, void *data) static int accel_name_info(struct seq_file *m, void *data)
...@@ -117,7 +112,7 @@ void accel_debugfs_register(struct drm_device *dev) ...@@ -117,7 +112,7 @@ void accel_debugfs_register(struct drm_device *dev)
void accel_set_device_instance_params(struct device *kdev, int index) void accel_set_device_instance_params(struct device *kdev, int index)
{ {
kdev->devt = MKDEV(ACCEL_MAJOR, index); kdev->devt = MKDEV(ACCEL_MAJOR, index);
kdev->class = accel_class; kdev->class = &accel_class;
kdev->type = &accel_sysfs_device_minor; kdev->type = &accel_sysfs_device_minor;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment