Hi So Ville pointed a problem on patch 02/26 of the previous series, and the nice fix for that would make me rebase most of the subsequent patches. In order to avoid blocking the other patches on the review of patch 02 and in order to avoid having to rebase everything again and again during this I decided to change the order of the patches on the series and try to get a smaller chunk of patches merged before moving on to the others. These first 13 patches are somewhat trivial and are mostly just moving code around, with only one or two simple bug fixes. If you wonder why some of these changes are needed, please go check the complete series (the short answer is: most of the changes are needed for the new model with enable/disable + activate/deactivate). After all these patches are merged I'll resend the rest. Thanks for all the reviews so far, Paulo Paulo Zanoni (13): drm/i915: rename intel_fbc_nuke to intel_fbc_recompress drm/i915: extract fbc_on_pipe_a_only() drm/i915: remove unnecessary check for crtc->primary->fb drm/i915: extract crtc_is_valid() on the FBC code drm/i915: use struct intel_crtc *crtc at __intel_fbc_update() drm/i915: fix the __intel_fbc_update() comments drm/i915: don't disable_fbc() if FBC is already disabled drm/i915: refactor FBC deactivation at init drm/i915: remove too-frequent FBC debug message drm/i915: fix the CFB size check drm/i915: clarify that checking the FB stride for CFB is intentional drm/i915: remove in_dbg_master check from intel_fbc.c drm/i915: remove newline from a no_fbc_reason message drivers/gpu/drm/i915/intel_display.c | 3 - drivers/gpu/drm/i915/intel_fbc.c | 123 ++++++++++++++++------------------- 2 files changed, 57 insertions(+), 69 deletions(-) -- 2.6.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx