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@xxxxxxxxx> wrote:
> > From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> >
> > This may save a few picoseconds on !HAS_FBC platforms. And it also
> > satisfies my OCD.
> >
> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> > ---
> > 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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
