Merge tag 'topic/drm-misc-2015-02-06' of git://anongit.freedesktop.org/drm-intel...
[linux-2.6-block.git] / drivers / gpu / drm / exynos / exynos_mixer.c
index 5da28443342df272dd35d28cd2967dbb894be73c..2fd2e5d46142e0b9b41e0389866f1bff542039cb 100644 (file)
@@ -84,10 +84,10 @@ enum mixer_version_id {
 };
 
 struct mixer_context {
-       struct exynos_drm_manager manager;
        struct platform_device *pdev;
        struct device           *dev;
        struct drm_device       *drm_dev;
+       struct exynos_drm_crtc  *crtc;
        int                     pipe;
        bool                    interlace;
        bool                    powered;
@@ -103,11 +103,6 @@ struct mixer_context {
        atomic_t                wait_vsync_event;
 };
 
-static inline struct mixer_context *mgr_to_mixer(struct exynos_drm_manager *mgr)
-{
-       return container_of(mgr, struct mixer_context, manager);
-}
-
 struct mixer_drv_data {
        enum mixer_version_id   version;
        bool                                    is_vp_enabled;
@@ -852,16 +847,15 @@ static int vp_resources_init(struct mixer_context *mixer_ctx)
        return 0;
 }
 
-static int mixer_initialize(struct exynos_drm_manager *mgr,
+static int mixer_initialize(struct mixer_context *mixer_ctx,
                        struct drm_device *drm_dev)
 {
        int ret;
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
        struct exynos_drm_private *priv;
        priv = drm_dev->dev_private;
 
-       mgr->drm_dev = mixer_ctx->drm_dev = drm_dev;
-       mgr->pipe = mixer_ctx->pipe = priv->pipe++;
+       mixer_ctx->drm_dev = drm_dev;
+       mixer_ctx->pipe = priv->pipe++;
 
        /* acquire resources: regs, irqs, clocks */
        ret = mixer_resources_init(mixer_ctx);
@@ -885,17 +879,15 @@ static int mixer_initialize(struct exynos_drm_manager *mgr,
        return drm_iommu_attach_device(mixer_ctx->drm_dev, mixer_ctx->dev);
 }
 
-static void mixer_mgr_remove(struct exynos_drm_manager *mgr)
+static void mixer_ctx_remove(struct mixer_context *mixer_ctx)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
-
        if (is_drm_iommu_supported(mixer_ctx->drm_dev))
                drm_iommu_detach_device(mixer_ctx->drm_dev, mixer_ctx->dev);
 }
 
-static int mixer_enable_vblank(struct exynos_drm_manager *mgr)
+static int mixer_enable_vblank(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
        struct mixer_resources *res = &mixer_ctx->mixer_res;
 
        if (!mixer_ctx->powered) {
@@ -910,34 +902,34 @@ static int mixer_enable_vblank(struct exynos_drm_manager *mgr)
        return 0;
 }
 
-static void mixer_disable_vblank(struct exynos_drm_manager *mgr)
+static void mixer_disable_vblank(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
        struct mixer_resources *res = &mixer_ctx->mixer_res;
 
        /* disable vsync interrupt */
        mixer_reg_writemask(res, MXR_INT_EN, 0, MXR_INT_EN_VSYNC);
 }
 
-static void mixer_win_mode_set(struct exynos_drm_manager *mgr,
-                       struct exynos_drm_overlay *overlay)
+static void mixer_win_mode_set(struct exynos_drm_crtc *crtc,
+                       struct exynos_drm_plane *plane)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
        struct hdmi_win_data *win_data;
        int win;
 
-       if (!overlay) {
-               DRM_ERROR("overlay is NULL\n");
+       if (!plane) {
+               DRM_ERROR("plane is NULL\n");
                return;
        }
 
        DRM_DEBUG_KMS("set [%d]x[%d] at (%d,%d) to [%d]x[%d] at (%d,%d)\n",
-                                overlay->fb_width, overlay->fb_height,
-                                overlay->fb_x, overlay->fb_y,
-                                overlay->crtc_width, overlay->crtc_height,
-                                overlay->crtc_x, overlay->crtc_y);
+                                plane->fb_width, plane->fb_height,
+                                plane->fb_x, plane->fb_y,
+                                plane->crtc_width, plane->crtc_height,
+                                plane->crtc_x, plane->crtc_y);
 
-       win = overlay->zpos;
+       win = plane->zpos;
        if (win == DEFAULT_ZPOS)
                win = MIXER_DEFAULT_WIN;
 
@@ -948,32 +940,32 @@ static void mixer_win_mode_set(struct exynos_drm_manager *mgr,
 
        win_data = &mixer_ctx->win_data[win];
 
-       win_data->dma_addr = overlay->dma_addr[0];
-       win_data->chroma_dma_addr = overlay->dma_addr[1];
-       win_data->pixel_format = overlay->pixel_format;
-       win_data->bpp = overlay->bpp;
+       win_data->dma_addr = plane->dma_addr[0];
+       win_data->chroma_dma_addr = plane->dma_addr[1];
+       win_data->pixel_format = plane->pixel_format;
+       win_data->bpp = plane->bpp;
 
-       win_data->crtc_x = overlay->crtc_x;
-       win_data->crtc_y = overlay->crtc_y;
-       win_data->crtc_width = overlay->crtc_width;
-       win_data->crtc_height = overlay->crtc_height;
+       win_data->crtc_x = plane->crtc_x;
+       win_data->crtc_y = plane->crtc_y;
+       win_data->crtc_width = plane->crtc_width;
+       win_data->crtc_height = plane->crtc_height;
 
-       win_data->fb_x = overlay->fb_x;
-       win_data->fb_y = overlay->fb_y;
-       win_data->fb_width = overlay->fb_width;
-       win_data->fb_height = overlay->fb_height;
-       win_data->src_width = overlay->src_width;
-       win_data->src_height = overlay->src_height;
+       win_data->fb_x = plane->fb_x;
+       win_data->fb_y = plane->fb_y;
+       win_data->fb_width = plane->fb_width;
+       win_data->fb_height = plane->fb_height;
+       win_data->src_width = plane->src_width;
+       win_data->src_height = plane->src_height;
 
-       win_data->mode_width = overlay->mode_width;
-       win_data->mode_height = overlay->mode_height;
+       win_data->mode_width = plane->mode_width;
+       win_data->mode_height = plane->mode_height;
 
-       win_data->scan_flags = overlay->scan_flag;
+       win_data->scan_flags = plane->scan_flag;
 }
 
-static void mixer_win_commit(struct exynos_drm_manager *mgr, int zpos)
+static void mixer_win_commit(struct exynos_drm_crtc *crtc, int zpos)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
        int win = zpos == DEFAULT_ZPOS ? MIXER_DEFAULT_WIN : zpos;
 
        DRM_DEBUG_KMS("win: %d\n", win);
@@ -993,9 +985,9 @@ static void mixer_win_commit(struct exynos_drm_manager *mgr, int zpos)
        mixer_ctx->win_data[win].enabled = true;
 }
 
-static void mixer_win_disable(struct exynos_drm_manager *mgr, int zpos)
+static void mixer_win_disable(struct exynos_drm_crtc *crtc, int zpos)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
        struct mixer_resources *res = &mixer_ctx->mixer_res;
        int win = zpos == DEFAULT_ZPOS ? MIXER_DEFAULT_WIN : zpos;
        unsigned long flags;
@@ -1021,9 +1013,10 @@ static void mixer_win_disable(struct exynos_drm_manager *mgr, int zpos)
        mixer_ctx->win_data[win].enabled = false;
 }
 
-static void mixer_wait_for_vblank(struct exynos_drm_manager *mgr)
+static void mixer_wait_for_vblank(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *mixer_ctx = mgr_to_mixer(mgr);
+       struct mixer_context *mixer_ctx = crtc->ctx;
+       int err;
 
        mutex_lock(&mixer_ctx->mixer_mutex);
        if (!mixer_ctx->powered) {
@@ -1032,7 +1025,11 @@ static void mixer_wait_for_vblank(struct exynos_drm_manager *mgr)
        }
        mutex_unlock(&mixer_ctx->mixer_mutex);
 
-       drm_vblank_get(mgr->crtc->dev, mixer_ctx->pipe);
+       err = drm_vblank_get(mixer_ctx->drm_dev, mixer_ctx->pipe);
+       if (err < 0) {
+               DRM_DEBUG_KMS("failed to acquire vblank counter\n");
+               return;
+       }
 
        atomic_set(&mixer_ctx->wait_vsync_event, 1);
 
@@ -1045,26 +1042,26 @@ static void mixer_wait_for_vblank(struct exynos_drm_manager *mgr)
                                HZ/20))
                DRM_DEBUG_KMS("vblank wait timed out.\n");
 
