Commit 85a96e7a authored by Maarten Lankhorst's avatar Maarten Lankhorst Committed by Jani Nikula

drm/i915: Make sure all planes and connectors are added on modeset.

Add missing calls to drm_atomic_add_affected_*. This is needed
to convert to atomic planes. When converting to atomic all planes
are needed on modeset. For good measure make sure all connectors
are added too.
Signed-off-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: default avatarMatt Roper <matthew.d.roper@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
parent 53d9f4e9
...@@ -5959,7 +5959,7 @@ static int valleyview_modeset_global_pipes(struct drm_atomic_state *state) ...@@ -5959,7 +5959,7 @@ static int valleyview_modeset_global_pipes(struct drm_atomic_state *state)
struct drm_crtc *crtc; struct drm_crtc *crtc;
struct drm_crtc_state *crtc_state; struct drm_crtc_state *crtc_state;
int max_pixclk = intel_mode_max_pixclk(state->dev, state); int max_pixclk = intel_mode_max_pixclk(state->dev, state);
int cdclk, i; int cdclk, ret = 0;
if (max_pixclk < 0) if (max_pixclk < 0)
return max_pixclk; return max_pixclk;
...@@ -5974,20 +5974,25 @@ static int valleyview_modeset_global_pipes(struct drm_atomic_state *state) ...@@ -5974,20 +5974,25 @@ static int valleyview_modeset_global_pipes(struct drm_atomic_state *state)
/* add all active pipes to the state */ /* add all active pipes to the state */
for_each_crtc(state->dev, crtc) { for_each_crtc(state->dev, crtc) {
if (!crtc->state->active)
continue;
crtc_state = drm_atomic_get_crtc_state(state, crtc); crtc_state = drm_atomic_get_crtc_state(state, crtc);
if (IS_ERR(crtc_state)) if (IS_ERR(crtc_state))
return PTR_ERR(crtc_state); return PTR_ERR(crtc_state);
}
/* disable/enable all currently active pipes while we change cdclk */ if (!crtc_state->active || needs_modeset(crtc_state))
for_each_crtc_in_state(state, crtc, crtc_state, i) continue;
if (crtc_state->active)
crtc_state->mode_changed = true;
return 0; crtc_state->mode_changed = true;
ret = drm_atomic_add_affected_connectors(state, crtc);
if (ret)
break;
ret = drm_atomic_add_affected_planes(state, crtc);
if (ret)
break;
}
return ret;
} }
static void vlv_program_pfi_credits(struct drm_i915_private *dev_priv) static void vlv_program_pfi_credits(struct drm_i915_private *dev_priv)
...@@ -12185,8 +12190,10 @@ intel_modeset_pipe_config(struct drm_crtc *crtc, ...@@ -12185,8 +12190,10 @@ intel_modeset_pipe_config(struct drm_crtc *crtc,
/* Check if we need to force a modeset */ /* Check if we need to force a modeset */
if (pipe_config->has_audio != if (pipe_config->has_audio !=
to_intel_crtc_state(crtc->state)->has_audio) to_intel_crtc_state(crtc->state)->has_audio) {
pipe_config->base.mode_changed = true; pipe_config->base.mode_changed = true;
ret = drm_atomic_add_affected_planes(state, crtc);
}
/* /*
* Note we have an issue here with infoframes: current code * Note we have an issue here with infoframes: current code
...@@ -12194,8 +12201,6 @@ intel_modeset_pipe_config(struct drm_crtc *crtc, ...@@ -12194,8 +12201,6 @@ intel_modeset_pipe_config(struct drm_crtc *crtc,
* requirements. So here we should be checking for any * requirements. So here we should be checking for any
* required changes and forcing a mode set. * required changes and forcing a mode set.
*/ */
return 0;
fail: fail:
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment