drm/i915: HDMI pixel clock check
authorMika Kahola <mika.kahola@intel.com>
Tue, 2 Feb 2016 13:16:39 +0000 (15:16 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 11 Feb 2016 09:15:31 +0000 (10:15 +0100)
It is possible the we request to have a mode that has
higher pixel clock than our HW can support. This patch
checks if requested pixel clock is lower than the one
supported by the HW. The requested mode is discarded
if we cannot support the requested pixel clock.

This patch applies to HDMI.

V2:
- removed computation for max dot clock

V3:
- cleanup by removing unnecessary lines

V4:
- max_pixclk variable renamed as max_dotclk
- check for stereo mode added

Signed-off-by: Mika Kahola <mika.kahola@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1454419003-6001-3-git-send-email-mika.kahola@intel.com
drivers/gpu/drm/i915/intel_hdmi.c

index 8698a643d027f70f5dd9a212c8b13eaa8f220e0e..edb7e901ba4abfb809f363cf1351505002fc05e4 100644 (file)
@@ -1202,11 +1202,19 @@ intel_hdmi_mode_valid(struct drm_connector *connector,
        struct drm_device *dev = intel_hdmi_to_dev(hdmi);
        enum drm_mode_status status;
        int clock;
+       int max_dotclk = to_i915(connector->dev)->max_dotclk_freq;
 
        if (mode->flags & DRM_MODE_FLAG_DBLSCAN)
                return MODE_NO_DBLESCAN;
 
        clock = mode->clock;
+
+       if ((mode->flags & DRM_MODE_FLAG_3D_MASK) == DRM_MODE_FLAG_3D_FRAME_PACKING)
+               clock *= 2;
+
+       if (clock > max_dotclk)
+               return MODE_CLOCK_HIGH;
+
        if (mode->flags & DRM_MODE_FLAG_DBLCLK)
                clock *= 2;