On Fri, Oct 5, 2012 at 1:53 PM, Paulo Zanoni <[email protected]> wrote: > 2012/10/4 Lespiau, Damien <[email protected]>: >> On Tue, Oct 2, 2012 at 9:51 PM, Paulo Zanoni <[email protected]> wrote: >>> @@ -5356,8 +5361,6 @@ static int haswell_crtc_mode_set(struct drm_crtc >>> *crtc, >>> >>> haswell_set_pipeconf(crtc, adjusted_mode, dither); >>> >>> - intel_wait_for_vblank(dev, pipe); >>> - >> >> I guess this hunk could go into a separate commit. > > My way of thinking is that since the pipe is disabled there's no > reason to wait for vblank here.
Oh, sure I was just saying that it could deserve a separate commit from the one adding the warning, but bikeshedding really. -- Damien _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
