drm/sti: do not set gdp pixel clock rate if mode is not set
authorVincent Abriou <vincent.abriou@st.com>
Thu, 2 Feb 2017 08:49:02 +0000 (09:49 +0100)
committerVincent Abriou <vincent.abriou@st.com>
Fri, 10 Feb 2017 08:34:39 +0000 (09:34 +0100)
Fix a division by 0 case : in some cases, when the GDP plane is being
disabled atomic_check() is called with "mode->clock = 0".
In that case, do not set parent and pixel clock rate.

Signed-off-by: Vincent Abriou <vincent.abriou@st.com>
drivers/gpu/drm/sti/sti_gdp.c

index 7255234670c019874f89aa6ba41bee43c7e46686..7f6d0790cca6d9064d9484848309aec5c8e60b4d 100644 (file)
@@ -648,7 +648,7 @@ static int sti_gdp_atomic_check(struct drm_plane *drm_plane,
        }
 
        /* Set gdp clock */
-       if (gdp->clk_pix) {
+       if (mode->clock && gdp->clk_pix) {
                struct clk *clkp;
                int rate = mode->clock * 1000;
                int res;