Em Qua, 2015-12-02 às 12:37 +0000, Chris Wilson escreveu:
> On Wed, Dec 02, 2015 at 10:15:16AM -0200, Paulo Zanoni wrote:
> > Hi
> >
> > This series includes the implementation for the review feedback
> > given by Chris.
> > I also removed the patch that transformed our 50ms timeout into a
> > vblank-based
> > timeout due to performance concerns. The only patch that still
> > needs review is
> > patch 6.
> >
> > Thanks,
> > Paulo
> >
> > Paulo Zanoni (11):
> > drm/i915: fix the CFB size check
> > drm/i915: set dev_priv->fbc.crtc before scheduling the enable
> > work
> > drm/i915: pass the crtc as an argument to intel_fbc_update()
> > drm/i915: introduce is_active/activate/deactivate to the FBC
> > terminology
> > drm/i915: introduce intel_fbc_{enable,disable}
> > drm/i915: alloc/free the FBC CFB during enable/disable
> > drm/i915: check for FBC planes in the same place as the pipes
> > drm/i915: use a single intel_fbc_work struct
> > drm/i915: kill fbc.uncompressed_size
> > drm/i915: get rid of FBC {,de}activation messages
> > drm/i915: only recompress FBC after flushing a drawing operation
>
> My r-b still stand.
What about patch 6? Any concerns or additional requests?
Thanks,
Paulo
> -Chris
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
