drm: exynos: hdmi: remove drm common hdmi platform data struct
authorRahul Sharma <rahul.sharma@samsung.com>
Thu, 4 Oct 2012 15:18:56 +0000 (20:48 +0530)
committerInki Dae <inki.dae@samsung.com>
Fri, 5 Oct 2012 10:15:16 +0000 (19:15 +0900)
exynos-drm-hdmi need context pointers from hdmi and mixer. These
pointers were expected from the plf data. Cleaned this dependency
by exporting i/f which are called by hdmi, mixer driver probes
for setting their context.

Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
drivers/gpu/drm/exynos/exynos_drm_hdmi.c
drivers/gpu/drm/exynos/exynos_drm_hdmi.h
drivers/gpu/drm/exynos/exynos_hdmi.c
drivers/gpu/drm/exynos/exynos_mixer.c
include/drm/exynos_drm.h

index 0584132dc60868470d94634de11cc0832916a8d8..85304c46a8cd1468f9614aef069bd60b3027f73c 100644 (file)
 #define get_ctx_from_subdrv(subdrv)    container_of(subdrv,\
                                        struct drm_hdmi_context, subdrv);
 
+/* Common hdmi subdrv needs to access the hdmi and mixer though context.
+* These should be initialied by the repective drivers */
+static struct exynos_drm_hdmi_context *hdmi_ctx;
+static struct exynos_drm_hdmi_context *mixer_ctx;
+
 /* these callback points shoud be set by specific drivers. */
 static struct exynos_hdmi_ops *hdmi_ops;
 static struct exynos_mixer_ops *mixer_ops;
@@ -41,6 +46,18 @@ struct drm_hdmi_context {
        bool    enabled[MIXER_WIN_NR];
 };
 
+void exynos_hdmi_drv_attach(struct exynos_drm_hdmi_context *ctx)
+{
+       if (ctx)
+               hdmi_ctx = ctx;
+}
+
+void exynos_mixer_drv_attach(struct exynos_drm_hdmi_context *ctx)
+{
+       if (ctx)
+               mixer_ctx = ctx;
+}
+
 void exynos_hdmi_ops_register(struct exynos_hdmi_ops *ops)
 {
        DRM_DEBUG_KMS("%s\n", __FILE__);
@@ -303,46 +320,30 @@ static int hdmi_subdrv_probe(struct drm_device *drm_dev,
 {
        struct exynos_drm_subdrv *subdrv = to_subdrv(dev);
        struct drm_hdmi_context *ctx;
-       struct platform_device *pdev = to_platform_device(dev);
-       struct exynos_drm_common_hdmi_pd *pd;
 
        DRM_DEBUG_KMS("%s\n", __FILE__);
 
-       pd = pdev->dev.platform_data;
-
-       if (!pd) {
-               DRM_DEBUG_KMS("platform data is null.\n");
+       if (!hdmi_ctx) {
+               DRM_ERROR("hdmi context not initialized.\n");
                return -EFAULT;
        }
 
-       if (!pd->hdmi_dev) {
-               DRM_DEBUG_KMS("hdmi device is null.\n");
-               return -EFAULT;
-       }
-
-       if (!pd->mixer_dev) {
-               DRM_DEBUG_KMS("mixer device is null.\n");
+       if (!mixer_ctx) {
+               DRM_ERROR("mixer context not initialized.\n");
                return -EFAULT;
        }
 
        ctx = get_ctx_from_subdrv(subdrv);
 
-       ctx->hdmi_ctx = (struct exynos_drm_hdmi_context *)
-                               to_context(pd->hdmi_dev);
-       if (!ctx->hdmi_ctx) {
-               DRM_DEBUG_KMS("hdmi context is null.\n");
+       if (!ctx) {
+               DRM_ERROR("no drm hdmi context.\n");
                return -EFAULT;
        }
 
-       ctx->hdmi_ctx->drm_dev = drm_dev;
-
-       ctx->mixer_ctx = (struct exynos_drm_hdmi_context *)
-                               to_context(pd->mixer_dev);
-       if (!ctx->mixer_ctx) {
-               DRM_DEBUG_KMS("mixer context is null.\n");
-               return -EFAULT;
-       }
+       ctx->hdmi_ctx = hdmi_ctx;
+       ctx->mixer_ctx = mixer_ctx;
 
+       ctx->hdmi_ctx->drm_dev = drm_dev;
        ctx->mixer_ctx->drm_dev = drm_dev;
 
        return 0;
index d9f9e9fcc2b667d943d1ac861e3190a27e349d51..2da5ffd3a0591382ae5d9d62d3f01e38734d092f 100644 (file)
@@ -73,6 +73,8 @@ struct exynos_mixer_ops {
        void (*win_disable)(void *ctx, int zpos);
 };
 
+void exynos_hdmi_drv_attach(struct exynos_drm_hdmi_context *ctx);
+void exynos_mixer_drv_attach(struct exynos_drm_hdmi_context *ctx);
 void exynos_hdmi_ops_register(struct exynos_hdmi_ops *ops);
 void exynos_mixer_ops_register(struct exynos_mixer_ops *ops);
 #endif
index 5caf49f366e3e4d51445e660a3d78315c0936182..e6b784d1527dbfdf93b0d4acf9d932daebe04fad 100644 (file)
@@ -2454,6 +2454,9 @@ static int __devinit hdmi_probe(struct platform_device *pdev)
                goto err_free_irq;
        }
 
+       /* Attach HDMI Driver to common hdmi. */
+       exynos_hdmi_drv_attach(drm_hdmi_ctx);
+
        /* register specific callbacks to common hdmi. */
        exynos_hdmi_ops_register(&hdmi_ops);
 
index 39d2b95087aeffdf3b73b4ee81df47b760913ed4..1a319e4431d803920c35b1715c48747e51468a31 100644 (file)
@@ -1171,6 +1171,9 @@ static int __devinit mixer_probe(struct platform_device *pdev)
                }
        }
 
+       /* attach mixer driver to common hdmi. */
+       exynos_mixer_drv_attach(drm_hdmi_ctx);
+
        /* register specific callback point to common hdmi. */
        exynos_mixer_ops_register(&mixer_ops);
 
index 8bdd49a0abd0dbc2e4352fd8e740c46969bc3169..8ac4079e6458e4b1afd6a7df06edf7c22502b0d5 100644 (file)
@@ -240,19 +240,5 @@ struct exynos_drm_fimd_pdata {
        unsigned int                    bpp;
 };
 
-/**
- * Platform Specific Structure for DRM based HDMI.
- *
- * @hdmi_dev: device point to specific hdmi driver.
- * @mixer_dev: device point to specific mixer driver.
- *
- * this structure is used for common hdmi driver and each device object
- * would be used to access specific device driver(hdmi or mixer driver)
- */
-struct exynos_drm_common_hdmi_pd {
-       struct device *hdmi_dev;
-       struct device *mixer_dev;
-};
-
 #endif /* __KERNEL__ */
 #endif /* _EXYNOS_DRM_H_ */