drm/pci: Deprecate drm_pci_init/exit completely

The magic switching between proper pci driver and shadow-attach isn't
useful anymore since there's no ums+kms drivers left. Let's split this
up properly, calling pci_register_driver for kms drivers and renaming
the shadow-attach init to drm_legacy_pci_init/exit.

Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170524145212.27837-6-daniel.vetter@ffwll.ch

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index e7a4bce6358d849ec8465d6925df358529a7a3e6..4911d304d8b96e10c6bb61762b13a5a27bd51be0 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -816,7 +816,7 @@ static int __init amdgpu_init(void)
        driver->num_ioctls = amdgpu_max_kms_ioctl;
        amdgpu_register_atpx_handler();
        /* let modprobe override vga console setting */
-       return drm_pci_init(driver, pdriver);
+       return pci_register_driver(pdriver);
 
 error_sched:
        amdgpu_fence_slab_fini();
@@ -831,7 +831,7 @@ error_sync:
 static void __exit amdgpu_exit(void)
 {
        amdgpu_amdkfd_fini();
-       drm_pci_exit(driver, pdriver);
+       pci_unregister_driver(pdriver);
        amdgpu_unregister_atpx_handler();
        amdgpu_sync_fini();
        amd_sched_fence_slab_fini();

diff --git a/drivers/gpu/drm/ast/ast_drv.c b/drivers/gpu/drm/ast/ast_drv.c
index f6794745a024bf9d9b6784ba1a0345a61b2edad9..3022b39c00f37b5ee7c079cb89d4beb2c6ff731d 100644 (file)
--- a/drivers/gpu/drm/ast/ast_drv.c
+++ b/drivers/gpu/drm/ast/ast_drv.c
@@ -220,11 +220,11 @@ static int __init ast_init(void)
 
        if (ast_modeset == 0)
                return -EINVAL;
-       return drm_pci_init(&driver, &ast_pci_driver);
+       return pci_register_driver(&ast_pci_driver);
 }
 static void __exit ast_exit(void)
 {
-       drm_pci_exit(&driver, &ast_pci_driver);
+       pci_unregister_driver(&ast_pci_driver);
 }
 
 module_init(ast_init);

diff --git a/drivers/gpu/drm/bochs/bochs_drv.c b/drivers/gpu/drm/bochs/bochs_drv.c
index 8fccd3cf000d6e12ff3aebb832aeba74badab571..a1d28845da5fefc1aa13554c582edd5a24ebbe75 100644 (file)
--- a/drivers/gpu/drm/bochs/bochs_drv.c
+++ b/drivers/gpu/drm/bochs/bochs_drv.c
@@ -223,12 +223,12 @@ static int __init bochs_init(void)
        if (bochs_modeset == 0)
                return -EINVAL;
 
-       return drm_pci_init(&bochs_driver, &bochs_pci_driver);
+       return pci_register_driver(&bochs_pci_driver);
 }
 
 static void __exit bochs_exit(void)
 {
-       drm_pci_exit(&bochs_driver, &bochs_pci_driver);
+       pci_unregister_driver(&bochs_pci_driver);
 }
 
 module_init(bochs_init);

diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.c b/drivers/gpu/drm/cirrus/cirrus_drv.c
index c48b9eb767121055d0980aec44bd8f7765ac0818..910c300f5c37d0d42816d0a8d22a5fc1b1bffbe0 100644 (file)
--- a/drivers/gpu/drm/cirrus/cirrus_drv.c
+++ b/drivers/gpu/drm/cirrus/cirrus_drv.c
@@ -165,12 +165,12 @@ static int __init cirrus_init(void)
 
        if (cirrus_modeset == 0)
                return -EINVAL;
-       return drm_pci_init(&driver, &cirrus_pci_driver);
+       return pci_register_driver(&cirrus_pci_driver);
 }
 
 static void __exit cirrus_exit(void)
 {
-       drm_pci_exit(&driver, &cirrus_pci_driver);
+       pci_unregister_driver(&cirrus_pci_driver);
 }
 
 module_init(cirrus_init);

diff --git a/drivers/gpu/drm/drm_pci.c b/drivers/gpu/drm/drm_pci.c
index ad31d95e77c9a25b061b67ec4c8c07f44a69ab82..1235c9877d6f1cebb50cf5c478c2f086d9da42a0 100644 (file)
--- a/drivers/gpu/drm/drm_pci.c
+++ b/drivers/gpu/drm/drm_pci.c
@@ -280,20 +280,15 @@ err_free:
 EXPORT_SYMBOL(drm_get_pci_dev);
 
 /**
- * drm_pci_init - Register matching PCI devices with the DRM subsystem
+ * drm_legacy_pci_init - shadow-attach a legacy DRM PCI driver
  * @driver: DRM device driver
  * @pdriver: PCI device driver
  *
- * Initializes a drm_device structures, registering the stubs and initializing
- * the AGP device.
- *
- * NOTE: This function is deprecated. Modern modesetting drm drivers should use
- * pci_register_driver() directly, this function only provides shadow-binding
- * support for old legacy drivers on top of that core pci function.
+ * This is only used by legacy dri1 drivers and deprecated.
  *
  * Return: 0 on success or a negative error code on failure.
  */
-int drm_pci_init(struct drm_driver *driver, struct pci_driver *pdriver)
+int drm_legacy_pci_init(struct drm_driver *driver, struct pci_driver *pdriver)
 {
        struct pci_dev *pdev = NULL;
        const struct pci_device_id *pid;
@@ -301,8 +296,8 @@ int drm_pci_init(struct drm_driver *driver, struct pci_driver *pdriver)
 
        DRM_DEBUG("\n");
 
-       if (!(driver->driver_features & DRIVER_LEGACY))
-               return pci_register_driver(pdriver);
+       if (WARN_ON(!(driver->driver_features & DRIVER_LEGACY)))
+               return -EINVAL;
 
        /* If not using KMS, fall back to stealth mode manual scanning. */
        INIT_LIST_HEAD(&driver->legacy_dev_list);
@@ -329,6 +324,7 @@ int drm_pci_init(struct drm_driver *driver, struct pci_driver *pdriver)
        }
        return 0;
 }
