On Mon, Feb 22, 2016 at 02:18:07PM +0000, Lionel Landwerlin wrote:
> The moves a couple of functions programming the gamma LUT and CSC
> units into their own file.
> 
> On generations prior to Haswell there is only a gamma LUT. From
> haswell on there is also a new enhanced color correction unit that
> isn't used yet. This is why we need to set the GAMMA_MODE register,
> either we're using the legacy 8bits LUT or enhanced LUTs (of 10 or
> 12bits).
> 
> The CSC unit is only available from Haswell on.
> 
> We also need to make a special case for CherryView which is recognized
> as a gen 8 but doesn't have the same enhanced color correction unit
> from Haswell on.
> 
> v2: Fix access to GAMMA_MODE register on older generations than
>     Haswell (from Matt Roper's comments)
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

Reviewed-by: Matt Roper <matthew.d.roper at intel.com>

> ---
>  drivers/gpu/drm/i915/Makefile        |   1 +
>  drivers/gpu/drm/i915/i915_drv.h      |   2 +
>  drivers/gpu/drm/i915/intel_color.c   | 195 
> +++++++++++++++++++++++++++++++++++
>  drivers/gpu/drm/i915/intel_display.c | 163 +++--------------------------
>  drivers/gpu/drm/i915/intel_drv.h     |  10 ++
>  5 files changed, 220 insertions(+), 151 deletions(-)
>  create mode 100644 drivers/gpu/drm/i915/intel_color.c
> 
> diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
> index 0851de07..0516300 100644
> --- a/drivers/gpu/drm/i915/Makefile
> +++ b/drivers/gpu/drm/i915/Makefile
> @@ -55,6 +55,7 @@ i915-y += intel_audio.o \
>         intel_atomic.o \
>         intel_atomic_plane.o \
>         intel_bios.o \
> +       intel_color.o \
>         intel_display.o \
>         intel_fbc.o \
>         intel_fifo_underrun.o \
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 6644c2e..6634c09 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -659,6 +659,8 @@ struct drm_i915_display_funcs {
>       /* render clock increase/decrease */
>       /* display clock increase/decrease */
>       /* pll clock increase/decrease */
> +
> +     void (*load_luts)(struct drm_crtc *crtc);
>  };
>  
>  enum forcewake_domain_id {
> diff --git a/drivers/gpu/drm/i915/intel_color.c 
> b/drivers/gpu/drm/i915/intel_color.c
> new file mode 100644
> index 0000000..cce0155
> --- /dev/null
> +++ b/drivers/gpu/drm/i915/intel_color.c
> @@ -0,0 +1,195 @@
> +/*
> + * Copyright © 2016 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> + * DEALINGS IN THE SOFTWARE.
> + *
> + */
> +
> +#include "intel_drv.h"
> +
> +/*
> + * Set up the pipe CSC unit.
> + *
> + * Currently only full range RGB to limited range RGB conversion
> + * is supported, but eventually this should handle various
> + * RGB<->YCbCr scenarios as well.
> + */
> +static void i9xx_load_csc_matrix(struct drm_crtc *crtc)
> +{
> +     struct drm_device *dev = crtc->dev;
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> +     int pipe = intel_crtc->pipe;
> +     uint16_t coeff = 0x7800; /* 1.0 */
> +
> +     /*
> +      * TODO: Check what kind of values actually come out of the pipe
> +      * with these coeff/postoff values and adjust to get the best
> +      * accuracy. Perhaps we even need to take the bpc value into
> +      * consideration.
> +      */
> +
> +     if (intel_crtc->config->limited_color_range)
> +             coeff = ((235 - 16) * (1 << 12) / 255) & 0xff8; /* 0.xxx... */
> +
> +     /*
> +      * GY/GU and RY/RU should be the other way around according
> +      * to BSpec, but reality doesn't agree. Just set them up in
> +      * a way that results in the correct picture.
> +      */
> +     I915_WRITE(PIPE_CSC_COEFF_RY_GY(pipe), coeff << 16);
> +     I915_WRITE(PIPE_CSC_COEFF_BY(pipe), 0);
> +
> +     I915_WRITE(PIPE_CSC_COEFF_RU_GU(pipe), coeff);
> +     I915_WRITE(PIPE_CSC_COEFF_BU(pipe), 0);
> +
> +     I915_WRITE(PIPE_CSC_COEFF_RV_GV(pipe), 0);
> +     I915_WRITE(PIPE_CSC_COEFF_BV(pipe), coeff << 16);
> +
> +     I915_WRITE(PIPE_CSC_PREOFF_HI(pipe), 0);
> +     I915_WRITE(PIPE_CSC_PREOFF_ME(pipe), 0);
> +     I915_WRITE(PIPE_CSC_PREOFF_LO(pipe), 0);
> +
> +     if (INTEL_INFO(dev)->gen > 6) {
> +             uint16_t postoff = 0;
> +
> +             if (intel_crtc->config->limited_color_range)
> +                     postoff = (16 * (1 << 12) / 255) & 0x1fff;
> +
> +             I915_WRITE(PIPE_CSC_POSTOFF_HI(pipe), postoff);
> +             I915_WRITE(PIPE_CSC_POSTOFF_ME(pipe), postoff);
> +             I915_WRITE(PIPE_CSC_POSTOFF_LO(pipe), postoff);
> +
> +             I915_WRITE(PIPE_CSC_MODE(pipe), 0);
> +     } else {
> +             uint32_t mode = CSC_MODE_YUV_TO_RGB;
> +
> +             if (intel_crtc->config->limited_color_range)
> +                     mode |= CSC_BLACK_SCREEN_OFFSET;
> +
> +             I915_WRITE(PIPE_CSC_MODE(pipe), mode);
> +     }
> +}
> +
> +void intel_color_set_csc(struct drm_crtc *crtc)
> +{
> +     i9xx_load_csc_matrix(crtc);
> +}
> +
> +/* Loads the palette/gamma unit for the CRTC with the prepared values. */
> +static void i9xx_load_luts(struct drm_crtc *crtc)
> +{
> +     struct drm_device *dev = crtc->dev;
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> +     enum pipe pipe = intel_crtc->pipe;
> +     int i;
> +
> +     if (HAS_GMCH_DISPLAY(dev)) {
> +             if (intel_crtc->config->has_dsi_encoder)
> +                     assert_dsi_pll_enabled(dev_priv);
> +             else
> +                     assert_pll_enabled(dev_priv, pipe);
> +     }
> +
> +     for (i = 0; i < 256; i++) {
> +             uint32_t word = (intel_crtc->lut_r[i] << 16) |
> +                     (intel_crtc->lut_g[i] << 8) |
> +                     intel_crtc->lut_b[i];
> +             if (HAS_GMCH_DISPLAY(dev))
> +                     I915_WRITE(PALETTE(pipe, i), word);
> +             else
> +                     I915_WRITE(LGC_PALETTE(pipe, i), word);
> +     }
> +}
> +
> +/* Loads the legacy palette/gamma unit for the CRTC on Haswell+. */
> +static void haswell_load_luts(struct drm_crtc *crtc)
> +{
> +     struct drm_device *dev = crtc->dev;
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> +     bool reenable_ips = false;
> +
> +     /*
> +      * Workaround : Do not read or write the pipe palette/gamma data while
> +      * GAMMA_MODE is configured for split gamma and IPS_CTL has IPS enabled.
> +      */
> +     if (IS_HASWELL(dev) && intel_crtc->config->ips_enabled &&
> +         ((I915_READ(GAMMA_MODE(intel_crtc->pipe)) & GAMMA_MODE_MODE_MASK) ==
> +          GAMMA_MODE_MODE_SPLIT)) {
> +             hsw_disable_ips(intel_crtc);
> +             reenable_ips = true;
> +     }
> +     I915_WRITE(GAMMA_MODE(intel_crtc->pipe), GAMMA_MODE_MODE_8BIT);
> +
> +     i9xx_load_luts(crtc);
> +
> +     if (reenable_ips)
> +             hsw_enable_ips(intel_crtc);
> +}
> +
> +void intel_color_load_luts(struct drm_crtc *crtc)
> +{
> +     struct drm_device *dev = crtc->dev;
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +
> +     /* The clocks have to be on to load the palette. */
> +     if (!crtc->state->active)
> +             return;
> +
> +     dev_priv->display.load_luts(crtc);
> +}
> +
> +void intel_color_legacy_gamma_set(struct drm_crtc *crtc, u16 *red, u16 
> *green,
> +                               u16 *blue, uint32_t start, uint32_t size)
> +{
> +     int end = (start + size > 256) ? 256 : start + size, i;
> +     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> +
> +     for (i = start; i < end; i++) {
> +             intel_crtc->lut_r[i] = red[i] >> 8;
> +             intel_crtc->lut_g[i] = green[i] >> 8;
> +             intel_crtc->lut_b[i] = blue[i] >> 8;
> +     }
> +
> +     intel_color_load_luts(crtc);
> +}
> +
> +void intel_color_init(struct drm_crtc *crtc)
> +{
> +     struct drm_device *dev = crtc->dev;
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> +     int i;
> +
> +     drm_mode_crtc_set_gamma_size(crtc, 256);
> +     for (i = 0; i < 256; i++) {
> +             intel_crtc->lut_r[i] = i;
> +             intel_crtc->lut_g[i] = i;
> +             intel_crtc->lut_b[i] = i;
> +     }
> +
> +     if (IS_HASWELL(dev) ||
> +         (INTEL_INFO(dev)->gen >= 8 && !IS_CHERRYVIEW(dev)))
> +             dev_priv->display.load_luts = haswell_load_luts;
> +     else
> +             dev_priv->display.load_luts = i9xx_load_luts;
> +}
> diff --git a/drivers/gpu/drm/i915/intel_display.c 
> b/drivers/gpu/drm/i915/intel_display.c
> index deee560..acbb1d9 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -101,7 +101,6 @@ static void intel_cpu_transcoder_set_m_n(struct 
> intel_crtc *crtc,
>                                        struct intel_link_m_n *m2_n2);
>  static void ironlake_set_pipeconf(struct drm_crtc *crtc);
>  static void haswell_set_pipeconf(struct drm_crtc *crtc);
> -static void intel_set_pipe_csc(struct drm_crtc *crtc);
>  static void vlv_prepare_pll(struct intel_crtc *crtc,
>                           const struct intel_crtc_state *pipe_config);
>  static void chv_prepare_pll(struct intel_crtc *crtc,
> @@ -1165,7 +1164,7 @@ void assert_pll(struct drm_i915_private *dev_priv,
>  }
>  
>  /* XXX: the dsi pll is shared between MIPI DSI ports */
> -static void assert_dsi_pll(struct drm_i915_private *dev_priv, bool state)
> +void assert_dsi_pll(struct drm_i915_private *dev_priv, bool state)
>  {
>       u32 val;
>       bool cur_state;
> @@ -1179,8 +1178,6 @@ static void assert_dsi_pll(struct drm_i915_private 
> *dev_priv, bool state)
>            "DSI PLL state assertion failure (expected %s, current %s)\n",
>                       onoff(state), onoff(cur_state));
>  }
> -#define assert_dsi_pll_enabled(d) assert_dsi_pll(d, true)
> -#define assert_dsi_pll_disabled(d) assert_dsi_pll(d, false)
>  
>  struct intel_shared_dpll *
>  intel_crtc_to_shared_dpll(struct intel_crtc *crtc)
> @@ -3315,7 +3312,7 @@ static void intel_update_pipe_config(struct intel_crtc 
> *crtc,
>                     pipe_config->pipe_src_w, pipe_config->pipe_src_h);
>  
>       if (HAS_DDI(dev))
> -             intel_set_pipe_csc(&crtc->base);
> +             intel_color_set_csc(&crtc->base);
>  
>       /*
>        * Update pipe size and adjust fitter if needed: the reason for this is
> @@ -4626,55 +4623,6 @@ void hsw_disable_ips(struct intel_crtc *crtc)
>       intel_wait_for_vblank(dev, crtc->pipe);
>  }
>  
> -/** Loads the palette/gamma unit for the CRTC with the prepared values */
> -static void intel_crtc_load_lut(struct drm_crtc *crtc)
> -{
> -     struct drm_device *dev = crtc->dev;
> -     struct drm_i915_private *dev_priv = dev->dev_private;
> -     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> -     enum pipe pipe = intel_crtc->pipe;
> -     int i;
> -     bool reenable_ips = false;
> -
> -     /* The clocks have to be on to load the palette. */
> -     if (!crtc->state->active)
> -             return;
> -
> -     if (HAS_GMCH_DISPLAY(dev_priv->dev)) {
> -             if (intel_crtc->config->has_dsi_encoder)
> -                     assert_dsi_pll_enabled(dev_priv);
> -             else
> -                     assert_pll_enabled(dev_priv, pipe);
> -     }
> -
> -     /* Workaround : Do not read or write the pipe palette/gamma data while
> -      * GAMMA_MODE is configured for split gamma and IPS_CTL has IPS enabled.
> -      */
> -     if (IS_HASWELL(dev) && intel_crtc->config->ips_enabled &&
> -         ((I915_READ(GAMMA_MODE(pipe)) & GAMMA_MODE_MODE_MASK) ==
> -          GAMMA_MODE_MODE_SPLIT)) {
> -             hsw_disable_ips(intel_crtc);
> -             reenable_ips = true;
> -     }
> -
> -     for (i = 0; i < 256; i++) {
> -             i915_reg_t palreg;
> -
> -             if (HAS_GMCH_DISPLAY(dev))
> -                     palreg = PALETTE(pipe, i);
> -             else
> -                     palreg = LGC_PALETTE(pipe, i);
> -
> -             I915_WRITE(palreg,
> -                        (intel_crtc->lut_r[i] << 16) |
> -                        (intel_crtc->lut_g[i] << 8) |
> -                        intel_crtc->lut_b[i]);
> -     }
> -
> -     if (reenable_ips)
> -             hsw_enable_ips(intel_crtc);
> -}
> -
>  static void intel_crtc_dpms_overlay_disable(struct intel_crtc *intel_crtc)
>  {
>       if (intel_crtc->overlay) {
> @@ -4923,7 +4871,7 @@ static void ironlake_crtc_enable(struct drm_crtc *crtc)
>        * On ILK+ LUT must be loaded before the pipe is running but with
>        * clocks enabled
>        */
> -     intel_crtc_load_lut(crtc);
> +     intel_color_load_luts(crtc);
>  
>       intel_update_watermarks(crtc);
>       intel_enable_pipe(intel_crtc);
> @@ -4989,7 +4937,7 @@ static void haswell_crtc_enable(struct drm_crtc *crtc)
>  
>       haswell_set_pipeconf(crtc);
>  
> -     intel_set_pipe_csc(crtc);
> +     intel_color_set_csc(crtc);
>  
>       intel_crtc->active = true;
>  
> @@ -5018,7 +4966,7 @@ static void haswell_crtc_enable(struct drm_crtc *crtc)
>        * On ILK+ LUT must be loaded before the pipe is running but with
>        * clocks enabled
>        */
> -     intel_crtc_load_lut(crtc);
> +     intel_color_load_luts(crtc);
>  
>       intel_ddi_set_pipe_settings(crtc);
>       if (!intel_crtc->config->has_dsi_encoder)
> @@ -6229,7 +6177,7 @@ static void valleyview_crtc_enable(struct drm_crtc 
> *crtc)
>  
>       i9xx_pfit_enable(intel_crtc);
>  
> -     intel_crtc_load_lut(crtc);
> +     intel_color_load_luts(crtc);
>  
>       intel_enable_pipe(intel_crtc);
>  
> @@ -6282,7 +6230,7 @@ static void i9xx_crtc_enable(struct drm_crtc *crtc)
>  
>       i9xx_pfit_enable(intel_crtc);
>  
> -     intel_crtc_load_lut(crtc);
> +     intel_color_load_luts(crtc);
>  
>       intel_update_watermarks(crtc);
>       intel_enable_pipe(intel_crtc);
> @@ -8737,70 +8685,6 @@ static void ironlake_set_pipeconf(struct drm_crtc 
> *crtc)
>       POSTING_READ(PIPECONF(pipe));
>  }
>  
> -/*
> - * Set up the pipe CSC unit.
> - *
> - * Currently only full range RGB to limited range RGB conversion
> - * is supported, but eventually this should handle various
> - * RGB<->YCbCr scenarios as well.
> - */
> -static void intel_set_pipe_csc(struct drm_crtc *crtc)
> -{
> -     struct drm_device *dev = crtc->dev;
> -     struct drm_i915_private *dev_priv = dev->dev_private;
> -     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> -     int pipe = intel_crtc->pipe;
> -     uint16_t coeff = 0x7800; /* 1.0 */
> -
> -     /*
> -      * TODO: Check what kind of values actually come out of the pipe
> -      * with these coeff/postoff values and adjust to get the best
> -      * accuracy. Perhaps we even need to take the bpc value into
> -      * consideration.
> -      */
> -
> -     if (intel_crtc->config->limited_color_range)
> -             coeff = ((235 - 16) * (1 << 12) / 255) & 0xff8; /* 0.xxx... */
> -
> -     /*
> -      * GY/GU and RY/RU should be the other way around according
> -      * to BSpec, but reality doesn't agree. Just set them up in
> -      * a way that results in the correct picture.
> -      */
> -     I915_WRITE(PIPE_CSC_COEFF_RY_GY(pipe), coeff << 16);
> -     I915_WRITE(PIPE_CSC_COEFF_BY(pipe), 0);
> -
> -     I915_WRITE(PIPE_CSC_COEFF_RU_GU(pipe), coeff);
> -     I915_WRITE(PIPE_CSC_COEFF_BU(pipe), 0);
> -
> -     I915_WRITE(PIPE_CSC_COEFF_RV_GV(pipe), 0);
> -     I915_WRITE(PIPE_CSC_COEFF_BV(pipe), coeff << 16);
> -
> -     I915_WRITE(PIPE_CSC_PREOFF_HI(pipe), 0);
> -     I915_WRITE(PIPE_CSC_PREOFF_ME(pipe), 0);
> -     I915_WRITE(PIPE_CSC_PREOFF_LO(pipe), 0);
> -
> -     if (INTEL_INFO(dev)->gen > 6) {
> -             uint16_t postoff = 0;
> -
> -             if (intel_crtc->config->limited_color_range)
> -                     postoff = (16 * (1 << 12) / 255) & 0x1fff;
> -
> -             I915_WRITE(PIPE_CSC_POSTOFF_HI(pipe), postoff);
> -             I915_WRITE(PIPE_CSC_POSTOFF_ME(pipe), postoff);
> -             I915_WRITE(PIPE_CSC_POSTOFF_LO(pipe), postoff);
> -
> -             I915_WRITE(PIPE_CSC_MODE(pipe), 0);
> -     } else {
> -             uint32_t mode = CSC_MODE_YUV_TO_RGB;
> -
> -             if (intel_crtc->config->limited_color_range)
> -                     mode |= CSC_BLACK_SCREEN_OFFSET;
> -
> -             I915_WRITE(PIPE_CSC_MODE(pipe), mode);
> -     }
> -}
> -
>  static void haswell_set_pipeconf(struct drm_crtc *crtc)
>  {
>       struct drm_device *dev = crtc->dev;
> @@ -8823,9 +8707,6 @@ static void haswell_set_pipeconf(struct drm_crtc *crtc)
>       I915_WRITE(PIPECONF(cpu_transcoder), val);
>       POSTING_READ(PIPECONF(cpu_transcoder));
>  
> -     I915_WRITE(GAMMA_MODE(intel_crtc->pipe), GAMMA_MODE_MODE_8BIT);
> -     POSTING_READ(GAMMA_MODE(intel_crtc->pipe));
> -
>       if (IS_BROADWELL(dev) || INTEL_INFO(dev)->gen >= 9) {
>               val = 0;
>  
> @@ -10238,21 +10119,6 @@ static bool cursor_size_ok(struct drm_device *dev,
>       return true;
>  }
>  
> -static void intel_crtc_gamma_set(struct drm_crtc *crtc, u16 *red, u16 *green,
> -                              u16 *blue, uint32_t start, uint32_t size)
> -{
> -     int end = (start + size > 256) ? 256 : start + size, i;
> -     struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> -
> -     for (i = start; i < end; i++) {
> -             intel_crtc->lut_r[i] = red[i] >> 8;
> -             intel_crtc->lut_g[i] = green[i] >> 8;
> -             intel_crtc->lut_b[i] = blue[i] >> 8;
> -     }
> -
> -     intel_crtc_load_lut(crtc);
> -}
> -
>  /* VESA 640x480x72Hz mode to set on the pipe */
>  static struct drm_display_mode load_detect_mode = {
>       DRM_MODE("640x480", DRM_MODE_TYPE_DEFAULT, 31500, 640, 664,
> @@ -12001,7 +11867,7 @@ static int intel_crtc_atomic_check(struct drm_crtc 
> *crtc,
>  
>  static const struct drm_crtc_helper_funcs intel_helper_funcs = {
>       .mode_set_base_atomic = intel_pipe_set_base_atomic,
> -     .load_lut = intel_crtc_load_lut,
> +     .load_lut = intel_color_load_luts,
>       .atomic_begin = intel_begin_crtc_commit,
>       .atomic_flush = intel_finish_crtc_commit,
>       .atomic_check = intel_crtc_atomic_check,
> @@ -13621,7 +13487,7 @@ out:
>  #undef for_each_intel_crtc_masked
>  
>  static const struct drm_crtc_funcs intel_crtc_funcs = {
> -     .gamma_set = intel_crtc_gamma_set,
> +     .gamma_set = intel_color_legacy_gamma_set,
>       .set_config = drm_atomic_helper_set_config,
>       .destroy = intel_crtc_destroy,
>       .page_flip = intel_crtc_page_flip,
> @@ -14228,7 +14094,7 @@ static void intel_crtc_init(struct drm_device *dev, 
> int pipe)
>       struct intel_crtc_state *crtc_state = NULL;
>       struct drm_plane *primary = NULL;
>       struct drm_plane *cursor = NULL;
> -     int i, ret;
> +     int ret;
>  
>       intel_crtc = kzalloc(sizeof(*intel_crtc), GFP_KERNEL);
>       if (intel_crtc == NULL)
> @@ -14264,13 +14130,6 @@ static void intel_crtc_init(struct drm_device *dev, 
> int pipe)
>       if (ret)
>               goto fail;
>  
> -     drm_mode_crtc_set_gamma_size(&intel_crtc->base, 256);
> -     for (i = 0; i < 256; i++) {
> -             intel_crtc->lut_r[i] = i;
> -             intel_crtc->lut_g[i] = i;
> -             intel_crtc->lut_b[i] = i;
> -     }
> -
>       /*
>        * On gen2/3 only plane A can do fbc, but the panel fitter and lvds port
>        * is hooked to pipe B. Hence we want plane A feeding pipe B.
> @@ -14295,6 +14154,8 @@ static void intel_crtc_init(struct drm_device *dev, 
> int pipe)
>  
>       drm_crtc_helper_add(&intel_crtc->base, &intel_helper_funcs);
>  
> +     intel_color_init(&intel_crtc->base);
> +
>       WARN_ON(drm_crtc_index(&intel_crtc->base) != intel_crtc->pipe);
>       return;
>  
> diff --git a/drivers/gpu/drm/i915/intel_drv.h 
> b/drivers/gpu/drm/i915/intel_drv.h
> index 285b057..40fc486 100644
> --- a/drivers/gpu/drm/i915/intel_drv.h
> +++ b/drivers/gpu/drm/i915/intel_drv.h
> @@ -1164,6 +1164,9 @@ void assert_pll(struct drm_i915_private *dev_priv,
>               enum pipe pipe, bool state);
>  #define assert_pll_enabled(d, p) assert_pll(d, p, true)
>  #define assert_pll_disabled(d, p) assert_pll(d, p, false)
> +void assert_dsi_pll(struct drm_i915_private *dev_priv, bool state);
> +#define assert_dsi_pll_enabled(d) assert_dsi_pll(d, true)
> +#define assert_dsi_pll_disabled(d) assert_dsi_pll(d, false)
>  void assert_fdi_rx_pll(struct drm_i915_private *dev_priv,
>                      enum pipe pipe, bool state);
>  #define assert_fdi_rx_pll_enabled(d, p) assert_fdi_rx_pll(d, p, true)
> @@ -1616,4 +1619,11 @@ void intel_plane_destroy_state(struct drm_plane *plane,
>                              struct drm_plane_state *state);
>  extern const struct drm_plane_helper_funcs intel_plane_helper_funcs;
>  
> +/* intel_color.c */
> +void intel_color_init(struct drm_crtc *crtc);
> +void intel_color_set_csc(struct drm_crtc *crtc);
> +void intel_color_load_luts(struct drm_crtc *crtc);
> +void intel_color_legacy_gamma_set(struct drm_crtc *crtc, u16 *red, u16 
> *green,
> +                               u16 *blue, uint32_t start, uint32_t size);
> +
>  #endif /* __INTEL_DRV_H__ */
> -- 
> 2.7.0
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Matt Roper
Graphics Software Engineer
IoTG Platform Enabling & Development
Intel Corporation
(916) 356-2795

Reply via email to