diff options
| author | Jiri Slaby <jslaby@suse.cz> | 2021-06-18 08:14:51 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-06-18 13:10:01 +0200 |
| commit | dcb04e21bba349875c97467062a0d5cb06f7524d (patch) | |
| tree | d346f8650108e0ec295a052e152ba280d3e95ae3 /drivers | |
| parent | d450f0852fccccc4ae7ec4103e146ae1b61a6fef (diff) | |
mxser: pci, switch to managed resources
Switch to managed resources for PCI using pcim_enable_device.
Regions and device disabling is now taken care of by the core and we
need not unwind. Neither in ->probe, nor in ->remove. That simplifies
the code.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Link: https://lore.kernel.org/r/20210618061516.662-46-jslaby@suse.cz
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/tty/mxser.c | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/drivers/tty/mxser.c b/drivers/tty/mxser.c index 196750676400..be58ee025180 100644 --- a/drivers/tty/mxser.c +++ b/drivers/tty/mxser.c @@ -1978,7 +1978,7 @@ static int mxser_probe(struct pci_dev *pdev, mxser_cards[ent->driver_data].name, pdev->bus->number, PCI_SLOT(pdev->devfn)); - retval = pci_enable_device(pdev); + retval = pcim_enable_device(pdev); if (retval) { dev_err(&pdev->dev, "PCI enable failed\n"); goto err; @@ -1988,7 +1988,7 @@ static int mxser_probe(struct pci_dev *pdev, ioaddress = pci_resource_start(pdev, 2); retval = pci_request_region(pdev, 2, "mxser(IO)"); if (retval) - goto err_dis; + goto err; brd->info = &mxser_cards[ent->driver_data]; for (i = 0; i < brd->info->nports; i++) @@ -2007,7 +2007,7 @@ static int mxser_probe(struct pci_dev *pdev, /* mxser_initbrd will hook ISR. */ retval = mxser_initbrd(brd); if (retval) - goto err_rel3; + goto err_zero; for (i = 0; i < brd->info->nports; i++) { tty_dev = tty_port_register_device(&brd->ports[i].port, @@ -2028,13 +2028,8 @@ err_relbrd: for (i = 0; i < brd->info->nports; i++) tty_port_destroy(&brd->ports[i].port); free_irq(brd->irq, brd); -err_rel3: - pci_release_region(pdev, 3); err_zero: brd->info = NULL; - pci_release_region(pdev, 2); -err_dis: - pci_disable_device(pdev); err: return retval; } @@ -2051,9 +2046,6 @@ static void mxser_remove(struct pci_dev *pdev) free_irq(brd->irq, brd); - pci_release_region(pdev, 2); - pci_release_region(pdev, 3); - pci_disable_device(pdev); brd->info = NULL; } |