Commit fe61b290 authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Hans Verkuil
Browse files

media: intel/ipu6: Fix some redundant resources freeing in ipu6_pci_remove() 



pcim_iomap_regions() and pcim_enable_device() are used in the probe. So
the corresponding managed resources don't need to be freed explicitly in
the remove function.

Remove the incorrect pci_release_regions() and pci_disable_device() calls.

Fixes: 25fedc02 ("media: intel/ipu6: add Intel IPU6 PCI device driver")
Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: default avatarBingbu Cao <bingbu.cao@intel.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent fd7ccfb1

drivers/media/pci/intel/ipu6/ipu6.c

+0 −3
Original line number Diff line number Diff line
@@ -727,9 +727,6 @@ static void ipu6_pci_remove(struct pci_dev *pdev) 
	pm_runtime_forbid(&pdev->dev);

	pm_runtime_get_noresume(&pdev->dev);



	pci_release_regions(pdev);

	pci_disable_device(pdev);



	release_firmware(isp->cpd_fw);



	ipu6_mmu_cleanup(psys_mmu);