To avoid duplicate logic for the same

Signed-off-by: Satendra Singh Thakur <satendr...@samsung.com>
Cc: Madhur Verma <madhur.ve...@samsung.com>
Cc: Hemanshu Srivastava <hemansh...@samsung.com>
---
 drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c | 29 ++++++++++++-----------------
 1 file changed, 12 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c 
b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c
index 0e37524..3c651f8 100644
--- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c
+++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c
@@ -17,6 +17,7 @@
 #include <drm/drm_atomic_helper.h>
 #include <drm/drm_crtc.h>
 #include <drm/drm_crtc_helper.h>
+#include <video/videomode.h>
 
 #include "fsl_dcu_drm_crtc.h"
 #include "fsl_dcu_drm_drv.h"
@@ -85,19 +86,13 @@ static void fsl_dcu_drm_crtc_mode_set_nofb(struct drm_crtc 
*crtc)
        struct fsl_dcu_drm_device *fsl_dev = dev->dev_private;
        struct drm_connector *con = &fsl_dev->connector.base;
        struct drm_display_mode *mode = &crtc->state->mode;
-       unsigned int hbp, hfp, hsw, vbp, vfp, vsw, index, pol = 0;
+       unsigned int  index, pol = 0;
+       struct videomode vm;
 
        index = drm_crtc_index(crtc);
-       clk_set_rate(fsl_dev->pix_clk, mode->clock * 1000);
-
-       /* Configure timings: */
-       hbp = mode->htotal - mode->hsync_end;
-       hfp = mode->hsync_start - mode->hdisplay;
-       hsw = mode->hsync_end - mode->hsync_start;
-       vbp = mode->vtotal - mode->vsync_end;
-       vfp = mode->vsync_start - mode->vdisplay;
-       vsw = mode->vsync_end - mode->vsync_start;
-
+       /* Configure timings:*/
+       drm_display_mode_to_videomode(mode, &vm);
+       clk_set_rate(fsl_dev->pix_clk, vm.pixelclock);
        /* INV_PXCK as default (most display sample data on rising edge) */
        if (!(con->display_info.bus_flags & DRM_BUS_FLAG_PIXDATA_POSEDGE))
                pol |= DCU_SYN_POL_INV_PXCK;
@@ -109,13 +104,13 @@ static void fsl_dcu_drm_crtc_mode_set_nofb(struct 
drm_crtc *crtc)
                pol |= DCU_SYN_POL_INV_VS_LOW;
 
        regmap_write(fsl_dev->regmap, DCU_HSYN_PARA,
-                    DCU_HSYN_PARA_BP(hbp) |
-                    DCU_HSYN_PARA_PW(hsw) |
-                    DCU_HSYN_PARA_FP(hfp));
+                    DCU_HSYN_PARA_BP(vm.hback_porch) |
+                    DCU_HSYN_PARA_PW(vm.hsync_len) |
+                    DCU_HSYN_PARA_FP(vm.hfront_porch));
        regmap_write(fsl_dev->regmap, DCU_VSYN_PARA,
-                    DCU_VSYN_PARA_BP(vbp) |
-                    DCU_VSYN_PARA_PW(vsw) |
-                    DCU_VSYN_PARA_FP(vfp));
+                    DCU_VSYN_PARA_BP(vm.vback_porch) |
+                    DCU_VSYN_PARA_PW(vm.vsync_len) |
+                    DCU_VSYN_PARA_FP(vm.vfront_porch));
        regmap_write(fsl_dev->regmap, DCU_DISP_SIZE,
                     DCU_DISP_SIZE_DELTA_Y(mode->vdisplay) |
                     DCU_DISP_SIZE_DELTA_X(mode->hdisplay));
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to