All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: [PATCH v3 18/19] drm/i915: Remove transitional references from intel_plane_atomic_check.
Date: Mon, 15 Jun 2015 12:33:55 +0200	[thread overview]
Message-ID: <1434364436-6366-19-git-send-email-maarten.lankhorst@linux.intel.com> (raw)
In-Reply-To: <1434364436-6366-1-git-send-email-maarten.lankhorst@linux.intel.com>

All transitional plane helpers are gone, party!

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/i915/intel_atomic_plane.c | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_atomic_plane.c b/drivers/gpu/drm/i915/intel_atomic_plane.c
index 10a8ecedc942..f1ab8e4b9c11 100644
--- a/drivers/gpu/drm/i915/intel_atomic_plane.c
+++ b/drivers/gpu/drm/i915/intel_atomic_plane.c
@@ -115,6 +115,7 @@ static int intel_plane_atomic_check(struct drm_plane *plane,
 	struct intel_crtc_state *crtc_state;
 	struct intel_plane *intel_plane = to_intel_plane(plane);
 	struct intel_plane_state *intel_state = to_intel_plane_state(state);
+	struct drm_crtc_state *drm_crtc_state;
 	int ret;
 
 	crtc = crtc ? crtc : plane->state->crtc;
@@ -129,19 +130,11 @@ static int intel_plane_atomic_check(struct drm_plane *plane,
 	if (!crtc)
 		return 0;
 
-	/* FIXME: temporary hack necessary while we still use the plane update
-	 * helper. */
-	if (state->state) {
-		struct drm_crtc_state *drm_crtc_state =
-			drm_atomic_get_existing_crtc_state(state->state, crtc);
+	drm_crtc_state = drm_atomic_get_existing_crtc_state(state->state, crtc);
+	if (WARN_ON(!drm_crtc_state))
+		return -EINVAL;
 
-		if (WARN_ON(!drm_crtc_state))
-			return -EINVAL;
-
-		crtc_state = to_intel_crtc_state(drm_crtc_state);
-	} else {
-		crtc_state = intel_crtc->config;
-	}
+	crtc_state = to_intel_crtc_state(drm_crtc_state);
 
 	/*
 	 * The original src/dest coordinates are stored in state->base, but
@@ -191,7 +184,7 @@ static int intel_plane_atomic_check(struct drm_plane *plane,
 
 	intel_state->visible = false;
 	ret = intel_plane->check_plane(plane, crtc_state, intel_state);
-	if (ret || !state->state)
+	if (ret)
 		return ret;
 
 	return intel_plane_atomic_calc_changes(&crtc_state->base, state);
-- 
2.1.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  parent reply	other threads:[~2015-06-15 10:34 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-15 10:33 [PATCH v3 00/19] Convert to atomic, part 3 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 01/19] drm/i915: Use crtc state in intel_modeset_pipe_config Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 02/19] drm/i915: Clean up intel_atomic_setup_scalers slightly Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 03/19] drm/i915: Add a simple atomic crtc check function, v2 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 04/19] drm/i915: Move scaler setup to check crtc " Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 05/19] drm/i915: Assign a new pll from the crtc check " Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 06/19] drm/i915: Split skl_update_scaler, v3 Maarten Lankhorst
2015-06-18  1:48   ` Matt Roper
2015-06-18  5:42     ` Maarten Lankhorst
2015-06-18  6:51       ` Daniel Vetter
2015-06-22  7:50         ` [PATCH v3.5 06/19] drm/i915: Split skl_update_scaler, v4 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 07/19] drm/i915: Split plane updates of crtc->atomic into a helper, v2 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 08/19] drm/i915: clean up plane commit functions Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 09/19] drm/i915: clean up atomic plane check functions, v2 Maarten Lankhorst
2015-06-18  1:48   ` Matt Roper
2015-06-22  7:23     ` Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 10/19] drm/i915: remove force argument from disable_plane Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 11/19] drm/i915: move detaching scalers to begin_crtc_commit, v2 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 12/19] drm/i915: Move crtc commit updates to separate functions Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 13/19] drm/i915: Do not run most checks when there's no modeset Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 14/19] drm/i915: Handle disabling planes better, v2 Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 15/19] drm/i915: atomic plane updates in a nutshell Maarten Lankhorst
2015-06-18 14:21   ` Matt Roper
2015-06-19  3:57     ` Maarten Lankhorst
2015-06-18 15:28   ` Ville Syrjälä
2015-06-19  4:01     ` Maarten Lankhorst
2015-06-22  9:49       ` Ville Syrjälä
2015-06-22 10:08         ` Maarten Lankhorst
2015-06-22 12:33           ` Daniel Vetter
2015-06-15 10:33 ` [PATCH v3 16/19] drm/i915: Update less state during modeset Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 17/19] drm/i915: Make setting color key atomic Maarten Lankhorst
2015-06-18 14:21   ` Matt Roper
2015-06-19  4:03     ` Maarten Lankhorst
2015-06-15 10:33 ` Maarten Lankhorst [this message]
2015-06-18 14:21   ` [PATCH v3 18/19] drm/i915: Remove transitional references from intel_plane_atomic_check Matt Roper
2015-06-19  4:05     ` Maarten Lankhorst
2015-06-15 10:33 ` [PATCH v3 19/19] drm/i915: Make cdclk part of the atomic state Maarten Lankhorst
2015-06-19 22:29 ` [PATCH v3 00/19] Convert to atomic, part 3 Matt Roper
2015-06-22 12:31   ` Daniel Vetter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1434364436-6366-19-git-send-email-maarten.lankhorst@linux.intel.com \
    --to=maarten.lankhorst@linux.intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.