drm/i915: add trailing newlines to msgs
authorJim Cromie <jim.cromie@gmail.com>
Sun, 3 Sep 2023 18:46:00 +0000 (12:46 -0600)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Wed, 6 Sep 2023 16:38:29 +0000 (12:38 -0400)
By at least strong convention, a print-buffer's trailing newline says
"message complete, send it".  The exception (no TNL, followed by a call
to pr_cont) proves the general rule.

Most DRM.debug calls already comport with this: 207 DRM_DEV_DEBUG,
1288 drm_dbg.  Clean up the remainders, in maintainer sized chunks.

No functional changes.

Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
[Rodrigo changed subject while pushing]
Link: https://patchwork.freedesktop.org/patch/msgid/20230903184607.272198-4-jim.cromie@gmail.com
drivers/gpu/drm/i915/display/intel_ddi.c
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c

index b7f4281..690e1e0 100644 (file)
@@ -4173,7 +4173,7 @@ static int intel_ddi_compute_config_late(struct intel_encoder *encoder,
        struct drm_connector *connector = conn_state->connector;
        u8 port_sync_transcoders = 0;
 
-       drm_dbg_kms(&i915->drm, "[ENCODER:%d:%s] [CRTC:%d:%s]",
+       drm_dbg_kms(&i915->drm, "[ENCODER:%d:%s] [CRTC:%d:%s]\n",
                    encoder->base.base.id, encoder->base.name,
                    crtc_state->uapi.crtc->base.id, crtc_state->uapi.crtc->name);
 
index 5a687a3..7d04ec7 100644 (file)
@@ -1436,7 +1436,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
        if (unlikely(reloc->write_domain & (reloc->write_domain - 1))) {
                drm_dbg(&i915->drm, "reloc with multiple write domains: "
                          "target %d offset %d "
-                         "read %08x write %08x",
+                         "read %08x write %08x\n",
                          reloc->target_handle,
                          (int) reloc->offset,
                          reloc->read_domains,
@@ -1447,7 +1447,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
                     & ~I915_GEM_GPU_DOMAINS)) {
                drm_dbg(&i915->drm, "reloc with read/write non-GPU domains: "
                          "target %d offset %d "
-                         "read %08x write %08x",
+                         "read %08x write %08x\n",
                          reloc->target_handle,
                          (int) reloc->offset,
                          reloc->read_domains,