On Fri, 2015-05-08 at 17:45 +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä <ville.syrjala at linux.intel.com> > > Currently we're adding CEA modes after the inferred modes, which > means > we might get multiple modes that are very close to each other, but > slightly different, which seems a bit silly. That's because duplicate > mode check that occurs when adding inferred modes would not consider > CEA modes as potential duplicates. Reverse the order so that CEA > modes get added before inferred modes, and are thus considered > potential > duplicates. > > Or as ajax put it on irc: > "< ajax> the point of the "pick a timing formula" heuristic was to > generate something the sink could _likely_ sink. if it tells us > timings it can sink explicitly then second-guessing seems dumb." > > Cc: Adam Jackson <ajax at redhat.com> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
Reviewed-by: Adam Jackson <ajax at redhat.com> - ajax
