PCI: endpoint: Make struct pci_epf_driver::remove return void

The driver core ignores the return value of pci_epf_device_remove()
(because there is only little it can do when a device disappears) and
there are no pci_epf_drivers with a remove callback.

So make it impossible for future drivers to return an unused error code
by changing the remove prototype to return void.

The real motivation for this change is the quest to make struct
bus_type::remove return void, too.

Acked-by: Kishon Vijay Abraham I <kishon@ti.com>
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20210713193522.1770306-2-u.kleine-koenig@pengutronix.de
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/pci/endpoint/pci-epf-core.c b/drivers/pci/endpoint/pci-epf-core.c
index e9289d10f82258c49e56a2ef09a77ca1dc1d48c9..4b9ad96bf1b26fd1da149583752a7e6334e9d94c 100644 (file)
--- a/drivers/pci/endpoint/pci-epf-core.c
+++ b/drivers/pci/endpoint/pci-epf-core.c
@@ -389,15 +389,14 @@ static int pci_epf_device_probe(struct device *dev)
 
 static int pci_epf_device_remove(struct device *dev)
 {
-       int ret = 0;
        struct pci_epf *epf = to_pci_epf(dev);
        struct pci_epf_driver *driver = to_pci_epf_driver(dev->driver);
 
        if (driver->remove)
-               ret = driver->remove(epf);
+               driver->remove(epf);
        epf->driver = NULL;
 
-       return ret;
+       return 0;
 }
 
 static struct bus_type pci_epf_bus_type = {

diff --git a/include/linux/pci-epf.h b/include/linux/pci-epf.h
index 2debc27ba95e5513ff1d78e3117f597d27cc4d82..8292420426f3d72c736895d8d5128fd2dfa6e62f 100644 (file)
--- a/include/linux/pci-epf.h
+++ b/include/linux/pci-epf.h
@@ -85,7 +85,7 @@ struct pci_epf_ops {
  */
 struct pci_epf_driver {
        int     (*probe)(struct pci_epf *epf);
-       int     (*remove)(struct pci_epf *epf);
+       void    (*remove)(struct pci_epf *epf);
 
        struct device_driver    driver;
        struct pci_epf_ops      *ops;