[PATCH] iommu: Unexport iommu_fwspec_free()
Jason Gunthorpe
jgg at ziepe.ca
Wed Mar 5 10:29:51 PST 2025
On Thu, Feb 27, 2025 at 02:47:47PM +0000, Robin Murphy wrote:
> The drivers doing their own fwspec parsing have no need to call
> iommu_fwspec_free() since fwspecs were moved into dev_iommu, as
> returning an error from .probe_device will tear down the whole lot
> anyway. Move it into the private interface now that it only serves
> for of_iommu to clean up in an error case.
>
> I have no idea what mtk_v1 was doing in effectively guaranteeing
> a NULL fwspec would be dereferenced if no "iommus" DT property was
> found, so add a check for that to at least make the code look sane.
> Signed-off-by: Robin Murphy <robin.murphy at arm.com>
> ---
> drivers/iommu/arm/arm-smmu/arm-smmu.c | 1 -
> drivers/iommu/iommu-priv.h | 2 ++
> drivers/iommu/iommu.c | 1 -
> drivers/iommu/mtk_iommu_v1.c | 14 ++++----------
> drivers/iommu/tegra-smmu.c | 1 -
> include/linux/iommu.h | 5 -----
> 6 files changed, 6 insertions(+), 18 deletions(-)
Reviewed-by: Jason Gunthorpe <jgg at nvidia.com>
Jason
More information about the linux-arm-kernel
mailing list