From: José Roberto de Souza <[email protected]>

MST and port sync have master and slaves pipes and it brings
dependencies between pipes to allow fastset.
For example if only MST master needs a modeset all of its slaves also
needs to do a modeset.

This patch adds the base for external dependencies check, the MST and
port sync bits will be added in another patches.

v3:
- moved handling to intel_atomic_check() this way is guarantee that
all pipes will have its state computed

v4:
- added a function to return if MST master neeeds modeset to simply
code in intel_atomic_check()

v5:
- fixed and moved code to check if MST master needs a modeset

v6:
- previons version of this patch was split into two patches

Cc: Ville Syrjälä <[email protected]>
Cc: Lucas De Marchi <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Cc: Manasi Navare <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]>
Signed-off-by: José Roberto de Souza <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_display.c | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c 
b/drivers/gpu/drm/i915/display/intel_display.c
index a43df7fc1941..cb23e4831d4b 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -13919,6 +13919,11 @@ static void intel_crtc_check_fastset(const struct 
intel_crtc_state *old_crtc_sta
 
        new_crtc_state->uapi.mode_changed = false;
        new_crtc_state->update_pipe = true;
+}
+
+static void intel_crtc_copy_fastset(const struct intel_crtc_state 
*old_crtc_state,
+                                   struct intel_crtc_state *new_crtc_state)
+{
 
        /*
         * If we're not doing the full modeset we want to
@@ -14094,9 +14099,19 @@ static int intel_atomic_check(struct drm_device *dev,
                        goto fail;
 
                intel_crtc_check_fastset(old_crtc_state, new_crtc_state);
+       }
 
-               if (needs_modeset(new_crtc_state))
+       for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,
+                                           new_crtc_state, i) {
+               if (needs_modeset(new_crtc_state)) {
                        any_ms = true;
+                       continue;
+               }
+
+               if (!new_crtc_state->update_pipe)
+                       continue;
+
+               intel_crtc_copy_fastset(old_crtc_state, new_crtc_state);
        }
 
        if (any_ms && !check_digital_port_conflicts(state)) {
-- 
2.19.1

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to