drm/tegra: vic: Use common IOMMU attach/detach code
authorThierry Reding <treding@nvidia.com>
Fri, 8 Feb 2019 13:15:44 +0000 (14:15 +0100)
committerThierry Reding <treding@nvidia.com>
Mon, 28 Oct 2019 10:18:36 +0000 (11:18 +0100)
Reuse common code to attach to or detach from an IOMMU domain.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/gpu/drm/tegra/vic.c

index d60e479..e4b17c7 100644 (file)
@@ -34,7 +34,7 @@ struct vic {
        void __iomem *regs;
        struct tegra_drm_client client;
        struct host1x_channel *channel;
-       struct iommu_domain *domain;
+       struct iommu_group *group;
        struct device *dev;
        struct clk *clk;
        struct reset_control *rst;
@@ -183,21 +183,16 @@ static const struct falcon_ops vic_falcon_ops = {
 static int vic_init(struct host1x_client *client)
 {
        struct tegra_drm_client *drm = host1x_to_drm_client(client);
-       struct iommu_group *group = iommu_group_get(client->dev);
        struct drm_device *dev = dev_get_drvdata(client->parent);
        struct tegra_drm *tegra = dev->dev_private;
        struct vic *vic = to_vic(drm);
        int err;
 
-       if (group && tegra->domain) {
-               err = iommu_attach_group(tegra->domain, group);
-               if (err < 0) {
-                       dev_err(vic->dev, "failed to attach to domain: %d\n",
-                               err);
-                       return err;
-               }
-
-               vic->domain = tegra->domain;
+       vic->group = host1x_client_iommu_attach(client, false);
+       if (IS_ERR(vic->group)) {
+               err = PTR_ERR(vic->group);
+               dev_err(vic->dev, "failed to attach to domain: %d\n", err);
+               return err;
        }
 
        vic->channel = host1x_channel_request(client);
@@ -229,8 +224,7 @@ free_syncpt:
 free_channel:
        host1x_channel_put(vic->channel);
 detach:
-       if (group && tegra->domain)
-               iommu_detach_group(tegra->domain, group);
+       host1x_client_iommu_detach(client, vic->group);
 
        return err;
 }
@@ -238,7 +232,6 @@ detach:
 static int vic_exit(struct host1x_client *client)
 {
        struct tegra_drm_client *drm = host1x_to_drm_client(client);
-       struct iommu_group *group = iommu_group_get(client->dev);
        struct drm_device *dev = dev_get_drvdata(client->parent);
        struct tegra_drm *tegra = dev->dev_private;
        struct vic *vic = to_vic(drm);
@@ -253,11 +246,7 @@ static int vic_exit(struct host1x_client *client)
 
        host1x_syncpt_free(client->syncpts[0]);
        host1x_channel_put(vic->channel);
-
-       if (vic->domain) {
-               iommu_detach_group(vic->domain, group);
-               vic->domain = NULL;
-       }
+       host1x_client_iommu_detach(client, vic->group);
 
        return 0;
 }