]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
net: tulip: Use pure PCI devres API
authorPhilipp Stanner <phasta@kernel.org>
Fri, 25 Apr 2025 08:57:36 +0000 (10:57 +0200)
committerJakub Kicinski <kuba@kernel.org>
Mon, 28 Apr 2025 23:19:15 +0000 (16:19 -0700)
The currently used function pci_request_regions() is one of the
problematic "hybrid devres" PCI functions, which are sometimes managed
through devres, and sometimes not (depending on whether
pci_enable_device() or pcim_enable_device() has been called before).

The PCI subsystem wants to remove this behavior and, therefore, needs to
port all users to functions that don't have this problem.

Replace pci_request_regions() with pcim_request_all_regions().

Signed-off-by: Philipp Stanner <phasta@kernel.org>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Link: https://patch.msgid.link/20250425085740.65304-5-phasta@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/dec/tulip/tulip_core.c
drivers/net/ethernet/dec/tulip/winbond-840.c

index c8c53121557f27952dfc8a560a57767670c4d411..bec76e7bf5dd14f94e8fb720436d304678ee94ab 100644 (file)
@@ -1411,7 +1411,7 @@ static int tulip_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 
        /* grab all resources from both PIO and MMIO regions, as we
         * don't want anyone else messing around with our hardware */
-       if (pci_request_regions(pdev, DRV_NAME))
+       if (pcim_request_all_regions(pdev, DRV_NAME))
                return -ENODEV;
 
        ioaddr = pcim_iomap(pdev, TULIP_BAR, tulip_tbl[chip_idx].io_size);
index 5930cdec6f2f6585e259d35e199ab55ec84b00fa..e593273b28675082f547bda247a20c6f544e7097 100644 (file)
@@ -375,7 +375,7 @@ static int w840_probe1(struct pci_dev *pdev, const struct pci_device_id *ent)
                return -ENOMEM;
        SET_NETDEV_DEV(dev, &pdev->dev);
 
-       if (pci_request_regions(pdev, DRV_NAME))
+       if (pcim_request_all_regions(pdev, DRV_NAME))
                goto err_out_netdev;
 
        ioaddr = pci_iomap(pdev, TULIP_BAR, netdev_res_size);