Re: [Intel-gfx] [PATCH 27/40] drm/i915: Split a few long debug prints

2014-07-29 Thread Jesse Barnes
On Sat, 28 Jun 2014 02:04:18 +0300
ville.syrj...@linux.intel.com wrote:

 From: Ville Syrjälä ville.syrj...@linux.intel.com
 
 Split some WM debug prints to multiple lines. This shouldn't hurt
 grappability since the important part is at the start and the rest
 is just repeated stuff for each pipe.
 
 Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com
 ---
  drivers/gpu/drm/i915/intel_pm.c | 6 --
  1 file changed, 4 insertions(+), 2 deletions(-)
 
 diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
 index f0516a7..cb0b4b4 100644
 --- a/drivers/gpu/drm/i915/intel_pm.c
 +++ b/drivers/gpu/drm/i915/intel_pm.c
 @@ -1345,7 +1345,8 @@ static void valleyview_update_wm(struct drm_crtc *crtc)
   plane_sr = cursor_sr = 0;
   }
  
 - DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, B: 
 plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
 + DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, 
 +   B: plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
 planea_wm, cursora_wm,
 planeb_wm, cursorb_wm,
 plane_sr, cursor_sr);
 @@ -1397,7 +1398,8 @@ static void g4x_update_wm(struct drm_crtc *crtc)
   plane_sr = cursor_sr = 0;
   }
  
 - DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, B: 
 plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
 + DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, 
 +   B: plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
 planea_wm, cursora_wm,
 planeb_wm, cursorb_wm,
 plane_sr, cursor_sr);

Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org

-- 
Jesse Barnes, Intel Open Source Technology Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx


[Intel-gfx] [PATCH 27/40] drm/i915: Split a few long debug prints

2014-06-27 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com

Split some WM debug prints to multiple lines. This shouldn't hurt
grappability since the important part is at the start and the rest
is just repeated stuff for each pipe.

Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com
---
 drivers/gpu/drm/i915/intel_pm.c | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index f0516a7..cb0b4b4 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -1345,7 +1345,8 @@ static void valleyview_update_wm(struct drm_crtc *crtc)
plane_sr = cursor_sr = 0;
}
 
-   DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, B: 
plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
+   DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, 
+ B: plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
  planea_wm, cursora_wm,
  planeb_wm, cursorb_wm,
  plane_sr, cursor_sr);
@@ -1397,7 +1398,8 @@ static void g4x_update_wm(struct drm_crtc *crtc)
plane_sr = cursor_sr = 0;
}
 
-   DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, B: 
plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
+   DRM_DEBUG_KMS(Setting FIFO watermarks - A: plane=%d, cursor=%d, 
+ B: plane=%d, cursor=%d, SR: plane=%d, cursor=%d\n,
  planea_wm, cursora_wm,
  planeb_wm, cursorb_wm,
  plane_sr, cursor_sr);
-- 
1.8.5.5

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