summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2015-11-21 09:16:38 -0200
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2015-12-03 15:09:28 -0200
commitf7488c50f3e9ff8fe1893f4e6d447ec96e721261 (patch)
tree23159f09758a08f0ee1c88244799bf9b953ba843
parentc1500ba0b61e9abf95e0e7ecd3c4ad877f019abe (diff)
downloadop-kernel-dev-f7488c50f3e9ff8fe1893f4e6d447ec96e721261.zip
op-kernel-dev-f7488c50f3e9ff8fe1893f4e6d447ec96e721261.tar.gz
[media] dm1105: Remove unnecessary synchronize_irq() before free_irq()
Calling synchronize_irq() right before free_irq() is quite useless. On one hand the IRQ can easily fire again before free_irq() is entered, on the other hand free_irq() itself calls synchronize_irq() internally (in a race condition free way), before any state associated with the IRQ is freed. Patch was generated using the following semantic patch: // <smpl> @@ expression irq; @@ -synchronize_irq(irq); free_irq(irq, ...); // </smpl> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r--drivers/media/pci/dm1105/dm1105.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/pci/dm1105/dm1105.c b/drivers/media/pci/dm1105/dm1105.c
index 88915fb..5dd5047 100644
--- a/drivers/media/pci/dm1105/dm1105.c
+++ b/drivers/media/pci/dm1105/dm1105.c
@@ -1206,7 +1206,6 @@ static void dm1105_remove(struct pci_dev *pdev)
i2c_del_adapter(&dev->i2c_adap);
dm1105_hw_exit(dev);
- synchronize_irq(pdev->irq);
free_irq(pdev->irq, dev);
pci_iounmap(pdev, dev->io_mem);
pci_release_regions(pdev);
OpenPOWER on IntegriCloud