-       drm_vblank_put(mgr->crtc->dev, mixer_ctx->pipe);
+       drm_vblank_put(mixer_ctx->drm_dev, mixer_ctx->pipe);
 }
 
-static void mixer_window_suspend(struct exynos_drm_manager *mgr)
+static void mixer_window_suspend(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *ctx = mgr_to_mixer(mgr);
+       struct mixer_context *ctx = crtc->ctx;
        struct hdmi_win_data *win_data;
        int i;
 
        for (i = 0; i < MIXER_WIN_NR; i++) {
                win_data = &ctx->win_data[i];
                win_data->resume = win_data->enabled;
-               mixer_win_disable(mgr, i);
+               mixer_win_disable(crtc, i);
        }
-       mixer_wait_for_vblank(mgr);
+       mixer_wait_for_vblank(crtc);
 }
 
-static void mixer_window_resume(struct exynos_drm_manager *mgr)
+static void mixer_window_resume(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *ctx = mgr_to_mixer(mgr);
+       struct mixer_context *ctx = crtc->ctx;
        struct hdmi_win_data *win_data;
        int i;
 
@@ -1073,13 +1070,13 @@ static void mixer_window_resume(struct exynos_drm_manager *mgr)
                win_data->enabled = win_data->resume;
                win_data->resume = false;
                if (win_data->enabled)
-                       mixer_win_commit(mgr, i);
+                       mixer_win_commit(crtc, i);
        }
 }
 
