drm/i915: Fix legacy gamma mode for ICL
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 15 Mar 2019 19:54:44 +0000 (21:54 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 18 Mar 2019 15:47:03 +0000 (17:47 +0200)
We must remember to actually enable the post CSC gamma if
we expect the legacy LUT to work. Seems to fix NV12 crc
tests on the SDR planes. Curiously we apparently managed to
get 100% match for the HDR planes even without chopping
off the low bits.

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

index 8038f0f..467fd1a 100644 (file)
@@ -832,6 +832,9 @@ int intel_color_check(struct intel_crtc_state *crtc_state)
        if (!crtc_state->gamma_enable ||
            crtc_state_is_legacy_gamma(crtc_state)) {
                crtc_state->gamma_mode = GAMMA_MODE_MODE_8BIT;
+               if (INTEL_GEN(dev_priv) >= 11 &&
+                   crtc_state->gamma_enable)
+                       crtc_state->gamma_mode |= POST_CSC_GAMMA_ENABLE;
                return 0;
        }