drm/i915/sideband: convert to using new struct drm_device logging macros
authorWambui Karuga <wambui.karugax@gmail.com>
Tue, 7 Jan 2020 15:13:32 +0000 (18:13 +0300)
committerJani Nikula <jani.nikula@intel.com>
Fri, 10 Jan 2020 14:11:48 +0000 (16:11 +0200)
Replace the use of printk based debugging macros with the struct
drm_device based logging macros in i915/intel_sideband.c.

Signed-off-by: Wambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/ae253ecf3ca878fae7f1f246d75c2136fb6bd72c.1578409433.git.wambui.karugax@gmail.com
drivers/gpu/drm/i915/intel_sideband.c

index e06b35b..cbfb717 100644 (file)
@@ -105,8 +105,8 @@ static int vlv_sideband_rw(struct drm_i915_private *i915,
        if (intel_wait_for_register(uncore,
                                    VLV_IOSF_DOORBELL_REQ, IOSF_SB_BUSY, 0,
                                    5)) {
-               DRM_DEBUG_DRIVER("IOSF sideband idle wait (%s) timed out\n",
-                                is_read ? "read" : "write");
+               drm_dbg(&i915->drm, "IOSF sideband idle wait (%s) timed out\n",
+                       is_read ? "read" : "write");
                return -EAGAIN;
        }
 
@@ -129,8 +129,8 @@ static int vlv_sideband_rw(struct drm_i915_private *i915,
                        *val = intel_uncore_read_fw(uncore, VLV_IOSF_DATA);
                err = 0;
        } else {
-               DRM_DEBUG_DRIVER("IOSF sideband finish wait (%s) timed out\n",
-                                is_read ? "read" : "write");
+               drm_dbg(&i915->drm, "IOSF sideband finish wait (%s) timed out\n",
+                       is_read ? "read" : "write");
                err = -ETIMEDOUT;
        }
 
@@ -283,7 +283,8 @@ static int intel_sbi_rw(struct drm_i915_private *i915, u16 reg,
        if (intel_wait_for_register_fw(uncore,
                                       SBI_CTL_STAT, SBI_BUSY, 0,
                                       100)) {
-               DRM_ERROR("timeout waiting for SBI to become ready\n");
+               drm_err(&i915->drm,
+                       "timeout waiting for SBI to become ready\n");
                return -EBUSY;
        }
 
@@ -301,12 +302,13 @@ static int intel_sbi_rw(struct drm_i915_private *i915, u16 reg,
        if (__intel_wait_for_register_fw(uncore,
                                         SBI_CTL_STAT, SBI_BUSY, 0,
                                         100, 100, &cmd)) {
-               DRM_ERROR("timeout waiting for SBI to complete read\n");
+               drm_err(&i915->drm,
+                       "timeout waiting for SBI to complete read\n");
                return -ETIMEDOUT;
        }
 
        if (cmd & SBI_RESPONSE_FAIL) {
-               DRM_ERROR("error during SBI read of reg %x\n", reg);
+               drm_err(&i915->drm, "error during SBI read of reg %x\n", reg);
                return -ENXIO;
        }
 
@@ -426,8 +428,9 @@ int sandybridge_pcode_read(struct drm_i915_private *i915, u32 mbox,
        mutex_unlock(&i915->sb_lock);
 
        if (err) {
-               DRM_DEBUG_DRIVER("warning: pcode (read from mbox %x) mailbox access failed for %ps: %d\n",
-                                mbox, __builtin_return_address(0), err);
+               drm_dbg(&i915->drm,
+                       "warning: pcode (read from mbox %x) mailbox access failed for %ps: %d\n",
+                       mbox, __builtin_return_address(0), err);
        }
 
        return err;
@@ -447,8 +450,9 @@ int sandybridge_pcode_write_timeout(struct drm_i915_private *i915,
        mutex_unlock(&i915->sb_lock);
 
        if (err) {
-               DRM_DEBUG_DRIVER("warning: pcode (write of 0x%08x to mbox %x) mailbox access failed for %ps: %d\n",
-                                val, mbox, __builtin_return_address(0), err);
+               drm_dbg(&i915->drm,
+                       "warning: pcode (write of 0x%08x to mbox %x) mailbox access failed for %ps: %d\n",
+                       val, mbox, __builtin_return_address(0), err);
        }
 
        return err;
@@ -519,7 +523,8 @@ int skl_pcode_request(struct drm_i915_private *i915, u32 mbox, u32 request,
         * requests, and for any quirks of the PCODE firmware that delays
         * the request completion.
         */
-       DRM_DEBUG_KMS("PCODE timeout, retrying with preemption disabled\n");
+       drm_dbg_kms(&i915->drm,
+                   "PCODE timeout, retrying with preemption disabled\n");
        WARN_ON_ONCE(timeout_base_ms > 3);
        preempt_disable();
        ret = wait_for_atomic(COND, 50);