Op 21-04-15 om 16:12 schreef Ander Conselvan de Oliveira: > Hi, > > This patch series changes the legacy modeset path to be a lot more > atomic like. Among other things, it > > - unifies the flip-only and the modeset path; > - implements a full state swap as part of the modeset; > - gets rid of the recovery logic in case of a failed modeset; > - replaces some i915 functions with drm atomic helper ones. > > Some of these patches were sent previously and the feedback so far has > been addressed. I also rebased this on top Maarten's series that > simplifies plane enabling/disabling. > > Thanks, > Ander > > Ander Conselvan de Oliveira (28): > drm/i915: Don't check for NULL before freeing state > drm/i915: Call drm helpers when duplicating crtc and plane states > drm/i915: Use for_each_connector_in_state helper macro > drm/i915: Extract mode_changed computation out of > stage_output_config() > drm/i915: Add crtc states before calling compute_config() > drm/i915: Don't pretend we can calculate multiple pipe_configs > drm/i915: Calculate a new pipe_config based on new enabled state > drm/i915: Remove all *_pipes flags from modeset > drm/i915: Remove saved_mode from __intel_set_mode() > drm/i915: Move compute part of __intel_set_mode() to separate function > drm/i915: Simplify error handling in __intel_set_mode() > drm/i915: Don't modeset with old mode when set_crtc fails > drm/i915: Add primary plane to atomic state in legacy modeset > drm/i915: Delete fb, x and y parameters from mode set functions > drm/i915: Don't use struct intel_set_config *_changed flags > drm/i915: Don't use staged config to calculate mode_changed flags > drm/i915: Unify modeset and flip paths of intel_crtc_set_config() > drm/i915: Simplify intel_set_config_compute_mode_changes() a bit > drm/i915: Stage new modeset state straight into atomic state > drm/i915: Remove save/restore logic from intel_crtc_set_config() > drm/i915: Update crtc state active flag based on DPMS > drm/atomic: Make mode_fixup() optional for check_modeset() > drm/i915: Use atomic helpers for computing changed flags > drm/i915: Take ownership of atomic state on success in > intel_set_mode() > drm/i915: Preserve shared DPLL information in new pipe_config > drm/i915: Don't use plane update helper in legacy mode set > drm/i915: Swap atomic state in legacy modeset > drm/i915: Get rid of intel_crtc_set_state() For the whole series: Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> \o/ fixing up atomic planes should be a lot more trivial now, killing the last of the plane helpers. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx