diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2018-02-07 14:47:54 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-03-14 12:54:09 -0400 |
commit | 96edf5376e9250eb61d2bbc5432c13846878edb2 (patch) | |
tree | 292b41ffdfc35cfbda526ad9baf40eb34ced99c9 /drivers/parport | |
parent | ad8ce834bfec77c8ae2fbb3a0593ae87bdecebcb (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.c | 12 |
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; |