[Intel-gfx] [PATCH 04/11] drm/i915: don't keep reassigning FBC_UNSUPPORTED

[Daniel Vetter]

On Fri, Dec 12, 2014 at 04:55:52PM -0800, Rodrigo Vivi wrote:
> On Mon, Dec 8, 2014 at 8:09 AM, Paulo Zanoni <przanoni at gmail.com> wrote:
> > From: Paulo Zanoni <paulo.r.zanoni at intel.com>
> >
> > This may save a few picoseconds on !HAS_FBC platforms. And it also
> > satisfies my OCD.
> >
> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> > ---
> >  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> > index 4647d57..f3d5764 100644
> > --- a/drivers/gpu/drm/i915/intel_fbc.c
> > +++ b/drivers/gpu/drm/i915/intel_fbc.c
> > @@ -516,10 +516,8 @@ void intel_fbc_update(struct drm_device *dev)
> >         const struct drm_display_mode *adjusted_mode;
> >         unsigned int max_width, max_height;
> >
> > -       if (!HAS_FBC(dev)) {
> > -               set_no_fbc_reason(dev_priv, FBC_UNSUPPORTED);
> 
> I really don't like this fbc_reasons... I believe a drm_debug here is enough.
> but I can try to kill that later.

Yeah, I think the original reason for no_fbc_reason was that it's to
reduce dmesg spam since we recheck for every pageflip. So horribly noisy.
Once we have all this restructured to only recheck when something changes
(at modest and when changing planes) and no longer for the on/off stuff we
do around pageflips (only needed on pre-g4x anyway) or when disabling the
primary plane we can go with dmesg debug output.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch