OSDN Git Service

drm/i915: Pimp link training debug prints
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 4 Oct 2021 17:05:34 +0000 (20:05 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 6 Oct 2021 19:12:54 +0000 (22:12 +0300)
commit6c4d46523bf30772d8f68a9233853c3357bbb8ae
tree0a93f991ece551fc9c19e55f86a164523978bd54
parent1f662675335b884fe76a7899901b0761fa2e472e
drm/i915: Pimp link training debug prints

Unify all debug prints during link training to include information
on both the encoder and the LTTPR. We unify the format to something
like "[ENCODER:1:FOO][LTTPR 1] Something something". Though not
sure if those brackets around the dp_phy just make it look like
line noise? I'll accept suggestions on better formatting.

I'm slightly on the fence about also including the connector,
but technically only the DPRX is the SST connector (ie.
intel_dp->attached_connector). I suppose you could think of it
as the branch device/whatever in the topology, and we're training
the link leading to it. So that could argue for its inclusion.
But it's all getting a bit long alrady, so not going to do it
I think.

v2: Keep the connector name in the final passed/failed debug print

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211004170535.4173-5-ville.syrjala@linux.intel.com
Reviewed-by: Imre Deak <imre.deak@intel.com>
drivers/gpu/drm/i915/display/intel_dp_link_training.c