diff options
author | Robin Murphy <robin.murphy@arm.com> | 2021-04-01 15:56:26 +0200 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2021-04-16 17:20:45 +0200 |
commit | 2d471b20c55e13c98d1dba413bf2de618e89cdac (patch) | |
tree | fe16cf7ce737c7c60d0cd414199cab0bfb4aea47 /drivers/iommu/msm_iommu.c | |
parent | iommu: Statically set module owner (diff) | |
download | linux-2d471b20c55e13c98d1dba413bf2de618e89cdac.tar.xz linux-2d471b20c55e13c98d1dba413bf2de618e89cdac.zip |
iommu: Streamline registration interface
Rather than have separate opaque setter functions that are easy to
overlook and lead to repetitive boilerplate in drivers, let's pass the
relevant initialisation parameters directly to iommu_device_register().
Acked-by: Will Deacon <will@kernel.org>
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/ab001b87c533b6f4db71eb90db6f888953986c36.1617285386.git.robin.murphy@arm.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu/msm_iommu.c')
-rw-r--r-- | drivers/iommu/msm_iommu.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/iommu/msm_iommu.c b/drivers/iommu/msm_iommu.c index f0ba6a09b434..7880f307cb2d 100644 --- a/drivers/iommu/msm_iommu.c +++ b/drivers/iommu/msm_iommu.c @@ -792,10 +792,7 @@ static int msm_iommu_probe(struct platform_device *pdev) goto fail; } - iommu_device_set_ops(&iommu->iommu, &msm_iommu_ops); - iommu_device_set_fwnode(&iommu->iommu, &pdev->dev.of_node->fwnode); - - ret = iommu_device_register(&iommu->iommu); + ret = iommu_device_register(&iommu->iommu, &msm_iommu_ops, &pdev->dev); if (ret) { pr_err("Could not register msm-smmu at %pa\n", &ioaddr); goto fail; |