[PATCH 7/9] drm/i915/vrr: Handle joiner with vrr

[Ankit Nautiyal]

Do not program transcoder registers for VRR for the secondary pipe of
the joiner. Remove check to skip VRR for joiner case.

Signed-off-by: Ankit Nautiyal <ankit.k.nauti...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_vrr.c | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c 
b/drivers/gpu/drm/i915/display/intel_vrr.c
index ae9bea854282..025063098370 100644
--- a/drivers/gpu/drm/i915/display/intel_vrr.c
+++ b/drivers/gpu/drm/i915/display/intel_vrr.c
@@ -171,13 +171,6 @@ intel_vrr_compute_config(struct intel_crtc_state 
*crtc_state,
        const struct drm_display_info *info = &connector->base.display_info;
        int vmin, vmax;
 
-       /*
-        * FIXME all joined pipes share the same transcoder.
-        * Need to account for that during VRR toggle/push/etc.
-        */
-       if (crtc_state->joiner_pipes)
-               return;
-
        if (adjusted_mode->flags & DRM_MODE_FLAG_INTERLACE)
                return;
 
@@ -274,6 +267,9 @@ void intel_vrr_set_transcoder_timings(const struct 
intel_crtc_state *crtc_state)
        struct drm_i915_private *dev_priv = to_i915(crtc_state->uapi.crtc->dev);
        enum transcoder cpu_transcoder = crtc_state->cpu_transcoder;
 
+       if (intel_crtc_is_joiner_secondary(crtc_state))
+               return;
+
        /*
         * This bit seems to have two meanings depending on the platform:
         * TGL: generate VRR "safe window" for DSB vblank waits
@@ -316,6 +312,9 @@ void intel_vrr_send_push(const struct intel_crtc_state 
*crtc_state)
        struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum transcoder cpu_transcoder = crtc_state->cpu_transcoder;
 
+       if (intel_crtc_is_joiner_secondary(crtc_state))
+               return;
+
        if (!crtc_state->vrr.enable || crtc_state->vrr.fixed_rr)
                return;
 
@@ -340,6 +339,9 @@ void intel_vrr_enable(const struct intel_crtc_state 
*crtc_state)
        struct drm_i915_private *dev_priv = to_i915(crtc_state->uapi.crtc->dev);
        enum transcoder cpu_transcoder = crtc_state->cpu_transcoder;
 
+       if (intel_crtc_is_joiner_secondary(crtc_state))
+               return;
+
        if (!crtc_state->vrr.enable)
                return;
 
@@ -369,6 +371,9 @@ void intel_vrr_disable(const struct intel_crtc_state 
*old_crtc_state)
        struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum transcoder cpu_transcoder = old_crtc_state->cpu_transcoder;
 
+       if (intel_crtc_is_joiner_secondary(old_crtc_state))
+               return;
+
        if (!old_crtc_state->vrr.enable)
                return;
 
-- 
2.40.1