drm/exynos: fimd: check error status for drm_iommu_attach_device
authorAjay Kumar <ajaykumar.rs@samsung.com>
Sun, 11 Jan 2015 16:57:07 +0000 (01:57 +0900)
committerInki Dae <daeinki@gmail.com>
Sun, 25 Jan 2015 12:28:09 +0000 (21:28 +0900)
check error status for drm_iommu_attach_device() and make sure
it propagates till the caller.

Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
Signed-off-by: Inki Dae <daeinki@gmail.com>
drivers/gpu/drm/exynos/exynos_drm_fimd.c

index d54ca07658c961b4e150ea626cf3ca0b51848827..682806ef4d337edb7748107a33ef138812616b49 100644 (file)
@@ -296,12 +296,19 @@ static int fimd_ctx_initialize(struct fimd_context *ctx,
 
        /* attach this sub driver to iommu mapping if supported. */
        if (is_drm_iommu_supported(ctx->drm_dev)) {
+               int ret;
+
                /*
                 * If any channel is already active, iommu will throw
                 * a PAGE FAULT when enabled. So clear any channel if enabled.
                 */
                fimd_clear_channel(ctx->crtc);
-               drm_iommu_attach_device(ctx->drm_dev, ctx->dev);
+               ret = drm_iommu_attach_device(ctx->drm_dev, ctx->dev);
+               if (ret) {
+                       DRM_ERROR("drm_iommu_attach failed.\n");
+                       return ret;
+               }
+
        }
 
        return 0;
@@ -1056,6 +1063,7 @@ static int fimd_bind(struct device *dev, struct device *master, void *data)
 {
        struct fimd_context *ctx = dev_get_drvdata(dev);
        struct drm_device *drm_dev = data;
+       int ret;
 
        ctx->crtc = exynos_drm_crtc_create(drm_dev, ctx->pipe,
                                           EXYNOS_DISPLAY_TYPE_LCD,
@@ -1063,7 +1071,12 @@ static int fimd_bind(struct device *dev, struct device *master, void *data)
        if (IS_ERR(ctx->crtc))
                return PTR_ERR(ctx->crtc);
 
-       fimd_ctx_initialize(ctx, drm_dev);
+       ret = fimd_ctx_initialize(ctx, drm_dev);
+       if (ret) {
+               DRM_ERROR("fimd_ctx_initialize failed.\n");
+               return ret;
+       }
+
 
        if (ctx->display)
                exynos_drm_create_enc_conn(drm_dev, ctx->display);