Commit 4978cc93 authored by Ander Conselvan de Oliveira's avatar Ander Conselvan de Oliveira Committed by Daniel Vetter

drm/i915: Preserve shared DPLL information in new pipe_config

When a new pipe_config is calculated, the fields related to shared dplls
are reset, under the assumption that they will be recalculated as part
of the modeset, which is true with the current state of the code.

As we convert to atomic, however, it will be possible to calculate a new
pipe_config and skip the modeset. In that case, after the state swap we
still want the shared dplls to be preserved.
Signed-off-by: default avatarAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Reviewed-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 2bfb4627
...@@ -11467,13 +11467,21 @@ clear_intel_crtc_state(struct intel_crtc_state *crtc_state) ...@@ -11467,13 +11467,21 @@ clear_intel_crtc_state(struct intel_crtc_state *crtc_state)
{ {
struct drm_crtc_state tmp_state; struct drm_crtc_state tmp_state;
struct intel_crtc_scaler_state scaler_state; struct intel_crtc_scaler_state scaler_state;
struct intel_dpll_hw_state dpll_hw_state;
enum intel_dpll_id shared_dpll;
/* Clear only the intel specific part of the crtc state excluding scalers */ /* Clear only the intel specific part of the crtc state excluding scalers */
tmp_state = crtc_state->base; tmp_state = crtc_state->base;
scaler_state = crtc_state->scaler_state; scaler_state = crtc_state->scaler_state;
shared_dpll = crtc_state->shared_dpll;
dpll_hw_state = crtc_state->dpll_hw_state;
memset(crtc_state, 0, sizeof *crtc_state); memset(crtc_state, 0, sizeof *crtc_state);
crtc_state->base = tmp_state; crtc_state->base = tmp_state;
crtc_state->scaler_state = scaler_state; crtc_state->scaler_state = scaler_state;
crtc_state->shared_dpll = shared_dpll;
crtc_state->dpll_hw_state = dpll_hw_state;
} }
static int static int
...@@ -11502,7 +11510,6 @@ intel_modeset_pipe_config(struct drm_crtc *crtc, ...@@ -11502,7 +11510,6 @@ intel_modeset_pipe_config(struct drm_crtc *crtc,
pipe_config->cpu_transcoder = pipe_config->cpu_transcoder =
(enum transcoder) to_intel_crtc(crtc)->pipe; (enum transcoder) to_intel_crtc(crtc)->pipe;
pipe_config->shared_dpll = DPLL_ID_PRIVATE;
/* /*
* Sanitize sync polarity flags based on requested ones. If neither * Sanitize sync polarity flags based on requested ones. If neither
...@@ -12266,9 +12273,14 @@ static int __intel_set_mode_setup_plls(struct drm_atomic_state *state) ...@@ -12266,9 +12273,14 @@ static int __intel_set_mode_setup_plls(struct drm_atomic_state *state)
for_each_crtc_in_state(state, crtc, crtc_state, i) { for_each_crtc_in_state(state, crtc, crtc_state, i) {
intel_crtc = to_intel_crtc(crtc); intel_crtc = to_intel_crtc(crtc);
intel_crtc_state = to_intel_crtc_state(crtc_state);
if (needs_modeset(crtc_state)) if (needs_modeset(crtc_state)) {
clear_pipes |= 1 << intel_crtc->pipe; clear_pipes |= 1 << intel_crtc->pipe;
intel_crtc_state->shared_dpll = DPLL_ID_PRIVATE;
memset(&intel_crtc_state->dpll_hw_state, 0,
sizeof(intel_crtc_state->dpll_hw_state));
}
} }
ret = intel_shared_dpll_start_config(dev_priv, clear_pipes); ret = intel_shared_dpll_start_config(dev_priv, clear_pipes);
......
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