On Thu, Sep 19, 2013 at 02:13:38PM +0100, Damien Lespiau wrote: > On Thu, Sep 19, 2013 at 02:53:58PM +0200, Daniel Vetter wrote: > > I always get royally confused how a modeline with all zeros could > > possible pass the paranoid pipe config checker. Until I realize again > > that we only check the crtc timings. So dump the crtc timings for the > > adjusted mode. > > > > This will be even more important for 3D support where the crtc timings > > are markedly different from the input modeline if we have > > frame-by-frame 3d output enabled. > > > > Cc: Damien Lespiau <damien.lespiau@xxxxxxxxx> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > > Reviewed-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> Thanks for the review, and I guess more reasons for crtc_clock ;-) Patch merged to dinq. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
