summaryrefslogtreecommitdiffstats
path: root/drivers/parport
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2018-02-07 14:47:54 -0500
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2018-03-14 12:54:09 -0400
commit96edf5376e9250eb61d2bbc5432c13846878edb2 (patch)
tree292b41ffdfc35cfbda526ad9baf40eb34ced99c9 /drivers/parport
parentad8ce834bfec77c8ae2fbb3a0593ae87bdecebcb (diff)
parport: Don't shadow error codes in ->probe()
When ->probe() calls helper functions return theirs error codes instead of shadowing them. Tested-by: Nikola Ciprich <nikola.ciprich@linuxbox.cz> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/parport')
-rw-r--r--drivers/parport/parport_serial.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c
index 050eeb06913a..c222698410c2 100644
--- a/drivers/parport/parport_serial.c
+++ b/drivers/parport/parport_serial.c
@@ -539,12 +539,10 @@ static int serial_register(struct pci_dev *dev, const struct pci_device_id *id)
539 struct serial_private *serial; 539 struct serial_private *serial;
540 540
541 board = &pci_parport_serial_boards[id->driver_data]; 541 board = &pci_parport_serial_boards[id->driver_data];
542
543 if (board->num_ports == 0) 542 if (board->num_ports == 0)
544 return 0; 543 return 0;
545 544
546 serial = pciserial_init_ports(dev, board); 545 serial = pciserial_init_ports(dev, board);
547
548 if (IS_ERR(serial)) 546 if (IS_ERR(serial))
549 return PTR_ERR(serial); 547 return PTR_ERR(serial);
550 548
@@ -630,14 +628,16 @@ static int parport_serial_pci_probe(struct pci_dev *dev,
630 if (err) 628 if (err)
631 return err; 629 return err;
632 630
633 if (parport_register(dev, id)) 631 err = parport_register(dev, id);
634 return -ENODEV; 632 if (err)
633 return err;
635 634
636 if (serial_register (dev, id)) { 635 err = serial_register(dev, id);
636 if (err) {
637 int i; 637 int i;
638 for (i = 0; i < priv->num_par; i++) 638 for (i = 0; i < priv->num_par; i++)
639 parport_pc_unregister_port (priv->port[i]); 639 parport_pc_unregister_port (priv->port[i]);
640 return -ENODEV; 640 return err;
641 } 641 }
642 642
643 return 0; 643 return 0;