From: Ville Syrjälä <[email protected]>

uapi.crtc is not set for joiner secondary pipes, so generally
should not be used anywhere after the initial state copy. Switch
to hw.crtc which actually indicates that the plane is enabled.

Signed-off-by: Ville Syrjälä <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_psr.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_psr.c 
b/drivers/gpu/drm/i915/display/intel_psr.c
index 00ac652809cc..22549b46467b 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -2707,7 +2707,7 @@ intel_psr2_sel_fetch_et_alignment(struct 
intel_atomic_state *state,
        for_each_new_intel_plane_in_state(state, plane, new_plane_state, i) {
                struct drm_rect inter;
 
-               if (new_plane_state->uapi.crtc != crtc_state->uapi.crtc)
+               if (new_plane_state->hw.crtc != crtc_state->uapi.crtc)
                        continue;
 
                if (plane->id != PLANE_CURSOR)
@@ -2844,7 +2844,7 @@ int intel_psr2_sel_fetch_update(struct intel_atomic_state 
*state,
                struct drm_rect src, damaged_area = { .x1 = 0, .y1 = -1,
                                                      .x2 = INT_MAX };
 
-               if (new_plane_state->uapi.crtc != crtc_state->uapi.crtc)
+               if (new_plane_state->hw.crtc != crtc_state->uapi.crtc)
                        continue;
 
                if (!new_plane_state->uapi.visible &&
@@ -2943,7 +2943,7 @@ int intel_psr2_sel_fetch_update(struct intel_atomic_state 
*state,
                struct drm_rect *sel_fetch_area, inter;
                struct intel_plane *linked = 
new_plane_state->planar_linked_plane;
 
-               if (new_plane_state->uapi.crtc != crtc_state->uapi.crtc ||
+               if (new_plane_state->hw.crtc != crtc_state->uapi.crtc ||
                    !new_plane_state->uapi.visible)
                        continue;
 
-- 
2.49.1

Reply via email to