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 drivers/gpu/drm/i915/i915_debugfs.c | 2 +- drivers/gpu/drm/i915/i915_drv.h | 17 +- drivers/gpu/drm/i915/intel_display.c | 23 +- drivers/gpu/drm/i915/intel_drv.h | 6 +- drivers/gpu/drm/i915/intel_fbc.c | 614 +++++++++++++++++++---------------- drivers/gpu/drm/i915/intel_pm.c | 2 +- 6 files changed, 363 insertions(+), 301 deletions(-) -- 2.6.2 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx