drm/i915: use drm_debug_enabled() to check for debug categories
authorJani Nikula <jani.nikula@intel.com>
Mon, 28 Oct 2019 10:38:15 +0000 (12:38 +0200)
committerJani Nikula <jani.nikula@intel.com>
Thu, 14 Nov 2019 12:08:19 +0000 (14:08 +0200)
Allow better abstraction of the drm_debug global variable in the
future. No functional changes.

Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Acked-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Acked-by: Sean Paul <sean@poorly.run>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/e94fe4977c5b8cac68556318be81f8e422e973fd.1572258936.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_display.c
drivers/gpu/drm/i915/display/intel_dp.c
drivers/gpu/drm/i915/i915_drv.c
drivers/gpu/drm/i915/i915_gem.h
drivers/gpu/drm/i915/i915_utils.c
drivers/gpu/drm/i915/intel_pm.c

index 2912abd85148d9c01469f575cc74bb180698e669..ee9e74a517804f81913016138bbbfbce5e8a38a7 100644 (file)
@@ -12213,7 +12213,7 @@ static void
 intel_dump_infoframe(struct drm_i915_private *dev_priv,
                     const union hdmi_infoframe *frame)
 {
-       if ((drm_debug & DRM_UT_KMS) == 0)
+       if (!drm_debug_enabled(DRM_UT_KMS))
                return;
 
        hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, frame);
@@ -12745,7 +12745,7 @@ pipe_config_infoframe_mismatch(struct drm_i915_private *dev_priv,
                               const union hdmi_infoframe *b)
 {
        if (fastset) {
-               if ((drm_debug & DRM_UT_KMS) == 0)
+               if (!drm_debug_enabled(DRM_UT_KMS))
                        return;
 
                DRM_DEBUG_KMS("fastset mismatch in %s infoframe\n", name);
index 5eeafa45831a567ed4df74fe4504925b36d14443..aa515261cb9f33ed7027708f74ea0acd2cc8d413 100644 (file)
@@ -1808,7 +1808,7 @@ static void intel_dp_print_rates(struct intel_dp *intel_dp)
 {
        char str[128]; /* FIXME: too big for stack? */
 
-       if ((drm_debug & DRM_UT_KMS) == 0)
+       if (!drm_debug_enabled(DRM_UT_KMS))
                return;
 
        snprintf_int_array(str, sizeof(str),
index 157ed22052a25176220e250e498a6563d80663f8..c795fa663bba969f095a12742a7dd65216deb2d2 100644 (file)
@@ -1382,7 +1382,7 @@ static void i915_driver_unregister(struct drm_i915_private *dev_priv)
 
 static void i915_welcome_messages(struct drm_i915_private *dev_priv)
 {
-       if (drm_debug & DRM_UT_DRIVER) {
+       if (drm_debug_enabled(DRM_UT_DRIVER)) {
                struct drm_printer p = drm_debug_printer("i915 device info:");
 
                drm_printf(&p, "pciid=0x%04x rev=0x%02x platform=%s (subplatform=0x%x) gen=%i\n",
index 2011f8e9a9f1fa6fb65b9442bc26d19b8e4a095a..05d23d013b683edb6ee79dfd697917382ec295f1 100644 (file)
@@ -34,7 +34,7 @@ struct drm_i915_private;
 
 #ifdef CONFIG_DRM_I915_DEBUG_GEM
 
-#define GEM_SHOW_DEBUG() (drm_debug & DRM_UT_DRIVER)
+#define GEM_SHOW_DEBUG() drm_debug_enabled(DRM_UT_DRIVER)
 
 #define GEM_BUG_ON(condition) do { if (unlikely((condition))) {        \
                GEM_TRACE_ERR("%s:%d GEM_BUG_ON(%s)\n", \
index 16acdf7bdbe6a659289cca53b1a29ee4b73fd97a..f66540e157933894dbde34d1c7accc3a4a9b58bd 100644 (file)
@@ -23,7 +23,7 @@ __i915_printk(struct drm_i915_private *dev_priv, const char *level,
        struct va_format vaf;
        va_list args;
 
-       if (is_debug && !(drm_debug & DRM_UT_DRIVER))
+       if (is_debug && !drm_debug_enabled(DRM_UT_DRIVER))
                return;
 
        va_start(args, fmt);
index 3622344490874f6135c5ffc5aeee43d4eaed8cde..08255a96e1ea80ebd6fb06d86de72a52d9e9c912 100644 (file)
@@ -5307,7 +5307,7 @@ skl_print_wm_changes(struct intel_atomic_state *state)
        struct intel_crtc *crtc;
        int i;
 
-       if ((drm_debug & DRM_UT_KMS) == 0)
+       if (!drm_debug_enabled(DRM_UT_KMS))
                return;
 
        for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,