Re: [PATCH] drm/i915: add port info to debugfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2019-08-26 at 16:10 +0300, Jani Nikula wrote:
> On Thu, 22 Aug 2019, Simon Ser <simon.ser@xxxxxxxxx> wrote:
> > This patch adds a line with the port name to connectors in
> > debugfs/i915_debug_info. If the port is Type-C, the Type-C port
> > number is
> > printed too.
> > 
> > Signed-off-by: Simon Ser <simon.ser@xxxxxxxxx>
> > Cc: Imre Deak <imre.deak@xxxxxxxxx>
> > Cc: Manasi Navare <nabasi.d.navare@xxxxxxxxx>
> > ---
> > 
> > Thanks for your comments, Imre and Manasi. Here is v2:
> > 
> > - Use same port formatting as intel_tc_port_init (e.g. "C/TC#1")
> > - Fix typo in commit message
> 
> Usually in drm we put the patch changelog as part of the commit
> message.

Err, ack.

> Please resend the patch to intel-gfx@, and you'll get our CI results
> too.

Yes, sorry about that. I've re-submitted the series here:
https://patchwork.freedesktop.org/series/65695/

> 
> BR,
> Jani.
> 
> 
> 
> >  drivers/gpu/drm/i915/i915_debugfs.c | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
> > b/drivers/gpu/drm/i915/i915_debugfs.c
> > index b39226d7f8d2..4ba508c954f8 100644
> > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > @@ -2557,9 +2557,11 @@ static void intel_lvds_info(struct seq_file
> > *m,
> >  static void intel_connector_info(struct seq_file *m,
> >  				 struct drm_connector *connector)
> >  {
> > +	struct drm_i915_private *i915 = to_i915(connector->dev);
> >  	struct intel_connector *intel_connector =
> > to_intel_connector(connector);
> >  	struct intel_encoder *intel_encoder = intel_connector->encoder;
> >  	struct drm_display_mode *mode;
> > +	enum tc_port tc_port;
> > 
> >  	seq_printf(m, "connector %d: type %s, status: %s\n",
> >  		   connector->base.id, connector->name,
> > @@ -2578,6 +2580,14 @@ static void intel_connector_info(struct
> > seq_file *m,
> >  	if (!intel_encoder)
> >  		return;
> > 
> > +	if (intel_encoder->port != PORT_NONE) {
> > +		seq_printf(m, "\tport: %c", port_name(intel_encoder-
> > >port));
> > +		tc_port = intel_port_to_tc(i915, intel_encoder->port);
> > +		if (tc_port != PORT_TC_NONE)
> > +			seq_printf(m, "/TC#%d", tc_port + 1);
> > +		seq_printf(m, "\n");
> > +	}
> > +
> >  	switch (connector->connector_type) {
> >  	case DRM_MODE_CONNECTOR_DisplayPort:
> >  	case DRM_MODE_CONNECTOR_eDP:
> > --
> > 2.22.1
> > 
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux