drm/omap: remove divider constraint from hsdiv
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 8 Aug 2016 10:40:35 +0000 (13:40 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 3 Apr 2017 09:36:40 +0000 (12:36 +0300)
The driver only uses even dividers for hsdiv when pclk >= 100MHz, as odd
dividers can create uneven duty cycle. However, while this holds true
for some dividers like DISPC's LCK and PCK dividers, it is not actually
true for hsdiv.

hsdiv always produces even duty cycle, so the constraint can be removed.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drivers/gpu/drm/omapdrm/dss/dpi.c

index e75162d26ac04a5f064d564b5f78d57044fd077e..e0b0c5c24c55f86e5c7297c001e14eee89db895c 100644 (file)
@@ -170,14 +170,6 @@ static bool dpi_calc_hsdiv_cb(int m_dispc, unsigned long dispc,
 {
        struct dpi_clk_calc_ctx *ctx = data;
 
-       /*
-        * Odd dividers give us uneven duty cycle, causing problem when level
-        * shifted. So skip all odd dividers when the pixel clock is on the
-        * higher side.
-        */
-       if (m_dispc > 1 && m_dispc % 2 != 0 && ctx->pck_min >= 100000000)
-               return false;
-
        ctx->pll_cinfo.mX[ctx->clkout_idx] = m_dispc;
        ctx->pll_cinfo.clkout[ctx->clkout_idx] = dispc;