Commit ed7951dc13aad4a14695ec8122e9f0e2ef25d39e

Authored by Lespiau, Damien
Committed by Dave Airlie
1 parent b9434d0f16

drm: Make the HPD status updates debug logs more readable

Instead of just printing "status updated from 1 to 2", make those enum
numbers immediately readable.

v2: Also patch output_poll_execute() (Daniel Vetter)
v3: Use drm_get_connector_status_name (Ville Syrjälä)

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org> (for v1)
Signed-off-by: Dave Airlie <airlied@redhat.com>

Showing 2 changed files with 7 additions and 4 deletions Side-by-side Diff

drivers/gpu/drm/drm_crtc.c
... ... @@ -250,6 +250,7 @@
250 250 else
251 251 return "unknown";
252 252 }
  253 +EXPORT_SYMBOL(drm_get_connector_status_name);
253 254  
254 255 /**
255 256 * drm_mode_object_get - allocate a new modeset identifier
drivers/gpu/drm/drm_crtc_helper.c
... ... @@ -1007,10 +1007,11 @@
1007 1007 continue;
1008 1008  
1009 1009 connector->status = connector->funcs->detect(connector, false);
1010   - DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %d to %d\n",
  1010 + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s\n",
1011 1011 connector->base.id,
1012 1012 drm_get_connector_name(connector),
1013   - old_status, connector->status);
  1013 + drm_get_connector_status_name(old_status),
  1014 + drm_get_connector_status_name(connector->status));
1014 1015 if (old_status != connector->status)
1015 1016 changed = true;
1016 1017 }
1017 1018  
... ... @@ -1085,10 +1086,11 @@
1085 1086 old_status = connector->status;
1086 1087  
1087 1088 connector->status = connector->funcs->detect(connector, false);
1088   - DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %d to %d\n",
  1089 + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s\n",
1089 1090 connector->base.id,
1090 1091 drm_get_connector_name(connector),
1091   - old_status, connector->status);
  1092 + drm_get_connector_status_name(old_status),
  1093 + drm_get_connector_status_name(connector->status));
1092 1094 if (old_status != connector->status)
1093 1095 changed = true;
1094 1096 }