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

Commit 1486017f authored by Ander Conselvan de Oliveira's avatar Ander Conselvan de Oliveira Committed by Daniel Vetter
Browse files

drm/i915: Don't use encoder->new_crtc in compute_baseline_pipe_bpp()



Move towards atomic by using the legacy modeset's drm_atomic_state
instead.

Signed-off-by: default avatarAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
[danvet: Keep the if (!connector) continue; separate so that it's
easier to eventually extract a for_each_connector_in_state iterator.
And because of the upcast it's also safer.]
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 0b901879
Loading
Loading
Loading
Loading
+10 −4
Original line number Diff line number Diff line
@@ -10319,8 +10319,9 @@ compute_baseline_pipe_bpp(struct intel_crtc *crtc,
			  struct intel_crtc_state *pipe_config)
{
	struct drm_device *dev = crtc->base.dev;
	struct drm_atomic_state *state;
	struct intel_connector *connector;
	int bpp;
	int bpp, i;

	switch (fb->pixel_format) {
	case DRM_FORMAT_C8:
@@ -10360,10 +10361,15 @@ compute_baseline_pipe_bpp(struct intel_crtc *crtc,

	pipe_config->pipe_bpp = bpp;

	state = pipe_config->base.state;

	/* Clamp display bpp to EDID value */
	for_each_intel_connector(dev, connector) {
		if (!connector->new_encoder ||
		    connector->new_encoder->new_crtc != crtc)
	for (i = 0; i < state->num_connector; i++) {
		if (!state->connectors[i])
			continue;

		connector = to_intel_connector(state->connectors[i]);
		if (state->connector_states[i]->crtc != &crtc->base)
			continue;

		connected_sink_compute_bpp(connector, pipe_config);