arm/tegra: Call bus_set_iommu() after iommu_device_register()
authorJoerg Roedel <jroedel@suse.de>
Wed, 30 Aug 2017 13:06:43 +0000 (15:06 +0200)
committerJoerg Roedel <jroedel@suse.de>
Wed, 30 Aug 2017 15:28:32 +0000 (17:28 +0200)
The bus_set_iommu() function will call the add_device()
call-back which needs the iommu to be registered.

Reported-by: Jon Hunter <jonathanh@nvidia.com>
Fixes: 0b480e447006 ('iommu/tegra: Add support for struct iommu_device')
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/tegra-smmu.c

index 2802e12e6a54db2fc916a0d872d656d6b0eaab21..3b6449e2cbf1c47d6d923f28d598d90e7382b043 100644 (file)
@@ -949,10 +949,6 @@ struct tegra_smmu *tegra_smmu_probe(struct device *dev,
 
        tegra_smmu_ahb_enable();
 
-       err = bus_set_iommu(&platform_bus_type, &tegra_smmu_ops);
-       if (err < 0)
-               return ERR_PTR(err);
-
        err = iommu_device_sysfs_add(&smmu->iommu, dev, NULL, dev_name(dev));
        if (err)
                return ERR_PTR(err);
@@ -965,6 +961,13 @@ struct tegra_smmu *tegra_smmu_probe(struct device *dev,
                return ERR_PTR(err);
        }
 
+       err = bus_set_iommu(&platform_bus_type, &tegra_smmu_ops);
+       if (err < 0) {
+               iommu_device_unregister(&smmu->iommu);
+               iommu_device_sysfs_remove(&smmu->iommu);
+               return ERR_PTR(err);
+       }
+
        if (IS_ENABLED(CONFIG_DEBUG_FS))
                tegra_smmu_debugfs_init(smmu);