+EXPORT_SYMBOL(drm_legacy_pci_init);
 
 int drm_pcie_get_speed_cap_mask(struct drm_device *dev, u32 *mask)
 {
@@ -390,11 +386,6 @@ EXPORT_SYMBOL(drm_pcie_get_max_link_width);
 
 #else
 
-int drm_pci_init(struct drm_driver *driver, struct pci_driver *pdriver)
-{
-       return -1;
-}
-
 void drm_pci_agp_destroy(struct drm_device *dev) {}
 
 int drm_irq_by_busid(struct drm_device *dev, void *data,
@@ -404,27 +395,21 @@ int drm_irq_by_busid(struct drm_device *dev, void *data,
 }
 #endif
 
-EXPORT_SYMBOL(drm_pci_init);
-
 /**
- * drm_pci_exit - Unregister matching PCI devices from the DRM subsystem
+ * drm_legacy_pci_exit - unregister shadow-attach legacy DRM driver
  * @driver: DRM device driver
  * @pdriver: PCI device driver
  *
- * Unregisters one or more devices matched by a PCI driver from the DRM
- * subsystem.
- *
- * NOTE: This function is deprecated. Modern modesetting drm drivers should use
- * pci_unregister_driver() directly, this function only provides shadow-binding
- * support for old legacy drivers on top of that core pci function.
+ * Unregister a DRM driver shadow-attached through drm_legacy_pci_init(). This
+ * is deprecated and only used by dri1 drivers.
  */
-void drm_pci_exit(struct drm_driver *driver, struct pci_driver *pdriver)
+void drm_legacy_pci_exit(struct drm_driver *driver, struct pci_driver *pdriver)
 {
        struct drm_device *dev, *tmp;
        DRM_DEBUG("\n");
 
        if (!(driver->driver_features & DRIVER_LEGACY)) {
-               pci_unregister_driver(pdriver);
+               WARN_ON(1);
        } else {
                list_for_each_entry_safe(dev, tmp, &driver->legacy_dev_list,
                                         legacy_dev_list) {
@@ -434,4 +419,4 @@ void drm_pci_exit(struct drm_driver *driver, struct pci_driver *pdriver)
        }
        DRM_INFO("Module unloaded\n");
 }
-EXPORT_SYMBOL(drm_pci_exit);
+EXPORT_SYMBOL(drm_legacy_pci_exit);

diff --git a/drivers/gpu/drm/gma500/psb_drv.c b/drivers/gpu/drm/gma500/psb_drv.c
index 37d4c36c80f2e4037001f7f19d67bb4c1eb3392c..747c06b227c5e7ed82910ba4e8c6c3c697b4fceb 100644 (file)
--- a/drivers/gpu/drm/gma500/psb_drv.c
+++ b/drivers/gpu/drm/gma500/psb_drv.c
@@ -516,12 +516,12 @@ static struct pci_driver psb_pci_driver = {
 
 static int __init psb_init(void)
 {
-       return drm_pci_init(&driver, &psb_pci_driver);
+       return pci_register_driver(&psb_pci_driver);
 }
 
 static void __exit psb_exit(void)
 {
-       drm_pci_exit(&driver, &psb_pci_driver);
+       pci_unregister_driver(&psb_pci_driver);
 }
 
 late_initcall(psb_init);

diff --git a/drivers/gpu/drm/i810/i810_drv.c b/drivers/gpu/drm/i810/i810_drv.c
index e1c35c710e24b9a7cab39cae3dedc405c5d7e147..c69d5c487f513400fdea5bdeff2455c39b85ee76 100644 (file)
--- a/drivers/gpu/drm/i810/i810_drv.c
+++ b/drivers/gpu/drm/i810/i810_drv.c
@@ -82,12 +82,12 @@ static int __init i810_init(void)
                return -EINVAL;
        }
        driver.num_ioctls = i810_max_ioctl;
-       return drm_pci_init(&driver, &i810_pci_driver);
+       return drm_legacy_pci_init(&driver, &i810_pci_driver);
 }
 
 static void __exit i810_exit(void)
 {
-       drm_pci_exit(&driver, &i810_pci_driver);
+       drm_legacy_pci_exit(&driver, &i810_pci_driver);
 }
 
 module_init(i810_init);

diff --git a/drivers/gpu/drm/mga/mga_drv.c b/drivers/gpu/drm/mga/mga_drv.c
index 2a36ec611a44c02448d5379a6f8e11d478aeaed3..1aad27813c23f6b5e823573ed3440aed141f194f 100644 (file)
--- a/drivers/gpu/drm/mga/mga_drv.c
+++ b/drivers/gpu/drm/mga/mga_drv.c
@@ -89,12 +89,12 @@ static struct pci_driver mga_pci_driver = {
 static int __init mga_init(void)
 {
        driver.num_ioctls = mga_max_ioctl;
-       return drm_pci_init(&driver, &mga_pci_driver);
+       return drm_legacy_pci_init(&driver, &mga_pci_driver);
 }
 
 static void __exit mga_exit(void)
 {
-       drm_pci_exit(&driver, &mga_pci_driver);
+       drm_legacy_pci_exit(&driver, &mga_pci_driver);
 }
 
 module_init(mga_init);

diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
index 53a5982a04c70de3787fd25b873f4f5004d99537..4189160af7264bc1c99f3f6acc01b5d10dfb504d 100644 (file)
--- a/drivers/gpu/drm/mgag200/mgag200_drv.c
+++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
@@ -119,12 +119,13 @@ static int __init mgag200_init(void)
 
        if (mgag200_modeset == 0)
                return -EINVAL;
-       return drm_pci_init(&driver, &mgag200_pci_driver);
+
+       return pci_register_driver(&mgag200_pci_driver);
 }
 
 static void __exit mgag200_exit(void)
 {
-       drm_pci_exit(&driver, &mgag200_pci_driver);
+       pci_unregister_driver(&mgag200_pci_driver);
 }
 
 module_init(mgag200_init);

diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c
index 3e1a8da9b20f3da4067629211fcd4851328f040d..efa5489c5aeda58caae8a13637ac749ecbb85413 100644 (file)
--- a/drivers/gpu/drm/nouveau/nouveau_drm.c
+++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
@@ -1120,7 +1120,12 @@ nouveau_drm_init(void)
 
        nouveau_register_dsm_handler();
        nouveau_backlight_ctor();
-       return drm_pci_init(&driver_pci, &nouveau_drm_pci_driver);
+
+#ifdef CONFIG_PCI
+       return pci_register_driver(&nouveau_drm_pci_driver);
+#else
+       return 0;
+#endif
 }
 
 static void __exit
@@ -1129,7 +1134,9 @@ nouveau_drm_exit(void)
        if (!nouveau_modeset)
                return;
 
-       drm_pci_exit(&driver_pci, &nouveau_drm_pci_driver);
+#ifdef CONFIG_PCI
+       pci_unregister_driver(&nouveau_drm_pci_driver);
+#endif
        nouveau_backlight_dtor();
        nouveau_unregister_dsm_handler();
 

diff --git a/drivers/gpu/drm/qxl/qxl_drv.c b/drivers/gpu/drm/qxl/qxl_drv.c
index bb2d8da7e553ad2572f6122c1275328b4863ce43..15c97b16ee2107ba74842cce84f43105073be7c9 100644 (file)
--- a/drivers/gpu/drm/qxl/qxl_drv.c
+++ b/drivers/gpu/drm/qxl/qxl_drv.c
@@ -301,12 +301,12 @@ static int __init qxl_init(void)
        if (qxl_modeset == 0)
                return -EINVAL;
        qxl_driver.num_ioctls = qxl_max_ioctls;
-       return drm_pci_init(&qxl_driver, &qxl_pci_driver);
+       return pci_register_driver(&qxl_pci_driver);
 }
 
 static void __exit qxl_exit(void)
 {
-       drm_pci_exit(&qxl_driver, &qxl_pci_driver);
+       pci_unregister_driver(&qxl_pci_driver);
 }
 
 module_init(qxl_init);

diff --git a/drivers/gpu/drm/r128/r128_drv.c b/drivers/gpu/drm/r128/r128_drv.c
index 1d43c434328daf3bbc106da20e8f4f33e61ecb25..0d2b7e42b3a7d876c6879c24017d3e5cc2c08cc6 100644 (file)
--- a/drivers/gpu/drm/r128/r128_drv.c
+++ b/drivers/gpu/drm/r128/r128_drv.c
@@ -95,12 +95,12 @@ static int __init r128_init(void)
 {
        driver.num_ioctls = r128_max_ioctl;
 
-       return drm_pci_init(&driver, &r128_pci_driver);
+       return drm_legacy_pci_init(&driver, &r128_pci_driver);
 }
 
 static void __exit r128_exit(void)
 {
-       drm_pci_exit(&driver, &r128_pci_driver);
+       drm_legacy_pci_exit(&driver, &r128_pci_driver);
 }
 
 module_init(r128_init);

diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
index dd5e86dafb296c114791ed12de1d9b46c085d6ef..0d02349674f63618e6fd733e9f9ce867c3865885 100644 (file)
--- a/drivers/gpu/drm/radeon/radeon_drv.c
+++ b/drivers/gpu/drm/radeon/radeon_drv.c
@@ -619,14 +619,13 @@ static int __init radeon_init(void)
                return -EINVAL;
        }
 
-       /* let modprobe override vga console setting */
-       return drm_pci_init(driver, pdriver);
+       return pci_register_driver(pdriver);
 }
 
 static void __exit radeon_exit(void)
 {
        radeon_kfd_fini();
-       drm_pci_exit(driver, pdriver);
+       pci_unregister_driver(pdriver);
        radeon_unregister_atpx_handler();
 }
 

diff --git a/drivers/gpu/drm/savage/savage_drv.c b/drivers/gpu/drm/savage/savage_drv.c
index 2a08da09dbcf39e213312f940c10d2ac0734037b..2bddeb8bf457d16f55c965c11c90ff70cf6bfa91 100644 (file)
--- a/drivers/gpu/drm/savage/savage_drv.c
+++ b/drivers/gpu/drm/savage/savage_drv.c
@@ -74,12 +74,12 @@ static struct pci_driver savage_pci_driver = {
 static int __init savage_init(void)
 {
        driver.num_ioctls = savage_max_ioctl;
-       return drm_pci_init(&driver, &savage_pci_driver);
+       return drm_legacy_pci_init(&driver, &savage_pci_driver);
 }
 
 static void __exit savage_exit(void)
 {
-       drm_pci_exit(&driver, &savage_pci_driver);
+       drm_legacy_pci_exit(&driver, &savage_pci_driver);
 }
 
 module_init(savage_init);

diff --git a/drivers/gpu/drm/sis/sis_drv.c b/drivers/gpu/drm/sis/sis_drv.c
index cdaced381f5d64fdb0da38337b524f3ffd954f46..e04a92658cd74c239a467114d5f38132afc8e6e1 100644 (file)
--- a/drivers/gpu/drm/sis/sis_drv.c
+++ b/drivers/gpu/drm/sis/sis_drv.c
@@ -124,12 +124,12 @@ static struct pci_driver sis_pci_driver = {
 static int __init sis_init(void)
 {
        driver.num_ioctls = sis_max_ioctl;
-       return drm_pci_init(&driver, &sis_pci_driver);
+       return drm_legacy_pci_init(&driver, &sis_pci_driver);
 }
 
 static void __exit sis_exit(void)
 {
-       drm_pci_exit(&driver, &sis_pci_driver);
+       drm_legacy_pci_exit(&driver, &sis_pci_driver);
 }
 
 module_init(sis_init);

diff --git a/drivers/gpu/drm/tdfx/tdfx_drv.c b/drivers/gpu/drm/tdfx/tdfx_drv.c
index acd5f8162bb66a242c13e46230a0e67bb637fdbb..3a1476818c6530bb95e3854cc87beef2e425fe92 100644 (file)
--- a/drivers/gpu/drm/tdfx/tdfx_drv.c
+++ b/drivers/gpu/drm/tdfx/tdfx_drv.c
@@ -71,12 +71,12 @@ static struct pci_driver tdfx_pci_driver = {
 
 static int __init tdfx_init(void)
 {
-       return drm_pci_init(&driver, &tdfx_pci_driver);
+       return drm_legacy_pci_init(&driver, &tdfx_pci_driver);
 }
 
 static void __exit tdfx_exit(void)
 {
-       drm_pci_exit(&driver, &tdfx_pci_driver);
+       drm_legacy_pci_exit(&driver, &tdfx_pci_driver);
 }
 
 module_init(tdfx_init);

diff --git a/drivers/gpu/drm/via/via_drv.c b/drivers/gpu/drm/via/via_drv.c
index 0ca4e0489c0b66ede43281efbf1e6d329cc4f1c6..aaf766f7cca25a96a05f211cceea71c24e1699b8 100644 (file)
--- a/drivers/gpu/drm/via/via_drv.c
+++ b/drivers/gpu/drm/via/via_drv.c
@@ -106,12 +106,12 @@ static int __init via_init(void)
 {
        driver.num_ioctls = via_max_ioctl;
        via_init_command_verifier();
-       return drm_pci_init(&driver, &via_pci_driver);
+       return drm_legacy_pci_init(&driver, &via_pci_driver);
 }
 
 static void __exit via_exit(void)
 {
-       drm_pci_exit(&driver, &via_pci_driver);
+       drm_legacy_pci_exit(&driver, &via_pci_driver);
 }
 
 module_init(via_init);

diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
index 63218033b0bea12373b2f9553727840167e19525..204bf181b69ef1fc9b7f1e428edb13be24c55ab0 100644 (file)
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
@@ -1570,7 +1570,7 @@ static int __init vmwgfx_init(void)
        if (vgacon_text_force())
                return -EINVAL;
 
-       ret = drm_pci_init(&driver, &vmw_pci_driver);
+       ret = pci_register_driver(&vmw_pci_driver);
        if (ret)
                DRM_ERROR("Failed initializing DRM.\n");
        return ret;
@@ -1578,7 +1578,7 @@ static int __init vmwgfx_init(void)
 
 static void __exit vmwgfx_exit(void)
 {
-       drm_pci_exit(&driver, &vmw_pci_driver);
+       pci_unregister_driver(&vmw_pci_driver);
 }
 
 module_init(vmwgfx_init);

diff --git a/include/drm/drm_pci.h b/include/drm/drm_pci.h
index 961b16f9b5533806475cf74d6efcb2b16a96acdb..674599025d7d3b15fb08e5a7337475f8f9d588a8 100644 (file)
--- a/include/drm/drm_pci.h
+++ b/include/drm/drm_pci.h
@@ -43,8 +43,8 @@ struct drm_dma_handle *drm_pci_alloc(struct drm_device *dev, size_t size,
                                     size_t align);
 void drm_pci_free(struct drm_device *dev, struct drm_dma_handle * dmah);
 
-int drm_pci_init(struct drm_driver *driver, struct pci_driver *pdriver);
-void drm_pci_exit(struct drm_driver *driver, struct pci_driver *pdriver);
+int drm_legacy_pci_init(struct drm_driver *driver, struct pci_driver *pdriver);
+void drm_legacy_pci_exit(struct drm_driver *driver, struct pci_driver *pdriver);
 #ifdef CONFIG_PCI
 int drm_get_pci_dev(struct pci_dev *pdev,
                    const struct pci_device_id *ent,