drm/exynos: fimd: move platform data parsing to separate function
authorAndrzej Hajda <a.hajda@samsung.com>
Wed, 21 Aug 2013 14:22:03 +0000 (16:22 +0200)
committerInki Dae <inki.dae@samsung.com>
Thu, 5 Sep 2013 04:43:44 +0000 (13:43 +0900)
The patch moves platfrom_data and device tree parsing
to separate function.

Signed-off-by: Andrzej Hajda <a.hajda@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_fimd.c

index 6afcaf1645490c2d0abec5a7b079e3137f1b717f..130dea50c3e1457bee9981dea4d943206db0b69e 100644 (file)
@@ -122,7 +122,7 @@ struct fimd_context {
        wait_queue_head_t               wait_vsync_queue;
        atomic_t                        wait_vsync_event;
 
-       struct exynos_drm_panel_info *panel;
+       struct exynos_drm_panel_info panel;
        struct fimd_driver_data *driver_data;
 };
 
@@ -164,7 +164,7 @@ static void *fimd_get_panel(struct device *dev)
 {
        struct fimd_context *ctx = get_fimd_context(dev);
 
-       return ctx->panel;
+       return &ctx->panel;
 }
 
 static int fimd_check_mode(struct device *dev, struct drm_display_mode *mode)
@@ -244,7 +244,7 @@ static void fimd_apply(struct device *subdrv_dev)
 static void fimd_commit(struct device *dev)
 {
        struct fimd_context *ctx = get_fimd_context(dev);
-       struct exynos_drm_panel_info *panel = ctx->panel;
+       struct exynos_drm_panel_info *panel = &ctx->panel;
        struct videomode *vm = &panel->vm;
        struct fimd_driver_data *driver_data;
        u32 val;
@@ -755,7 +755,7 @@ static void fimd_subdrv_remove(struct drm_device *drm_dev, struct device *dev)
 
 static int fimd_configure_clocks(struct fimd_context *ctx, struct device *dev)
 {
-       struct videomode *vm = &ctx->panel->vm;
+       struct videomode *vm = &ctx->panel.vm;
        unsigned long clk;
 
        ctx->bus_clk = devm_clk_get(dev, "fimd");
@@ -892,24 +892,13 @@ static int fimd_activate(struct fimd_context *ctx, bool enable)
        return 0;
 }
 
-static int fimd_probe(struct platform_device *pdev)
+static int fimd_get_platform_data(struct fimd_context *ctx, struct device *dev)
 {
-       struct device *dev = &pdev->dev;
-       struct fimd_context *ctx;
-       struct exynos_drm_subdrv *subdrv;
-       struct exynos_drm_fimd_pdata *pdata;
-       struct exynos_drm_panel_info *panel;
-       struct resource *res;
-       int win;
-       int ret = -EINVAL;
-
        if (dev->of_node) {
                struct videomode *vm;
-               pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL);
-               if (!pdata)
-                       return -ENOMEM;
+               int ret;
 
-               vm = &pdata->panel.vm;
+               vm = &ctx->panel.vm;
                ret = of_get_videomode(dev->of_node, vm, OF_USE_NATIVE_MODE);
                if (ret) {
                        DRM_ERROR("failed: of_get_videomode() : %d\n", ret);
@@ -917,31 +906,45 @@ static int fimd_probe(struct platform_device *pdev)
                }
 
                if (vm->flags & DISPLAY_FLAGS_VSYNC_LOW)
-                       pdata->vidcon1 |= VIDCON1_INV_VSYNC;
+                       ctx->vidcon1 |= VIDCON1_INV_VSYNC;
                if (vm->flags & DISPLAY_FLAGS_HSYNC_LOW)
-                       pdata->vidcon1 |= VIDCON1_INV_HSYNC;
+                       ctx->vidcon1 |= VIDCON1_INV_HSYNC;
                if (vm->flags & DISPLAY_FLAGS_DE_LOW)
-                       pdata->vidcon1 |= VIDCON1_INV_VDEN;
+                       ctx->vidcon1 |= VIDCON1_INV_VDEN;
                if (vm->flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE)
-                       pdata->vidcon1 |= VIDCON1_INV_VCLK;
+                       ctx->vidcon1 |= VIDCON1_INV_VCLK;
        } else {
-               pdata = dev->platform_data;
+               struct exynos_drm_fimd_pdata *pdata = dev->platform_data;
                if (!pdata) {
                        DRM_ERROR("no platform data specified\n");
                        return -EINVAL;
                }
+               ctx->vidcon0 = pdata->vidcon0;
+               ctx->vidcon1 = pdata->vidcon1;
+               ctx->default_win = pdata->default_win;
+               ctx->panel = pdata->panel;
        }
 
-       panel = &pdata->panel;
-       if (!panel) {
-               dev_err(dev, "panel is null.\n");
-               return -EINVAL;
-       }
+       return 0;
+}
+
+static int fimd_probe(struct platform_device *pdev)
+{
+       struct device *dev = &pdev->dev;
+       struct fimd_context *ctx;
+       struct exynos_drm_subdrv *subdrv;
+       struct resource *res;
+       int win;
+       int ret = -EINVAL;
 
        ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
        if (!ctx)
                return -ENOMEM;
 
+       ret = fimd_get_platform_data(ctx, dev);
+       if (ret)
+               return ret;
+
        ret = fimd_configure_clocks(ctx, dev);
        if (ret)
                return ret;
@@ -968,10 +971,6 @@ static int fimd_probe(struct platform_device *pdev)
        }
 
        ctx->driver_data = drm_fimd_get_driver_data(pdev);
-       ctx->vidcon0 = pdata->vidcon0;
-       ctx->vidcon1 = pdata->vidcon1;
-       ctx->default_win = pdata->default_win;
-       ctx->panel = panel;
        DRM_INIT_WAITQUEUE(&ctx->wait_vsync_queue);
        atomic_set(&ctx->wait_vsync_event, 0);