[Intel-gfx] [PATCH] drm/i915: Use the connector name in fbdev debug messages

[Chris Wilson]

On Tue, May 13, 2014 at 04:02:40PM +0200, Daniel Vetter wrote:
> On Tue, May 13, 2014 at 02:26:39PM +0100, Chris Wilson wrote:
> > During initial probing of the modes to assign to the fbdev console, we
> > use the CRTC and connector ids. These are much harder for us to
> > understand than if we used their actual names (or pipe in the CRTC
> > case). Similarly, we want to manually print the mode size rather than
> > rely on mode->name being set.
> > 
> > v2: Use '%c' for pipe_name()
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Reviewed-by: Damien Lespiau <damien.lespiau at intel.com>
> > ---
> >  drivers/gpu/drm/i915/intel_fbdev.c | 28 ++++++++++++++++------------
> >  1 file changed, 16 insertions(+), 12 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_fbdev.c b/drivers/gpu/drm/i915/intel_fbdev.c
> > index fbe7941f88c8..1d3f0a9cce31 100644
> > --- a/drivers/gpu/drm/i915/intel_fbdev.c
> > +++ b/drivers/gpu/drm/i915/intel_fbdev.c
> > @@ -344,14 +344,14 @@ static bool intel_fb_initial_config(struct drm_fb_helper *fb_helper,
> >  			num_connectors_detected++;
> >  
> >  		if (!enabled[i]) {
> > -			DRM_DEBUG_KMS("connector %d not enabled, skipping\n",
> > -				      connector->base.id);
> > +			DRM_DEBUG_KMS("connector %s not enabled, skipping\n",
> > +				      drm_get_connector_name(connector));
> >  			continue;
> >  		}
> >  
> >  		if (connector->force == DRM_FORCE_OFF) {
> 
> I seem to be lacking the patch to obey the connector->force stuff you have
> here. Instead I have an early bail-out at the top of fb_initial_config.
> 
> Do I miss some crucial patch or can you please rebase on top of dinq?

It's not crucial, just a patch you missed to fix up some of Jesse's
shortcomings.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre