aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2015-11-21 06:16:38 -0500
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2015-12-03 12:09:28 -0500
commitf7488c50f3e9ff8fe1893f4e6d447ec96e721261 (patch)
tree23159f09758a08f0ee1c88244799bf9b953ba843
parentc1500ba0b61e9abf95e0e7ecd3c4ad877f019abe (diff)
[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 88915fb87e80..5dd504741b12 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)
1206 i2c_del_adapter(&dev->i2c_adap); 1206 i2c_del_adapter(&dev->i2c_adap);
1207 1207
1208 dm1105_hw_exit(dev); 1208 dm1105_hw_exit(dev);
1209 synchronize_irq(pdev->irq);
1210 free_irq(pdev->irq, dev); 1209 free_irq(pdev->irq, dev);
1211 pci_iounmap(pdev, dev->io_mem); 1210 pci_iounmap(pdev, dev->io_mem);
1212 pci_release_regions(pdev); 1211 pci_release_regions(pdev);