Re: [PATCH 00/35] Make legacy modeset a lot more atomic-like

[Maarten Lankhorst <maarten.lankhorst@linux.intel.com>]

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@linux.intel.com>

\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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx