On Wed, 12 Dec 2012, Damien Lespiau <[email protected]> wrote: > From: Damien Lespiau <[email protected]> > > It's a bit useless to print out an all null DPCD when we are > disconnected and just clutter the debug logs.
NAK. Please have a look at intel_dp_detect_dpcd(). There are a number of ways to get a non-zero DPCD that lead to something other than connector_status_connected. The comment says, "XXX this is probably wrong for multiple downstream ports", so there's also some work to do in the area. IMO we need the debugs. Even intel_dp_get_dpcd() has one branch that returns false when we have non-zero DPCD. But if we ignored that part (it's an aux channel error anyway), we could move the debug dump to intel_dp_detect_dpcd() right after the intel_dp_get_dpcd() call. I think that would get maximum debug coverage with minimum zero DPCD noise. What do you think? BR, Jani. > > Signed-off-by: Damien Lespiau <[email protected]> > --- > drivers/gpu/drm/i915/intel_dp.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index fe3c22f..42058fa 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -2362,13 +2362,13 @@ intel_dp_detect(struct drm_connector *connector, bool > force) > else > status = g4x_dp_detect(intel_dp); > > + if (status != connector_status_connected) > + return status; > + > hex_dump_to_buffer(intel_dp->dpcd, sizeof(intel_dp->dpcd), > 32, 1, dpcd_hex_dump, sizeof(dpcd_hex_dump), false); > DRM_DEBUG_KMS("DPCD: %s\n", dpcd_hex_dump); > > - if (status != connector_status_connected) > - return status; > - > intel_dp_probe_oui(intel_dp); > > if (intel_dp->force_audio != HDMI_AUDIO_AUTO) { > -- > 1.7.11.7 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
