<div dir="ltr">On Mon, Mar 24, 2014 at 4:53 PM, Damien Lespiau <span dir="ltr"><<a href="mailto:damien.lespiau@intel.com" target="_blank">damien.lespiau@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">As patch 8/11 explains, I noticed that we where evaluating the arguments to<br>
drm_ut_debug_printk() even when drm.debug was 0, doing some work for no good<br>
reason. By pulling the test on drm_debug before calling drm_ut_debug_printk(),<br>
we skip those instructions that only need to be executed when logging is on.<br>
<br>
The remaining patches are bonus clean-ups, with the main goal of removing<br>
DRM_LOG* macros that are not really used throughout the code base. After that,<br>
it's possible to simplify a bit drm_ut_debug_printk(). All pretty<br>
straightforward cleanups, but still worthwhile IMHO.<br>
<br>
For driver-specific patches (why some of you are Cced in that series), I'd love<br>
if you could take the time to throw a Acked-by/Reviewed-by tag. Also, do you<br>
have any objection if the driver specific patch go through the DRM tree?<br>
should people judge that series worthwhile, of course.<br>
<br></blockquote><div><br>I'm ok with this going through the DRM tree.<br></div><div><br>Reviewed-by: Patrik Jakobsson <<a href="mailto:patrik.r.jakobsson@gmail.com">patrik.r.jakobsson@gmail.com</a>><br> <br></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--<br>
Damien<br>
<br>
Damien Lespiau (11):<br>
drm: Refresh the explanation of debug categories<br>
drm: Remove the unused (and unusable) DRM_LOG_MODE()<br>
drm/exynos: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()<br>
drm/gma500: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()<br>
drm/i915: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()<br>
staging: imx-drm: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()<br>
drm: Remove the now unused DRM_LOG* macros<br>
drm: Pull the test on drm_debug in the logging macros<br>
drm: drm_ut_debug_printk() isn't called with NULL anywmore<br>
drm: Remove the prefix argument of drm_ut_debug_printk()<br>
drm: Remove the ',' after the function name in debug logs<br>
<br>
drivers/gpu/drm/drm_stub.c | 24 +++----<br>
drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +-<br>
drivers/gpu/drm/exynos/exynos_drm_plane.c | 2 +-<br>
drivers/gpu/drm/gma500/psb_intel_sdvo.c | 20 +++---<br>
drivers/gpu/drm/i915/dvo_ch7xxx.c | 4 +-<br>
drivers/gpu/drm/i915/dvo_ivch.c | 30 ++++-----<br>
drivers/gpu/drm/i915/dvo_ns2501.c | 10 +--<br>
drivers/gpu/drm/i915/dvo_sil164.c | 10 +--<br>
drivers/gpu/drm/i915/dvo_tfp410.c | 24 +++----<br>
drivers/staging/imx-drm/ipuv3-plane.c | 2 +-<br>
include/drm/drmP.h | 106 +++++++++++-------------------<br>
11 files changed, 98 insertions(+), 136 deletions(-)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.8.3.1<br>
<br>
</font></span></blockquote></div><br></div></div>