Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4ed9fb37 authored by Ander Conselvan de Oliveira's avatar Ander Conselvan de Oliveira Committed by Jani Nikula
Browse files

drm/i915: Don't set enabled value of all CRTCs when restoring the mode 

The code in intel_crtc_restore_mode() sets the enabled value of all the
CRTCs when restoring the mode after a suspend/resume cycle. When more
than one CRTC is enabled, that causes drm_atomic_helper_check_modeset()
to fail if there is more than one pipe enabled, since all but one CRTC
has valid connector data. Instead, set only the enabled value for the
CRTC passed as an argument.

v2: Don't leak atomic state. (Matt)

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90468
References: https://bugs.freedesktop.org/show_bug.cgi?id=90396


Signed-off-by: default avatarAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Reviewed-by: default avatarMatt Roper <matthew.d.roper@intel.com>
parent 0d26fb89

drivers/gpu/drm/i915/intel_display.c

+10 −17
Original line number Diff line number Diff line
@@ -12683,7 +12683,6 @@ void intel_crtc_restore_mode(struct drm_crtc *crtc) 
{

	struct drm_device *dev = crtc->dev;

	struct drm_atomic_state *state;

	struct intel_crtc *intel_crtc;

	struct intel_encoder *encoder;

	struct intel_connector *connector;

	struct drm_connector_state *connector_state;
@@ -12726,24 +12725,18 @@ void intel_crtc_restore_mode(struct drm_crtc *crtc) 
		}

	}



	for_each_intel_crtc(dev, intel_crtc) {

		if (intel_crtc->new_enabled == intel_crtc->base.enabled)

			continue;



		crtc_state = intel_atomic_get_crtc_state(state, intel_crtc);

	crtc_state = intel_atomic_get_crtc_state(state, to_intel_crtc(crtc));

	if (IS_ERR(crtc_state)) {

		DRM_DEBUG_KMS("Failed to add [CRTC:%d] to state: %ld\n",

				      intel_crtc->base.base.id,

				      PTR_ERR(crtc_state));

			continue;

			      crtc->base.id, PTR_ERR(crtc_state));

		drm_atomic_state_free(state);

		return;

	}



	crtc_state->base.active = crtc_state->base.enable =

			intel_crtc->new_enabled;

		to_intel_crtc(crtc)->new_enabled;



		if (&intel_crtc->base == crtc)

	drm_mode_copy(&crtc_state->base.mode, &crtc->mode);

	}



	intel_modeset_setup_plane_state(state, crtc, &crtc->mode,

					crtc->primary->fb, crtc->x, crtc->y);