Commit 606856f0 authored by Wambui Karuga's avatar Wambui Karuga Committed by Jani Nikula

drm/i915/ring_submission: use drm_device based logging macros.

Replace the use of printk based drm logging macros to the struct
drm_device based logging macros in i915/gt/intel_ring_submission.c.
This was done using the following semantic patch that transforms based
on the existence of a drm_i915_private device:
@@
identifier fn, T;
@@

fn(...) {
...
struct drm_i915_private *T = ...;
<+...
(
-DRM_INFO(
+drm_info(&T->drm,
...)
|
-DRM_ERROR(
+drm_err(&T->drm,
...)
|
-DRM_WARN(
+drm_warn(&T->drm,
...)
|
-DRM_DEBUG_KMS(
+drm_dbg_kms(&T->drm,
...)
|
-DRM_DEBUG_DRIVER(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_ATOMIC(
+drm_dbg_atomic(&T->drm,
...)
)
...+>
}

@@
identifier fn, T;
@@

fn(...,struct drm_i915_private *T,...) {
<+...
(
-DRM_INFO(
+drm_info(&T->drm,
...)
|
-DRM_ERROR(
+drm_err(&T->drm,
...)
|
-DRM_WARN(
+drm_warn(&T->drm,
...)
|
-DRM_DEBUG_DRIVER(
+drm_dbg(&T->drm,
...)
|
-DRM_DEBUG_KMS(
+drm_dbg_kms(&T->drm,
...)
|
-DRM_DEBUG_ATOMIC(
+drm_dbg_atomic(&T->drm,
...)
)
...+>
}

New checkpatch warnings were fixed manually.

Note that this converts DRM_DEBUG_DRIVER to drm_dbg().

References: https://lists.freedesktop.org/archives/dri-devel/2020-January/253381.htmlSigned-off-by: default avatarWambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200314183344.17603-6-wambui.karugax@gmail.com
parent edf040f4
...@@ -577,8 +577,9 @@ static void flush_cs_tlb(struct intel_engine_cs *engine) ...@@ -577,8 +577,9 @@ static void flush_cs_tlb(struct intel_engine_cs *engine)
RING_INSTPM(engine->mmio_base), RING_INSTPM(engine->mmio_base),
INSTPM_SYNC_FLUSH, 0, INSTPM_SYNC_FLUSH, 0,
1000)) 1000))
DRM_ERROR("%s: wait for SyncFlush to complete for TLB invalidation timed out\n", drm_err(&dev_priv->drm,
engine->name); "%s: wait for SyncFlush to complete for TLB invalidation timed out\n",
engine->name);
} }
static void ring_setup_status_page(struct intel_engine_cs *engine) static void ring_setup_status_page(struct intel_engine_cs *engine)
...@@ -601,8 +602,9 @@ static bool stop_ring(struct intel_engine_cs *engine) ...@@ -601,8 +602,9 @@ static bool stop_ring(struct intel_engine_cs *engine)
MODE_IDLE, MODE_IDLE,
MODE_IDLE, MODE_IDLE,
1000)) { 1000)) {
DRM_ERROR("%s : timed out trying to stop ring\n", drm_err(&dev_priv->drm,
engine->name); "%s : timed out trying to stop ring\n",
engine->name);
/* /*
* Sometimes we observe that the idle flag is not * Sometimes we observe that the idle flag is not
...@@ -661,22 +663,23 @@ static int xcs_resume(struct intel_engine_cs *engine) ...@@ -661,22 +663,23 @@ static int xcs_resume(struct intel_engine_cs *engine)
/* WaClearRingBufHeadRegAtInit:ctg,elk */ /* WaClearRingBufHeadRegAtInit:ctg,elk */
if (!stop_ring(engine)) { if (!stop_ring(engine)) {
/* G45 ring initialization often fails to reset head to zero */ /* G45 ring initialization often fails to reset head to zero */
DRM_DEBUG_DRIVER("%s head not reset to zero " drm_dbg(&dev_priv->drm, "%s head not reset to zero "
"ctl %08x head %08x tail %08x start %08x\n",
engine->name,
ENGINE_READ(engine, RING_CTL),
ENGINE_READ(engine, RING_HEAD),
ENGINE_READ(engine, RING_TAIL),
ENGINE_READ(engine, RING_START));
if (!stop_ring(engine)) {
drm_err(&dev_priv->drm,
"failed to set %s head to zero "
"ctl %08x head %08x tail %08x start %08x\n", "ctl %08x head %08x tail %08x start %08x\n",
engine->name, engine->name,
ENGINE_READ(engine, RING_CTL), ENGINE_READ(engine, RING_CTL),
ENGINE_READ(engine, RING_HEAD), ENGINE_READ(engine, RING_HEAD),
ENGINE_READ(engine, RING_TAIL), ENGINE_READ(engine, RING_TAIL),
ENGINE_READ(engine, RING_START)); ENGINE_READ(engine, RING_START));
if (!stop_ring(engine)) {
DRM_ERROR("failed to set %s head to zero "
"ctl %08x head %08x tail %08x start %08x\n",
engine->name,
ENGINE_READ(engine, RING_CTL),
ENGINE_READ(engine, RING_HEAD),
ENGINE_READ(engine, RING_TAIL),
ENGINE_READ(engine, RING_START));
ret = -EIO; ret = -EIO;
goto out; goto out;
} }
...@@ -719,7 +722,7 @@ static int xcs_resume(struct intel_engine_cs *engine) ...@@ -719,7 +722,7 @@ static int xcs_resume(struct intel_engine_cs *engine)
RING_CTL(engine->mmio_base), RING_CTL(engine->mmio_base),
RING_VALID, RING_VALID, RING_VALID, RING_VALID,
50)) { 50)) {
DRM_ERROR("%s initialization failed " drm_err(&dev_priv->drm, "%s initialization failed "
"ctl %08x (valid? %d) head %08x [%08x] tail %08x [%08x] start %08x [expected %08x]\n", "ctl %08x (valid? %d) head %08x [%08x] tail %08x [%08x] start %08x [expected %08x]\n",
engine->name, engine->name,
ENGINE_READ(engine, RING_CTL), ENGINE_READ(engine, RING_CTL),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment