Commit ac1b49be authored by Jani Nikula's avatar Jani Nikula

drm/i915/display: change who adds [] around crtc state dump context string

Add the brackets [] around crtc state dump context string in
intel_crtc_state_dump() so the callers don't have to.
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/c7d671279fb7d99eaf882bcb88c5c1d653755fb1.1655372759.git.jani.nikula@intel.com
parent 3e29d3b3
...@@ -154,7 +154,7 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config, ...@@ -154,7 +154,7 @@ void intel_crtc_state_dump(const struct intel_crtc_state *pipe_config,
char buf[64]; char buf[64];
int i; int i;
drm_dbg_kms(&dev_priv->drm, "[CRTC:%d:%s] enable: %s %s\n", drm_dbg_kms(&dev_priv->drm, "[CRTC:%d:%s] enable: %s [%s]\n",
crtc->base.base.id, crtc->base.name, crtc->base.base.id, crtc->base.name,
str_yes_no(pipe_config->hw.enable), context); str_yes_no(pipe_config->hw.enable), context);
......
...@@ -7091,7 +7091,7 @@ static int intel_atomic_check(struct drm_device *dev, ...@@ -7091,7 +7091,7 @@ static int intel_atomic_check(struct drm_device *dev,
intel_crtc_state_dump(new_crtc_state, state, intel_crtc_state_dump(new_crtc_state, state,
intel_crtc_needs_modeset(new_crtc_state) ? intel_crtc_needs_modeset(new_crtc_state) ?
"[modeset]" : "[fastset]"); "modeset" : "fastset");
} }
return 0; return 0;
...@@ -7106,7 +7106,7 @@ static int intel_atomic_check(struct drm_device *dev, ...@@ -7106,7 +7106,7 @@ static int intel_atomic_check(struct drm_device *dev,
*/ */
for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state, for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,
new_crtc_state, i) new_crtc_state, i)
intel_crtc_state_dump(new_crtc_state, state, "[failed]"); intel_crtc_state_dump(new_crtc_state, state, "failed");
return ret; return ret;
} }
...@@ -9595,7 +9595,7 @@ intel_modeset_setup_hw_state(struct drm_device *dev, ...@@ -9595,7 +9595,7 @@ intel_modeset_setup_hw_state(struct drm_device *dev,
to_intel_crtc_state(crtc->base.state); to_intel_crtc_state(crtc->base.state);
intel_sanitize_crtc(crtc, ctx); intel_sanitize_crtc(crtc, ctx);
intel_crtc_state_dump(crtc_state, NULL, "[setup_hw_state]"); intel_crtc_state_dump(crtc_state, NULL, "setup_hw_state");
} }
intel_modeset_update_connector_atomic_state(dev); intel_modeset_update_connector_atomic_state(dev);
......
...@@ -217,8 +217,8 @@ verify_crtc_state(struct intel_crtc *crtc, ...@@ -217,8 +217,8 @@ verify_crtc_state(struct intel_crtc *crtc,
if (!intel_pipe_config_compare(new_crtc_state, if (!intel_pipe_config_compare(new_crtc_state,
pipe_config, false)) { pipe_config, false)) {
I915_STATE_WARN(1, "pipe state doesn't match!\n"); I915_STATE_WARN(1, "pipe state doesn't match!\n");
intel_crtc_state_dump(pipe_config, NULL, "[hw state]"); intel_crtc_state_dump(pipe_config, NULL, "hw state");
intel_crtc_state_dump(new_crtc_state, NULL, "[sw state]"); intel_crtc_state_dump(new_crtc_state, NULL, "sw state");
} }
} }
......
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