diff options
author | 2023-03-22 14:49:51 +0800 | |
---|---|---|
committer | 2023-03-22 15:45:15 +0100 | |
commit | 24dfb197c3b7d274e5f78fce5e63392526a933e9 (patch) | |
tree | 3af5af7bebef7537baa64dfcf0aba1219217a3ac /drivers/iommu/iommu.c | |
parent | iommu: Use of_property_present() for testing DT property presence (diff) | |
download | linux-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