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 13/24] drm/i915: move detaching scalers to begin_crtc_commit
Date: Mon,  1 Jun 2015 15:27:16 +0200	[thread overview]
Message-ID: <1433165247-15928-14-git-send-email-maarten.lankhorst@linux.intel.com> (raw)
In-Reply-To: <1433165247-15928-1-git-send-email-maarten.lankhorst@linux.intel.com>

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/i915/intel_atomic.c  | 3 ---
 drivers/gpu/drm/i915/intel_display.c | 5 ++++-
 drivers/gpu/drm/i915/intel_drv.h     | 1 -
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_atomic.c b/drivers/gpu/drm/i915/intel_atomic.c
index a8202fa0daa8..dd6e6925dba0 100644
--- a/drivers/gpu/drm/i915/intel_atomic.c
+++ b/drivers/gpu/drm/i915/intel_atomic.c
@@ -144,9 +144,6 @@ int intel_atomic_commit(struct drm_device *dev,
 	for_each_crtc_in_state(state, crtc, crtc_state, i) {
 		to_intel_crtc(crtc)->config = to_intel_crtc_state(crtc->state);
 
-		if (INTEL_INFO(dev)->gen >= 9)
-			skl_detach_scalers(to_intel_crtc(crtc));
-
 		drm_atomic_helper_commit_planes_on_crtc(crtc_state);
 	}
 
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 772e70a4103a..3593a172d9ea 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -2911,7 +2911,7 @@ unsigned long intel_plane_obj_offset(struct intel_plane *intel_plane,
 /*
  * This function detaches (aka. unbinds) unused scalers in hardware
  */
-void skl_detach_scalers(struct intel_crtc *intel_crtc)
+static void skl_detach_scalers(struct intel_crtc *intel_crtc)
 {
 	struct drm_device *dev;
 	struct drm_i915_private *dev_priv;
@@ -13484,6 +13484,9 @@ static void intel_begin_crtc_commit(struct drm_crtc *crtc)
 	if (intel_crtc->atomic.wait_for_flips)
 		intel_crtc_wait_for_pending_flips(crtc);
 
+	if (INTEL_INFO(dev)->gen >= 9)
+		skl_detach_scalers(to_intel_crtc(crtc));
+
 	if (intel_crtc->atomic.disable_fbc)
 		intel_fbc_disable(dev);
 
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 4418e8abf6df..6262d9bcb59e 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -1139,7 +1139,6 @@ void intel_mode_from_pipe_config(struct drm_display_mode *mode,
 				 struct intel_crtc_state *pipe_config);
 void intel_crtc_wait_for_pending_flips(struct drm_crtc *crtc);
 void intel_modeset_preclose(struct drm_device *dev, struct drm_file *file);
-void skl_detach_scalers(struct intel_crtc *intel_crtc);
 
 int skl_update_scaler_crtc(struct intel_crtc_state *crtc_state, int force_detach);
 int skl_max_scale(struct intel_crtc *crtc, struct intel_crtc_state *crtc_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-01 13:27 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-01 13:27 [PATCH 00/24] Convert to atomic, part 3 Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 01/24] drm/i915: Always reset in intel_crtc_restore_mode Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 02/24] drm/i915: Use crtc state in intel_modeset_pipe_config Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 03/24] drm/i915: clean up intel_sanitize_crtc Maarten Lankhorst
2015-06-03  1:27   ` Matt Roper
2015-06-03  6:47     ` Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 04/24] drm/i915: Update power domains only on affected crtc's Maarten Lankhorst
2015-06-03  1:27   ` Matt Roper
2015-06-03  6:52     ` Maarten Lankhorst
2015-06-15 11:43       ` Daniel Vetter
2015-06-15 11:49         ` Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 05/24] drm/i915: add fastboot checks for has_audio and has_infoframe Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 06/24] drm/i915: Clean up intel_atomic_setup_scalers slightly Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 07/24] drm/i915: Add a simple atomic crtc check function Maarten Lankhorst
2015-06-03  1:28   ` Matt Roper
2015-06-03  6:56     ` Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 08/24] drm/i915: Do not add planes from intel_atomic_setup_scalers Maarten Lankhorst
2015-06-03  1:29   ` Matt Roper
2015-06-03  1:52     ` Konduru, Chandra
2015-06-03  7:01       ` Maarten Lankhorst
2015-06-03 19:32         ` Konduru, Chandra
2015-06-03 23:33           ` Matt Roper
2015-06-04  3:39             ` Maarten Lankhorst
2015-06-05 19:05               ` Konduru, Chandra
2015-06-06  6:39                 ` Maarten Lankhorst
2015-06-08 17:25                   ` Konduru, Chandra
2015-06-15 11:48               ` Daniel Vetter
2015-06-01 13:27 ` [PATCH 09/24] drm/i915: Assign a new pll from the crtc check function Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 10/24] drm/i915: Do not run most checks when there's no modeset Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 11/24] drm/i915: Split skl_update_scaler Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 12/24] drm/i915: Split plane updates of crtc->atomic into a helper Maarten Lankhorst
2015-06-01 13:27 ` Maarten Lankhorst [this message]
2015-06-01 13:27 ` [PATCH 14/24] drm/i915: Move crtc commit updates to separate functions Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 15/24] drm/i915: clean up plane commit functions Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 16/24] drm/i915: atomic plane updates in a nutshell Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 17/24] drm/i915: Update less state during modeset Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 18/24] drm/i915: get rid of intel_plane_restore in intel_crtc_page_flip Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 19/24] drm/i915: Make setting color key atomic Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 20/24] drm/i915: clean up atomic plane check functions Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 21/24] drm/i915: remove force argument from disable_plane Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 22/24] drm/i915: Use full atomic modeset Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 23/24] drm/i915: Unify plane updates Maarten Lankhorst
2015-06-01 13:27 ` [PATCH 24/24] drm/i915: always disable irqs in intel_pipe_update_start Maarten Lankhorst
2015-06-02  9:34 ` [PATCH 00/24] Convert to atomic, part 3 Maarten Lankhorst

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=1433165247-15928-14-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.