drm/i915: Remove the "pf" crc source
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 14 Feb 2019 19:22:16 +0000 (21:22 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 20 Feb 2019 20:39:19 +0000 (22:39 +0200)
The "pipe" and "pf" crc sources are in fact the same thing.
Remove the "pf" one.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190214192219.3858-1-ville.syrjala@linux.intel.com
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/intel_pipe_crc.c

index 1eaaf70..a5a2f5b 100644 (file)
@@ -1198,7 +1198,6 @@ enum intel_pipe_crc_source {
        INTEL_PIPE_CRC_SOURCE_NONE,
        INTEL_PIPE_CRC_SOURCE_PLANE1,
        INTEL_PIPE_CRC_SOURCE_PLANE2,
-       INTEL_PIPE_CRC_SOURCE_PF,
        INTEL_PIPE_CRC_SOURCE_PIPE,
        /* TV/DP on pre-gen5/vlv can't use the pipe source. */
        INTEL_PIPE_CRC_SOURCE_TV,
index a8554dc..a3a3ad7 100644 (file)
@@ -34,7 +34,6 @@ static const char * const pipe_crc_sources[] = {
        "none",
        "plane1",
        "plane2",
-       "pf",
        "pipe",
        "TV",
        "DP-B",
@@ -396,7 +395,7 @@ static int ivb_pipe_crc_ctl_reg(struct drm_i915_private *dev_priv,
                                bool set_wa)
 {
        if (*source == INTEL_PIPE_CRC_SOURCE_AUTO)
-               *source = INTEL_PIPE_CRC_SOURCE_PF;
+               *source = INTEL_PIPE_CRC_SOURCE_PIPE;
 
        switch (*source) {
        case INTEL_PIPE_CRC_SOURCE_PLANE1:
@@ -405,7 +404,7 @@ static int ivb_pipe_crc_ctl_reg(struct drm_i915_private *dev_priv,
        case INTEL_PIPE_CRC_SOURCE_PLANE2:
                *val = PIPE_CRC_ENABLE | PIPE_CRC_SOURCE_SPRITE_IVB;
                break;
-       case INTEL_PIPE_CRC_SOURCE_PF:
+       case INTEL_PIPE_CRC_SOURCE_PIPE:
                if (set_wa && (IS_HASWELL(dev_priv) ||
                     IS_BROADWELL(dev_priv)) && pipe == PIPE_A)
                        hsw_pipe_A_crc_wa(dev_priv, true);
@@ -532,7 +531,6 @@ static int ivb_crc_source_valid(struct drm_i915_private *dev_priv,
        case INTEL_PIPE_CRC_SOURCE_PIPE:
        case INTEL_PIPE_CRC_SOURCE_PLANE1:
        case INTEL_PIPE_CRC_SOURCE_PLANE2:
-       case INTEL_PIPE_CRC_SOURCE_PF:
        case INTEL_PIPE_CRC_SOURCE_NONE:
                return 0;
        default: