diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2010-05-27 08:18:16 -0400 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2010-05-28 13:43:14 -0400 |
commit | a939406fda8ddc7de69ee9186356d09dc6daaa2c (patch) | |
tree | 43f09a189c8ac38b67145b0e787eb916d741848b | |
parent | 808b24d6ed8b155aac17007788390ebfde263f30 (diff) |
drm/i915: Only print "nothing to do" debug message as required.
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@chris-wilson.co.uk>
Signed-off-by: Eric Anholt <eric@anholt.net>
-rw-r--r-- | 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 1845a068cca5..e504fdb4ae31 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -1248,10 +1248,11 @@ static void intel_update_fbc(struct drm_crtc *crtc, | |||
1248 | return; | 1248 | return; |
1249 | 1249 | ||
1250 | out_disable: | 1250 | out_disable: |
1251 | DRM_DEBUG_KMS("unsupported config, disabling FBC\n"); | ||
1252 | /* Multiple disables should be harmless */ | 1251 | /* Multiple disables should be harmless */ |
1253 | if (intel_fbc_enabled(dev)) | 1252 | if (intel_fbc_enabled(dev)) { |
1253 | DRM_DEBUG_KMS("unsupported config, disabling FBC\n"); | ||
1254 | intel_disable_fbc(dev); | 1254 | intel_disable_fbc(dev); |
1255 | } | ||
1255 | } | 1256 | } |
1256 | 1257 | ||
1257 | static int | 1258 | static int |