aboutsummaryrefslogtreecommitdiff
path: root/drivers/iommu/iommu.c
diff options
context:
space:
mode:
authorGravatar Lu Baolu <baolu.lu@linux.intel.com> 2023-03-22 14:49:51 +0800
committerGravatar Joerg Roedel <jroedel@suse.de> 2023-03-22 15:45:15 +0100
commit24dfb197c3b7d274e5f78fce5e63392526a933e9 (patch)
tree3af5af7bebef7537baa64dfcf0aba1219217a3ac /drivers/iommu/iommu.c
parentiommu: Use of_property_present() for testing DT property presence (diff)
downloadlinux-24dfb197c3b7d274e5f78fce5e63392526a933e9.tar.gz
linux-24dfb197c3b7d274e5f78fce5e63392526a933e9.tar.bz2
linux-24dfb197c3b7d274e5f78fce5e63392526a933e9.zip
iommu/ipmmu-vmsa: Call arm_iommu_release_mapping() in release path
In the iommu driver's release_device operation, the driver should detach the device from any attached domain and release the resources allocated in the probe_device and probe_finalize paths. Replace arm_iommu_detach_device() with arm_iommu_release_mapping() in the release path of the ipmmu-vmsa driver. The device_release callback is called in device_del(), this device is not coming back. Zeroing out pointers and testing for a condition which cannot be true by construction is simply a waste of time and code. The bonus is that it also removes a obstacle of arm_iommu_detach_device() re-entering the iommu core during release_device. With this removed, the iommu core code could be simplified a lot. Signed-off-by: Jason Gunthorpe <jgg@nvidia.com> Suggested-by: Robin Murphy <robin.murphy@arm.com> Link: https://lore.kernel.org/linux-iommu/7b248ba1-3967-5cd8-82e9-0268c706d320@arm.com/ Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com> Reviewed-by: Jason Gunthorpe <jgg@nvidia.com> Link: https://lore.kernel.org/r/20230322064956.263419-2-baolu.lu@linux.intel.com Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu/iommu.c')
0 files changed, 0 insertions, 0 deletions