[Intel-gfx] [PATCH 2/2] drm/i915: Perform dpll commit first.

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Mon Nov 30 01:08:46 PST 2015


Op 26-11-15 om 15:41 schreef Ander Conselvan De Oliveira:
> On Tue, 2015-11-24 at 10:44 +0100, Maarten Lankhorst wrote:
>> Warn for the wrong mask in enable only. Disable will have the wrong mask
>> now because the new state is committed before disabling the old state.
> Why?
>
> Ander
>
This is required for async commit. In that case the state has to be committed while
still holding the connection_mutex, but the modeset itself is run asynchronously.
This also means that any modeset has to wait for previous modesets to complete,
otherwise the pll state may be updated incorrectly.

~Maarten


More information about the Intel-gfx mailing list