On Thu, Jun 01, 2017 at 03:48:51PM +0100, Chris Wilson wrote: > On Thu, Jun 01, 2017 at 05:36:15PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä <ville.syrj...@linux.intel.com> > > > > The magic "enable the DPLL three times" sequence feels like it > > deserves a loop. > > Hmm, I thought once upon a time we figured out what the magic was about. > Or was that another loop?
At least I've never tried to understand this particular loop. The VGA_MODE_DISABLE thing I did manage to figure out. Whether that might have been a contributing factor for adding this loop is unclear. We actually have a couple more DPLL writes just before the loop, so we might want consider folding those into the loop as well. > > Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
