Op 27-07-15 om 13:16 schreef Daniel Vetter: > On Mon, Jul 27, 2015 at 01:04:20PM +0200, Maarten Lankhorst wrote: >> This is required for DPMS to work correctly, during a modeset >> the DPMS property should be turned off, unless the state is >> crtc is made active in which case it should be set to DPMS on. >> >> The legacy dpms handling performs its own dpms updates, so add a >> property to prevent updating the legacy dpms state and use it >> in the atomic dpms helpers and i915 suspend/resume. >> >> Changes since v1: >> - Set DPMS to off when a connector is removed from a crtc too. >> - Update the legacy dpms property too. >> - Add an exception for the legacy dpms paths, it updates its own state. >> - Add an exception for i915 suspend/resume, it should preserve dpms state. > My idea behind updating the dpms prop was to not confuse legacy userspace. > And legacy userspace always does an all-or-nothing dpms over all > connectors anyway, so I don't think we need to go to any length to > preserve that. If we'd need to we won't be able to move dpms from > connector to the crtc anyway. Therefore I think preserve_dpms isn't needed > and we can drop that one. > In that case I'll respin.. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
