dmanegine: idxd: open code the dsa_drv registration
authorDave Jiang <dave.jiang@intel.com>
Thu, 15 Jul 2021 18:44:13 +0000 (11:44 -0700)
committerVinod Koul <vkoul@kernel.org>
Wed, 21 Jul 2021 04:39:16 +0000 (10:09 +0530)
Don't need a wrapper to register the driver. Just do it directly.

Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/162637465319.744545.16325178432532362906.stgit@djiang5-desk3.ch.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/idxd/idxd.h
drivers/dma/idxd/init.c
drivers/dma/idxd/sysfs.c

index 1c8abba134704e7328ba4d52405a44106ba67f4f..7fc26b7727c0c3d7956667e2ea8f46c6ae8d809f 100644 (file)
@@ -56,6 +56,8 @@ struct idxd_device_driver {
        struct device_driver drv;
 };
 
+extern struct idxd_device_driver dsa_drv;
+
 struct idxd_irq_entry {
        struct idxd_device *idxd;
        int id;
index c22225b14c5d70d684fda07cbcf21456d0fc3d25..5b628e6c04bfff0a001657fe7e5bad075c9662af 100644 (file)
@@ -840,9 +840,9 @@ static int __init idxd_init_module(void)
        if (err < 0)
                return err;
 
-       err = idxd_register_driver();
+       err = idxd_driver_register(&dsa_drv);
        if (err < 0)
-               goto err_idxd_driver_register;
+               goto err_dsa_driver_register;
 
        err = idxd_cdev_register();
        if (err)
@@ -857,8 +857,8 @@ static int __init idxd_init_module(void)
 err_pci_register:
        idxd_cdev_remove();
 err_cdev_register:
-       idxd_unregister_driver();
-err_idxd_driver_register:
+       idxd_driver_unregister(&dsa_drv);
+err_dsa_driver_register:
        idxd_unregister_bus_type();
        return err;
 }
@@ -866,7 +866,7 @@ module_init(idxd_init_module);
 
 static void __exit idxd_exit_module(void)
 {
-       idxd_unregister_driver();
+       idxd_driver_unregister(&dsa_drv);
        pci_unregister_driver(&idxd_pci_driver);
        idxd_cdev_remove();
        idxd_unregister_bus_type();
index abea8aca67996cda67d81cb9bb28d4964f5c3054..9f2d06c2aa98a094361a1fcb04d151593aa66b98 100644 (file)
@@ -79,23 +79,12 @@ static void idxd_dsa_drv_remove(struct idxd_dev *idxd_dev)
        }
 }
 
-static struct idxd_device_driver dsa_drv = {
+struct idxd_device_driver dsa_drv = {
        .name = "dsa",
        .probe = idxd_dsa_drv_probe,
        .remove = idxd_dsa_drv_remove,
 };
 
-/* IDXD generic driver setup */
-int idxd_register_driver(void)
-{
-       return idxd_driver_register(&dsa_drv);
-}
-
-void idxd_unregister_driver(void)
-{
-       idxd_driver_unregister(&dsa_drv);
-}
-
 /* IDXD engine attributes */
 static ssize_t engine_group_id_show(struct device *dev,
                                    struct device_attribute *attr, char *buf)