]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
drm/i915: Generalize transcoder looping
authorMika Kahola <mika.kahola@intel.com>
Thu, 9 Nov 2017 08:37:50 +0000 (10:37 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 14 Nov 2017 14:17:51 +0000 (16:17 +0200)
To make looping through transcoders in intel_ddi.c more generic, let's switch
to use 'for_each_pipe()' macro to do this.

v2: Add a notion that we are dealing with transcoders instead of pipes (Jani)

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Mika Kahola <mika.kahola@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1510216670-16848-1-git-send-email-mika.kahola@intel.com
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
drivers/gpu/drm/i915/intel_ddi.c

index 65ab55496ab74aa82df8de2056277f3e7427da35..eff3b51872ebc47f6447be9547fb9cdbf33e3264 100644 (file)
@@ -1681,8 +1681,8 @@ bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
        struct drm_device *dev = encoder->base.dev;
        struct drm_i915_private *dev_priv = to_i915(dev);
        enum port port = encoder->port;
+       enum pipe p;
        u32 tmp;
-       int i;
        bool ret;
 
        if (!intel_display_power_get_if_enabled(dev_priv,
@@ -1717,15 +1717,17 @@ bool intel_ddi_get_hw_state(struct intel_encoder *encoder,
                goto out;
        }
 
-       for (i = TRANSCODER_A; i <= TRANSCODER_C; i++) {
-               tmp = I915_READ(TRANS_DDI_FUNC_CTL(i));
+       for_each_pipe(dev_priv, p) {
+               enum transcoder cpu_transcoder = (enum transcoder) p;
+
+               tmp = I915_READ(TRANS_DDI_FUNC_CTL(cpu_transcoder));
 
                if ((tmp & TRANS_DDI_PORT_MASK) == TRANS_DDI_SELECT_PORT(port)) {
                        if ((tmp & TRANS_DDI_MODE_SELECT_MASK) ==
                            TRANS_DDI_MODE_SELECT_DP_MST)
                                goto out;
 
-                       *pipe = i;
+                       *pipe = p;
                        ret = true;
 
                        goto out;