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

drm/i915: Allocate a crtc_state also when the crtc is being disabled

For consistency, allocate a new crtc_state for a crtc that is being
disabled. Previously only the enabled value of the current state would
change.

v2: Rebase on v5 of previous patch. (Ander)
Signed-off-by: default avatarAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
[danvet: Resolve rebase conflict.]
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 83a57153
...@@ -11291,13 +11291,20 @@ intel_modeset_compute_config(struct drm_crtc *crtc, ...@@ -11291,13 +11291,20 @@ intel_modeset_compute_config(struct drm_crtc *crtc,
unsigned *prepare_pipes, unsigned *prepare_pipes,
unsigned *disable_pipes) unsigned *disable_pipes)
{ {
struct drm_device *dev = crtc->dev;
struct intel_crtc_state *pipe_config = NULL; struct intel_crtc_state *pipe_config = NULL;
struct intel_crtc *intel_crtc;
intel_modeset_affected_pipes(crtc, modeset_pipes, intel_modeset_affected_pipes(crtc, modeset_pipes,
prepare_pipes, disable_pipes); prepare_pipes, disable_pipes);
if ((*modeset_pipes) == 0) for_each_intel_crtc_masked(dev, *disable_pipes, intel_crtc) {
return NULL; pipe_config = intel_atomic_get_crtc_state(state, intel_crtc);
if (IS_ERR(pipe_config))
return pipe_config;
pipe_config->base.enable = false;
}
/* /*
* Note this needs changes when we start tracking multiple modes * Note this needs changes when we start tracking multiple modes
...@@ -11305,14 +11312,21 @@ intel_modeset_compute_config(struct drm_crtc *crtc, ...@@ -11305,14 +11312,21 @@ intel_modeset_compute_config(struct drm_crtc *crtc,
* (i.e. one pipe_config for each crtc) rather than just the one * (i.e. one pipe_config for each crtc) rather than just the one
* for this crtc. * for this crtc.
*/ */
pipe_config = intel_modeset_pipe_config(crtc, fb, mode, state); for_each_intel_crtc_masked(dev, *modeset_pipes, intel_crtc) {
if (IS_ERR(pipe_config)) /* FIXME: For now we still expect modeset_pipes has at most
return pipe_config; * one bit set. */
if (WARN_ON(&intel_crtc->base != crtc))
continue;
intel_dump_pipe_config(to_intel_crtc(crtc), pipe_config, pipe_config = intel_modeset_pipe_config(crtc, fb, mode, state);
"[modeset]"); if (IS_ERR(pipe_config))
return pipe_config;
return pipe_config; intel_dump_pipe_config(to_intel_crtc(crtc), pipe_config,
"[modeset]");
}
return intel_atomic_get_crtc_state(state, to_intel_crtc(crtc));;
} }
static int __intel_set_mode_setup_plls(struct drm_device *dev, static int __intel_set_mode_setup_plls(struct drm_device *dev,
......
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