Skip to content

Commit 2bdd4c2

Browse files
uma-intelrodrigovivi
authored andcommitted
drm/i915/display: Fix mode private_flags comparison at atomic_check
This patch fixes the private_flags of mode to be checked and compared against uapi.mode and not from hw.mode. This helps properly trigger modeset at boot if desired by driver. It helps resolve audio_codec initialization issues if display is connected at boot. Initial discussion on this issue has happened on below thread: https://patchwork.freedesktop.org/series/74828/ v2: No functional change. Fixed the Closes tag and added Maarten's RB. v3: Added Fixes tag. Cc: Ville Syrjä <[email protected]> Cc: Maarten Lankhorst <[email protected]> Cc: Kai Vehmanen <[email protected]> Cc: Souza, Jose <[email protected]> Fixes: 58d124e ("drm/i915: Complete crtc hw/uapi split, v6.") Closes: https://gitlab.freedesktop.org/drm/intel/issues/1363 Suggested-by: Ville Syrjä <[email protected]> Signed-off-by: Uma Shankar <[email protected]> Signed-off-by: SweeAun Khor <[email protected]> Reviewed-by: Maarten Lankhorst <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] (cherry picked from commit d5e5670) Signed-off-by: Rodrigo Vivi <[email protected]>
1 parent a97b786 commit 2bdd4c2

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/i915/display/intel_display.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14748,8 +14748,8 @@ static int intel_atomic_check(struct drm_device *dev,
1474814748
/* Catch I915_MODE_FLAG_INHERITED */
1474914749
for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,
1475014750
new_crtc_state, i) {
14751-
if (new_crtc_state->hw.mode.private_flags !=
14752-
old_crtc_state->hw.mode.private_flags)
14751+
if (new_crtc_state->uapi.mode.private_flags !=
14752+
old_crtc_state->uapi.mode.private_flags)
1475314753
new_crtc_state->uapi.mode_changed = true;
1475414754
}
1475514755

0 commit comments

Comments
 (0)