[Intel-gfx] [PATCH 22/25] drm/i915: remove some unneeded debug messages

Daniel Vetter daniel at ffwll.ch
Wed Mar 21 22:39:43 CET 2012


On Wed, Mar 21, 2012 at 10:36:22PM +0100, Daniel Vetter wrote:
> On Wed, Mar 21, 2012 at 12:48:43PM -0700, Jesse Barnes wrote:
> > Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> 
> So you've achieved the contentless-error-msg nirvana? Luckily for you one
> of these is quality work from China ;-) Patch applied to next, thanks.

Ok, not applied. My brain seems to be somewhere else atm - I've found two
of these in drm/i915 and somehow thought you'd kill these two. But this
patch just moves it. Can you enlighten me as to what's going on here?
-Daniel

> > ---
> >  drivers/gpu/drm/i915/intel_display.c |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index 58accf5..541c0a6 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -2075,14 +2075,14 @@ static void intel_update_fbc(struct drm_device *dev)
> >  	struct drm_i915_gem_object *obj;
> >  	int enable_fbc;
> >  
> > -	DRM_DEBUG_KMS("\n");
> > -
> >  	if (!i915_powersave)
> >  		return;
> >  
> >  	if (!I915_HAS_FBC(dev))
> >  		return;
> >  
> > +	DRM_DEBUG_KMS("\n");
> > +
> >  	/*
> >  	 * If FBC is already on, we just have to verify that we can
> >  	 * keep it that way...
> > -- 
> > 1.7.5.4
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Daniel Vetter
> Mail: daniel at ffwll.ch
> Mobile: +41 (0)79 365 57 48

-- 
Daniel Vetter
Mail: daniel at ffwll.ch
Mobile: +41 (0)79 365 57 48



More information about the Intel-gfx mailing list