drm/i915: Add debugs for the C8 vs. legacy LUT case
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 13 May 2019 13:39:04 +0000 (16:39 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 28 May 2019 17:46:55 +0000 (20:46 +0300)
Leave a hint in dmesg when we reject a configuration attempting
to use C8 planes without the legacy LUT loaded.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190513133904.20374-3-ville.syrjala@linux.intel.com
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
drivers/gpu/drm/i915/intel_color.c

index 962db12..4564990 100644 (file)
@@ -959,8 +959,10 @@ static int check_luts(const struct intel_crtc_state *crtc_state)
                return 0;
 
        /* C8 relies on its palette being stored in the legacy LUT */
-       if (crtc_state->c8_planes)
+       if (crtc_state->c8_planes) {
+               DRM_DEBUG_KMS("C8 pixelformat requires the legacy LUT\n");
                return -EINVAL;
+       }
 
        degamma_length = INTEL_INFO(dev_priv)->color.degamma_lut_size;
        gamma_length = INTEL_INFO(dev_priv)->color.gamma_lut_size;