Ville pointed out two ugly defects in the FBC code, and while trying to fix them I spotted a few extra things. No real-world bugs fixed here, but IMHO the code is much easier to read now. Paulo Zanoni (7): drm/i915/fbc: move the intel_fbc_can_choose() call out of the loop drm/i915/fbc: replace a loop with drm_atomic_get_existing_crtc_state() drm/i915/fbc: extract intel_fbc_can_enable() drm/i915/fbc: inline intel_fbc_can_choose() drm/i915/fbc: use drm_atomic_get_existing_crtc_state when appropriate drm/i915/fbc: move from crtc_state->enable_fbc to plane_state->enable_fbc drm/i915/fbc: convert intel_fbc.c to use INTEL_GEN() drivers/gpu/drm/i915/intel_drv.h | 4 +- drivers/gpu/drm/i915/intel_fbc.c | 97 ++++++++++++++++++---------------------- 2 files changed, 46 insertions(+), 55 deletions(-) -- 2.7.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx