Hi

Am 01.07.26 um 19:05 schrieb Uwe Kleine-König (The Capable Hub):
Having an empty remove callback is equivalent to no remove callback.
(The only minor difference is that with an empty remove callback
pm_runtime_get_sync() and pm_runtime_put_noidle() are called.)

Drop this useless function.

Signed-off-by: Uwe Kleine-König (The Capable Hub) <[email protected]>
---
  drivers/gpu/drm/hyperv/hyperv_drm_drv.c | 5 -----
  1 file changed, 5 deletions(-)

diff --git a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c 
b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
index e766d87b7a9d..e3f41336a831 100644
--- a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
+++ b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
@@ -45,10 +45,6 @@ static int hv_drm_pci_probe(struct pci_dev *pdev,
        return 0;
  }
-static void hv_drm_pci_remove(struct pci_dev *pdev)
-{

It would be better to call drm_dev_unplug() from here.  With a bit more work, the driver can have hot-unplug functionality.

Best regards
Thomas

-}
-
  static const struct pci_device_id hv_drm_pci_tbl[] = {
        {
                PCI_VDEVICE_SUB(MICROSOFT, PCI_DEVICE_ID_HYPERV_VIDEO,
@@ -64,7 +60,6 @@ static struct pci_driver hv_drm_pci_driver = {
        .name =         KBUILD_MODNAME,
        .id_table =     hv_drm_pci_tbl,
        .probe =        hv_drm_pci_probe,
-       .remove =       hv_drm_pci_remove,
  };
static int hv_drm_setup_vram(struct hv_drm_device *hv,

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)



Reply via email to