diff mbox

[2/2] drm/i915: be less verbose when handling gmbus/aux irqs

Message ID 1354794292-3914-2-git-send-email-przanoni@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Paulo Zanoni Dec. 6, 2012, 11:44 a.m. UTC 
From: Paulo Zanoni <paulo.r.zanoni@intel.com>

Having 9500 lines repeated on dmesg does not help me at all.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
---
 drivers/gpu/drm/i915/i915_irq.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Daniel Vetter Dec. 6, 2012, 12:25 p.m. UTC | #1 
On Thu, Dec 06, 2012 at 09:44:52AM -0200, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni@intel.com>
> 
> Having 9500 lines repeated on dmesg does not help me at all.
> 
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Queued for -next, thanks for the patch. In general we might need to
rethink our debug logging, we just dump massive amounts of stuff into
dmesg, and it's getting unwielding imo ...
-Daniel
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 58bb11b..2e1d80d 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -529,8 +529,6 @@  static void gmbus_irq_handler(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = (drm_i915_private_t *) dev->dev_private;
 
-	DRM_DEBUG_DRIVER("GMBUS interrupt\n");
-
 	wake_up_all(&dev_priv->gmbus_wait_queue);
 }
 
@@ -538,8 +536,6 @@  static void dp_aux_irq_handler(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = (drm_i915_private_t *) dev->dev_private;
 
-	DRM_DEBUG_DRIVER("AUX channel interrupt\n");
-
 	wake_up_all(&dev_priv->gmbus_wait_queue);
 }