mxser: Use non-hybrid PCI devres API
authorPhilipp Stanner <phasta@kernel.org>
Thu, 17 Apr 2025 08:13:34 +0000 (10:13 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 25 Apr 2025 11:42:35 +0000 (13:42 +0200)
mxser enables its PCI device with pcim_enable_device(). This,
implicitly, switches the function pci_request_region() into managed
mode, where it becomes a devres function.

The PCI subsystem wants to remove this hybrid nature from its
interfaces. To do so, users of the aforementioned combination of
functions must be ported to non-hybrid functions.

Replace the call to sometimes-managed pci_request_region() with one to
the always-managed pcim_request_region().

Signed-off-by: Philipp Stanner <phasta@kernel.org>
Link: https://lore.kernel.org/r/20250417081333.20917-2-phasta@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/mxser.c

index 4d45eca4929abdfd60e90f96160a8f8bb005d02e..2fc13cc02cc51b04081553be19479556bd244cf5 100644 (file)
@@ -1812,7 +1812,7 @@ static int mxser_probe(struct pci_dev *pdev,
 
        /* io address */
        ioaddress = pci_resource_start(pdev, 2);
-       retval = pci_request_region(pdev, 2, "mxser(IO)");
+       retval = pcim_request_region(pdev, 2, "mxser(IO)");
        if (retval)
                goto err_zero;
 
@@ -1822,7 +1822,7 @@ static int mxser_probe(struct pci_dev *pdev,
 
        /* vector */
        ioaddress = pci_resource_start(pdev, 3);
-       retval = pci_request_region(pdev, 3, "mxser(vector)");
+       retval = pcim_request_region(pdev, 3, "mxser(vector)");
        if (retval)
                goto err_zero;
        brd->vector = ioaddress;