summaryrefslogtreecommitdiffstats
path: root/drivers/staging/comedi
diff options
context:
space:
mode:
authorBhumika Goyal <bhumirks@gmail.com>2016-09-12 23:02:47 +0530
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-09-13 14:51:53 +0200
commit25b80dbe3fa6d76d8f19af0623d1fd8ed0dd671e (patch)
tree0f2d1ba866a00a40e0544ae49bf4c9ac7df3b564 /drivers/staging/comedi
parent6e30293976983daf09eea8e882d48a09c80d5987 (diff)
downloadop-kernel-dev-25b80dbe3fa6d76d8f19af0623d1fd8ed0dd671e.zip
op-kernel-dev-25b80dbe3fa6d76d8f19af0623d1fd8ed0dd671e.tar.gz
Staging: comedi: drivers: Delete NULL check before pci_dev_put
The function pci_dev_put checks whether its argument is NULL and returns immediately. Therefore, NULL test before the call if not needed. Found using coccinelle: @@ expression x; @@ - if(x) - pci_dev_put(x); + pci_dev_put(x); Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> Reviewed-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/comedi')
-rw-r--r--drivers/staging/comedi/drivers/adl_pci9118.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/comedi/drivers/adl_pci9118.c b/drivers/staging/comedi/drivers/adl_pci9118.c
index be70bd3..86450c0 100644
--- a/drivers/staging/comedi/drivers/adl_pci9118.c
+++ b/drivers/staging/comedi/drivers/adl_pci9118.c
@@ -1693,8 +1693,7 @@ static void pci9118_detach(struct comedi_device *dev)
pci9118_reset(dev);
comedi_pci_detach(dev);
pci9118_free_dma(dev);
- if (pcidev)
- pci_dev_put(pcidev);
+ pci_dev_put(pcidev);
}
static struct comedi_driver adl_pci9118_driver = {
OpenPOWER on IntegriCloud