On Tue, 2016-11-29 at 11:24 +0200, Jani Nikula wrote:
> On Mon, 28 Nov 2016, [email protected] wrote:
> > From: Libin Yang <[email protected]>
> >
> > Add the DP MST info dump in debugfs.
> >
> > Signed-off-by: Libin Yang <[email protected]>
> > Signed-off-by: Dhinakaran Pandiyan <[email protected]>
> > Reviewed-by: Lyude <[email protected]>
> > Signed-off-by: Rodrigo Vivi <[email protected]>
> 
> I don't think Rodrigo's sob is proper here, even if he pushed the
> original version of the patch that got later reverted.

Jani is right. I haven't signed this off.
I believe the idea was to CC me probably.

> 
> Anyway,
> 
> Reviewed-by: Jani Nikula <[email protected]>
> 
> > ---
> >  drivers/gpu/drm/i915/i915_debugfs.c | 19 ++++++++++++++++++-
> >  1 file changed, 18 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c 
> > b/drivers/gpu/drm/i915/i915_debugfs.c
> > index e7f731f..f5f88b2 100644
> > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > @@ -2877,6 +2877,20 @@ static void intel_dp_info(struct seq_file *m,
> >                             &intel_dp->aux);
> >  }
> >  
> > +static void intel_dp_mst_info(struct seq_file *m,
> > +                     struct intel_connector *intel_connector)
> > +{
> > +   struct intel_encoder *intel_encoder = intel_connector->encoder;
> > +   struct intel_dp_mst_encoder *intel_mst =
> > +           enc_to_mst(&intel_encoder->base);
> > +   struct intel_digital_port *intel_dig_port = intel_mst->primary;
> > +   struct intel_dp *intel_dp = &intel_dig_port->dp;
> > +   bool has_audio = drm_dp_mst_port_has_audio(&intel_dp->mst_mgr,
> > +                                   intel_connector->port);
> > +
> > +   seq_printf(m, "\taudio support: %s\n", yesno(has_audio));
> > +}
> > +
> >  static void intel_hdmi_info(struct seq_file *m,
> >                         struct intel_connector *intel_connector)
> >  {
> > @@ -2919,7 +2933,10 @@ static void intel_connector_info(struct seq_file *m,
> >     switch (connector->connector_type) {
> >     case DRM_MODE_CONNECTOR_DisplayPort:
> >     case DRM_MODE_CONNECTOR_eDP:
> > -           intel_dp_info(m, intel_connector);
> > +           if (intel_encoder->type == INTEL_OUTPUT_DP_MST)
> > +                   intel_dp_mst_info(m, intel_connector);
> > +           else
> > +                   intel_dp_info(m, intel_connector);
> >             break;
> >     case DRM_MODE_CONNECTOR_LVDS:
> >             if (intel_encoder->type == INTEL_OUTPUT_LVDS)
> 

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to