"The operation is in theory redundant, and in the case of Haswell where we have multiple outputs aliasing to the same encoder, actually harmful." Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68030 Credits-to: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Tested-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> --- src/uxa/intel_display.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/uxa/intel_display.c b/src/uxa/intel_display.c index 7dc0f68..0cb63b3 100644 --- a/src/uxa/intel_display.c +++ b/src/uxa/intel_display.c @@ -1123,10 +1123,11 @@ intel_output_dpms(xf86OutputPtr output, int dpms) intel_output_dpms_backlight(output, intel_output->dpms_mode, dpms); - drmModeConnectorSetProperty(mode->fd, - intel_output->output_id, - props->prop_id, - dpms); + if (output->crtc) + drmModeConnectorSetProperty(mode->fd, + intel_output->output_id, + props->prop_id, + dpms); if (dpms != DPMSModeOff) intel_output_dpms_backlight(output, intel_output->dpms_mode, -- 1.8.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
