diff options
author | Olli Salonen <olli.salonen@iki.fi> | 2015-02-21 16:45:26 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-03-03 09:16:09 -0500 |
commit | 3f845f3c4cf4212bb4cbc8c06344d095cbb16bdd (patch) | |
tree | 51fc3bfef004e5682cf43a9b52f16a2c8bf443f1 /drivers/media/pci | |
parent | f35593fdc8851aa28f6949bd48fe90653d383017 (diff) |
[media] saa7164: free_irq before pci_disable_device
Free the IRQ before disabling the device. Otherwise errors like this when unloading the module:
[21135.458560] ------------[ cut here ]------------
[21135.458569] WARNING: CPU: 4 PID: 1696 at /home/apw/COD/linux/fs/proc/generic.c:521 remove_proc_entry+0x1a1/0x1b0()
[21135.458572] remove_proc_entry: removing non-empty directory 'irq/47', leaking at least 'saa7164[0]'
Signed-off-by: Olli Salonen <olli.salonen@iki.fi>
Reviewed-by: Steven Toth <stoth@kernellabs.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/pci')
-rw-r--r-- | drivers/media/pci/saa7164/saa7164-core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/pci/saa7164/saa7164-core.c b/drivers/media/pci/saa7164/saa7164-core.c index 4b0bec3766ed..9cf3c6cba498 100644 --- a/drivers/media/pci/saa7164/saa7164-core.c +++ b/drivers/media/pci/saa7164/saa7164-core.c | |||
@@ -1436,11 +1436,11 @@ static void saa7164_finidev(struct pci_dev *pci_dev) | |||
1436 | saa7164_i2c_unregister(&dev->i2c_bus[1]); | 1436 | saa7164_i2c_unregister(&dev->i2c_bus[1]); |
1437 | saa7164_i2c_unregister(&dev->i2c_bus[2]); | 1437 | saa7164_i2c_unregister(&dev->i2c_bus[2]); |
1438 | 1438 | ||
1439 | pci_disable_device(pci_dev); | ||
1440 | |||
1441 | /* unregister stuff */ | 1439 | /* unregister stuff */ |
1442 | free_irq(pci_dev->irq, dev); | 1440 | free_irq(pci_dev->irq, dev); |
1443 | 1441 | ||
1442 | pci_disable_device(pci_dev); | ||
1443 | |||
1444 | mutex_lock(&devlist); | 1444 | mutex_lock(&devlist); |
1445 | list_del(&dev->devlist); | 1445 | list_del(&dev->devlist); |
1446 | mutex_unlock(&devlist); | 1446 | mutex_unlock(&devlist); |