[Intel-gfx] [PATCH 06/37] drm/i915: Only print "nothing to do" debug message as required.

Chris Wilson chris at chris-wilson.co.uk
Wed Mar 10 23:44:53 CET 2010


If the FBC is already disabled, then we do not even attempt to disable
FBC and so there is no point emitting a debug statement at that point,
having already emitted one saying why we are disabling FBC.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/intel_display.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 979c628..3ee91c8 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -1233,10 +1233,11 @@ static void intel_update_fbc(struct drm_crtc *crtc,
 	return;
 
 out_disable:
-	DRM_DEBUG_KMS("unsupported config, disabling FBC\n");
 	/* Multiple disables should be harmless */
-	if (dev_priv->display.fbc_enabled(crtc))
+	if (dev_priv->display.fbc_enabled(crtc)) {
+		DRM_DEBUG_KMS("unsupported config, disabling FBC\n");
 		dev_priv->display.disable_fbc(dev);
+	}
 }
 
 static int
-- 
1.7.0




More information about the Intel-gfx mailing list