On Tue, Jun 10, 2014 at 08:59:42AM +0200, Daniel Vetter wrote: > On Mon, Jun 09, 2014 at 07:47:10AM +0100, Chris Wilson wrote: > > + } else { > > + /* Switch crtc and encoder back off if necessary */ > > + if (old->dpms_mode != DRM_MODE_DPMS_ON) > > + connector->funcs->dpms(connector, old->dpms_mode); > > + } > > This part looks like an unrelated change? It's just diff. The change was to rejig the braces to remove the duplicated unlock + return. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx