diff options
| author | Simon Geis <simon.geis@fau.de> | 2019-12-13 14:53:10 +0100 |
|---|---|---|
| committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2019-12-16 11:49:53 +0100 |
| commit | 908864641f635892d4df265fe92425a5166c748b (patch) | |
| tree | 419691ac7186fea8452eab11ea84060dde7ac0e9 /drivers/pcmcia | |
| parent | PCMCIA/i82092: change code indentation (diff) | |
| download | linux-908864641f635892d4df265fe92425a5166c748b.tar.xz linux-908864641f635892d4df265fe92425a5166c748b.zip | |
PCMCIA/i82092: move assignment out of if condition
Improve readability by moving the assignment out of if conditions.
Co-developed-by: Lukas Panzer <lukas.panzer@fau.de>
Signed-off-by: Lukas Panzer <lukas.panzer@fau.de>
Signed-off-by: Simon Geis <simon.geis@fau.de>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia')
| -rw-r--r-- | drivers/pcmcia/i82092.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/pcmcia/i82092.c b/drivers/pcmcia/i82092.c index c7996fe25622..2d31832fa439 100644 --- a/drivers/pcmcia/i82092.c +++ b/drivers/pcmcia/i82092.c @@ -77,7 +77,8 @@ static int i82092aa_pci_probe(struct pci_dev *dev, const struct pci_device_id *i enter("i82092aa_pci_probe"); - if ((ret = pci_enable_device(dev))) + ret = pci_enable_device(dev); + if (ret) return ret; pci_read_config_byte(dev, 0x40, &configbyte); /* PCI Configuration Control */ @@ -133,7 +134,9 @@ static int i82092aa_pci_probe(struct pci_dev *dev, const struct pci_device_id *i /* Register the interrupt handler */ dev_dbg(&dev->dev, "Requesting interrupt %i\n", dev->irq); - if ((ret = request_irq(dev->irq, i82092aa_interrupt, IRQF_SHARED, "i82092aa", i82092aa_interrupt))) { + ret = request_irq(dev->irq, i82092aa_interrupt, IRQF_SHARED, + "i82092aa", i82092aa_interrupt); + if (ret) { dev_err(&dev->dev, "Failed to register IRQ %d, aborting\n", dev->irq); goto err_out_free_res; |