On Thu, Nov 29, 2012 at 11:29:31AM -0200, Paulo Zanoni wrote: > From: Paulo Zanoni <paulo.r.zanoni at intel.com> > > We were previously doing exactly what the "mode set sequence for CRT" > document mandates, but whenever we failed to train the link in the > first tentative, all the other subsequent retries always failed. In > one of my monitors that has 47 modes, I was usually getting around 3 > failures when running "testdisplay -a". > > After this patch, even if we fail in the first tentative, we can > succeed in the next ones. So now when running "testdisplay -a" I see > around 3 times the message "FDI link training done on step 1" and no > failures. > > Notice that now the "retry" code looks a lot like the DP retry code. > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com> Picked up for -fixes, thanks for the patch. Upon re-reading it still irks me midly that the disable sequence here matches the begining of ddi_post_disable. Whatever. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch