PCI/VGA: Remove empty vga_arb_device_card_gone()
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 24 Feb 2022 22:47:50 +0000 (16:47 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Thu, 10 Mar 2022 00:31:24 +0000 (18:31 -0600)
vga_arb_device_card_gone() has always been empty.  Remove it.

Link: https://lore.kernel.org/r/20220224224753.297579-9-helgaas@kernel.org
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/vgaarb.c

index e8d5efd85ba6203fae6ac2cf90db43f35f6dcd2c..582f0054b71a7929fb12906b5ba6bf3733f55f0c 100644 (file)
@@ -123,8 +123,6 @@ both:
 /* this is only used a cookie - it should not be dereferenced */
 static struct pci_dev *vga_default;
 
-static void vga_arb_device_card_gone(struct pci_dev *pdev);
-
 /* Find somebody in our list */
 static struct vga_device *vgadev_find(struct pci_dev *pdev)
 {
@@ -878,10 +876,6 @@ static bool vga_arbiter_del_pci_device(struct pci_dev *pdev)
        /* Remove entry from list */
        list_del(&vgadev->list);
        vga_count--;
-       /* Notify userland driver that the device is gone so it discards
-        * it's copies of the pci_dev pointer
-        */
-       vga_arb_device_card_gone(pdev);
 
        /* Wake up all possible waiters */
        wake_up_all(&vga_wait_queue);
@@ -1131,9 +1125,7 @@ static ssize_t vga_arb_read(struct file *file, char __user *buf,
        if (lbuf == NULL)
                return -ENOMEM;
 
-       /* Shields against vga_arb_device_card_gone (pci_dev going
-        * away), and allows access to vga list
-        */
+       /* Protects vga_list */
        spin_lock_irqsave(&vga_lock, flags);
 
        /* If we are targeting the default, use it */
@@ -1150,8 +1142,6 @@ static ssize_t vga_arb_read(struct file *file, char __user *buf,
                /* Wow, it's not in the list, that shouldn't happen,
                 * let's fix us up and return invalid card
                 */
-               if (pdev == priv->target)
-                       vga_arb_device_card_gone(pdev);
                spin_unlock_irqrestore(&vga_lock, flags);
                len = sprintf(lbuf, "invalid");
                goto done;
@@ -1495,10 +1485,6 @@ static int vga_arb_release(struct inode *inode, struct file *file)
        return 0;
 }
 
-static void vga_arb_device_card_gone(struct pci_dev *pdev)
-{
-}
-
 /*
  * callback any registered clients to let them know we have a
  * change in VGA cards