-static void mixer_poweron(struct exynos_drm_manager *mgr)
+static void mixer_poweron(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *ctx = mgr_to_mixer(mgr);
+       struct mixer_context *ctx = crtc->ctx;
        struct mixer_resources *res = &ctx->mixer_res;
 
        mutex_lock(&ctx->mixer_mutex);
@@ -1108,12 +1105,12 @@ static void mixer_poweron(struct exynos_drm_manager *mgr)
        mixer_reg_write(res, MXR_INT_EN, ctx->int_en);
        mixer_win_reset(ctx);
 
-       mixer_window_resume(mgr);
+       mixer_window_resume(crtc);
 }
 
-static void mixer_poweroff(struct exynos_drm_manager *mgr)
+static void mixer_poweroff(struct exynos_drm_crtc *crtc)
 {
-       struct mixer_context *ctx = mgr_to_mixer(mgr);
+       struct mixer_context *ctx = crtc->ctx;
        struct mixer_resources *res = &ctx->mixer_res;
 
        mutex_lock(&ctx->mixer_mutex);
@@ -1124,7 +1121,7 @@ static void mixer_poweroff(struct exynos_drm_manager *mgr)
        mutex_unlock(&ctx->mixer_mutex);
 
        mixer_stop(ctx);
-       mixer_window_suspend(mgr);
+       mixer_window_suspend(crtc);
 
        ctx->int_en = mixer_reg_read(res, MXR_INT_EN);
 
@@ -1142,16 +1139,16 @@ static void mixer_poweroff(struct exynos_drm_manager *mgr)
        pm_runtime_put_sync(ctx->dev);
 }
 
-static void mixer_dpms(struct exynos_drm_manager *mgr, int mode)
+static void mixer_dpms(struct exynos_drm_crtc *crtc, int mode)
 {
        switch (mode) {
        case DRM_MODE_DPMS_ON:
-               mixer_poweron(mgr);
+               mixer_poweron(crtc);
                break;
        case DRM_MODE_DPMS_STANDBY:
        case DRM_MODE_DPMS_SUSPEND:
        case DRM_MODE_DPMS_OFF:
-               mixer_poweroff(mgr);
+               mixer_poweroff(crtc);
                break;
        default:
                DRM_DEBUG_KMS("unknown dpms mode: %d\n", mode);
@@ -1179,7 +1176,7 @@ int mixer_check_mode(struct drm_display_mode *mode)
        return -EINVAL;
 }
 
-static struct exynos_drm_manager_ops mixer_manager_ops = {
+static struct exynos_drm_crtc_ops mixer_crtc_ops = {
        .dpms                   = mixer_dpms,
        .enable_vblank          = mixer_enable_vblank,
        .disable_vblank         = mixer_disable_vblank,
@@ -1250,28 +1247,30 @@ static int mixer_bind(struct device *dev, struct device *manager, void *data)
        struct drm_device *drm_dev = data;
        int ret;
 
-       ret = mixer_initialize(&ctx->manager, drm_dev);
-       if (ret)
-               return ret;
-
-       ret = exynos_drm_crtc_create(&ctx->manager);
-       if (ret) {
-               mixer_mgr_remove(&ctx->manager);
-               return ret;
+       ctx->crtc = exynos_drm_crtc_create(drm_dev, ctx->pipe,
+                                    EXYNOS_DISPLAY_TYPE_HDMI,
+                                    &mixer_crtc_ops, ctx);
+       if (IS_ERR(ctx->crtc)) {
+               ret = PTR_ERR(ctx->crtc);
+               goto free_ctx;
        }
 
-       pm_runtime_enable(dev);
+       ret = mixer_initialize(ctx, drm_dev);
+       if (ret)
+               goto free_ctx;
 
        return 0;
+
+free_ctx:
+       devm_kfree(dev, ctx);
+       return ret;
 }
 
 static void mixer_unbind(struct device *dev, struct device *master, void *data)
 {
        struct mixer_context *ctx = dev_get_drvdata(dev);
 
-       mixer_mgr_remove(&ctx->manager);
-
-       pm_runtime_disable(dev);
+       mixer_ctx_remove(ctx);
 }
 
 static const struct component_ops mixer_component_ops = {
@@ -1294,9 +1293,6 @@ static int mixer_probe(struct platform_device *pdev)
 
        mutex_init(&ctx->mixer_mutex);
 
-       ctx->manager.type = EXYNOS_DISPLAY_TYPE_HDMI;
-       ctx->manager.ops = &mixer_manager_ops;
-
        if (dev->of_node) {
                const struct of_device_id *match;
 
@@ -1318,7 +1314,7 @@ static int mixer_probe(struct platform_device *pdev)
        platform_set_drvdata(pdev, ctx);
 
        ret = exynos_drm_component_add(&pdev->dev, EXYNOS_DEVICE_TYPE_CRTC,
-                                       ctx->manager.type);
+                                       EXYNOS_DISPLAY_TYPE_HDMI);
        if (ret)
                return ret;