drm/i915/skl: Propagate the error if we fail to find a suitable DPLL divider
authorDamien Lespiau <damien.lespiau@intel.com>
Thu, 7 May 2015 17:38:40 +0000 (18:38 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 29 May 2015 08:15:23 +0000 (10:15 +0200)
At the moment, even if we fail to find a suitable divider, we'll still
try to set the mode with bogus parameters.

Just fail the modeset if we can't generate the frequency.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_ddi.c

index c1abe6efff9d96b2d554c69795a15e1a21e7bf0a..f10f55690749c06b9f05b3f87d58b5199ac5f733 100644 (file)
@@ -1115,7 +1115,7 @@ struct skl_wrpll_params {
        uint32_t        central_freq;
 };
 
-static void
+static bool
 skl_ddi_calculate_wrpll(int clock /* in Hz */,
                        struct skl_wrpll_params *wrpll_params)
 {
@@ -1196,6 +1196,7 @@ found:
        if (min_dco_index > 2) {
                WARN(1, "No valid parameters found for pixel clock: %dHz\n",
                     clock);
+               return false;
        } else {
                wrpll_params->central_freq = dco_central_freq[min_dco_index];
 
@@ -1262,6 +1263,8 @@ found:
                                  wrpll_params->dco_integer * MHz(1)) * 0x8000), MHz(1));
 
        }
+
+       return true;
 }
 
 
@@ -1286,7 +1289,8 @@ skl_ddi_pll_select(struct intel_crtc *intel_crtc,
 
                ctrl1 |= DPLL_CTRL1_HDMI_MODE(0);
 
-               skl_ddi_calculate_wrpll(clock * 1000, &wrpll_params);
+               if (!skl_ddi_calculate_wrpll(clock * 1000, &wrpll_params))
+                       return false;
 
                cfgcr1 = DPLL_CFGCR1_FREQ_ENABLE |
                         DPLL_CFGCR1_DCO_FRACTION(wrpll_params.dco_fraction) |