Commit 6ac7c548 authored by Russell King's avatar Russell King Committed by Daniel Vetter

drm: ensure atomic messages consistently include the name of the component

Most DRM messages include three pieces of information: the type of the
component (CRTC, ENCODER, CONNECTOR etc), the DRM object ID of the
component, and the component name.  However, there are some messages
which omit the last piece of identifying information.  This makes it
harder to debug failures when these messages are printed, because the
DRM object ID doesn't supply enough information to know which piece of
hardware had a problem.

Update the atomic modeset code to always print the component name along
with the type and DRM object ID.

Fixes: 4cba6850 ("drm/atomic-helper: Reject legacy flips on a disabled pipe")
Fixes: 8d4d0d70 ("drm/atomic-helper: Print an error if vblank wait times out")
Fixes: 5481c8fb ("drm/atomic-helper: Check encoder/crtc constraints")
Fixes: 99cf4a29 ("drm/atomic: Add current-mode blob to CRTC state")
Fixes: cc4ceb48 ("drm: Global atomic state handling")
Signed-off-by: default avatarRussell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Acked-by: default avatarThierry Reding <treding@nvidia.com>
[danvet: Wiggle in conflicting hunk.]
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent cd57b48a
...@@ -623,8 +623,8 @@ static int drm_atomic_crtc_check(struct drm_crtc *crtc, ...@@ -623,8 +623,8 @@ static int drm_atomic_crtc_check(struct drm_crtc *crtc,
* pipe. * pipe.
*/ */
if (state->event && !state->active && !crtc->state->active) { if (state->event && !state->active && !crtc->state->active) {
DRM_DEBUG_ATOMIC("[CRTC:%d] requesting event but off\n", DRM_DEBUG_ATOMIC("[CRTC:%d:%s] requesting event but off\n",
crtc->base.id); crtc->base.id, crtc->name);
return -EINVAL; return -EINVAL;
} }
...@@ -1037,8 +1037,9 @@ drm_atomic_get_connector_state(struct drm_atomic_state *state, ...@@ -1037,8 +1037,9 @@ drm_atomic_get_connector_state(struct drm_atomic_state *state,
state->connectors[index].ptr = connector; state->connectors[index].ptr = connector;
connector_state->state = state; connector_state->state = state;
DRM_DEBUG_ATOMIC("Added [CONNECTOR:%d] %p state to %p\n", DRM_DEBUG_ATOMIC("Added [CONNECTOR:%d:%s] %p state to %p\n",
connector->base.id, connector_state, state); connector->base.id, connector->name,
connector_state, state);
if (connector_state->crtc) { if (connector_state->crtc) {
struct drm_crtc_state *crtc_state; struct drm_crtc_state *crtc_state;
......
...@@ -322,10 +322,11 @@ update_connector_routing(struct drm_atomic_state *state, ...@@ -322,10 +322,11 @@ update_connector_routing(struct drm_atomic_state *state,
} }
if (!drm_encoder_crtc_ok(new_encoder, connector_state->crtc)) { if (!drm_encoder_crtc_ok(new_encoder, connector_state->crtc)) {
DRM_DEBUG_ATOMIC("[ENCODER:%d:%s] incompatible with [CRTC:%d]\n", DRM_DEBUG_ATOMIC("[ENCODER:%d:%s] incompatible with [CRTC:%d:%s]\n",
new_encoder->base.id, new_encoder->base.id,
new_encoder->name, new_encoder->name,
connector_state->crtc->base.id); connector_state->crtc->base.id,
connector_state->crtc->name);
return -EINVAL; return -EINVAL;
} }
...@@ -1119,7 +1120,8 @@ drm_atomic_helper_wait_for_vblanks(struct drm_device *dev, ...@@ -1119,7 +1120,8 @@ drm_atomic_helper_wait_for_vblanks(struct drm_device *dev,
drm_crtc_vblank_count(crtc), drm_crtc_vblank_count(crtc),
msecs_to_jiffies(50)); msecs_to_jiffies(50));
WARN(!ret, "[CRTC:%d] vblank wait timed out\n", crtc->base.id); WARN(!ret, "[CRTC:%d:%s] vblank wait timed out\n",
crtc->base.id, crtc->name);
drm_crtc_vblank_put(crtc); drm_crtc_vblank_put(crtc);
} }
...@@ -2792,8 +2794,8 @@ static int page_flip_common( ...@@ -2792,8 +2794,8 @@ static int page_flip_common(
/* Make sure we don't accidentally do a full modeset. */ /* Make sure we don't accidentally do a full modeset. */
state->allow_modeset = false; state->allow_modeset = false;
if (!crtc_state->active) { if (!crtc_state->active) {
DRM_DEBUG_ATOMIC("[CRTC:%d] disabled, rejecting legacy flip\n", DRM_DEBUG_ATOMIC("[CRTC:%d:%s] disabled, rejecting legacy flip\n",
crtc->base.id); crtc->base.id, crtc->name);
return -EINVAL; return -EINVAL;
} }
......
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