[Intel-gfx] [PATCH v3 16/20] drm/i915: Make intel_display_suspend atomic, try 2.

[Maarten Lankhorst]

Hey,

Op 14-07-15 om 18:14 schreef Daniel Vetter:
> On Mon, Jul 13, 2015 at 04:30:29PM +0200, Maarten Lankhorst wrote:
>> Calculate all state using a normal transition, but afterwards fudge
>> crtc->state->active back to its old value. This should still allow
>> state restore in setup_hw_state to work properly.
>>
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Ok merged up to this patch with the exception of patch 8. I think
> reworking that logic to use mode->private_flags to make sure we do a
> modeset when userspace changes the mode, but not earlier is a sound
> approach which will address all my concerns.
Patch 8 doesn't affect whether a modeset will happen and is safe to merge. It just fills in some more mode flags.
What you just described was done in patch 11 you just merged. ;-)

~Maarten