On Fri, Jul 29, 2016 at 11:35:27AM +0200, Daniel Vetter wrote: > On Thu, Jul 28, 2016 at 05:50:46PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä <ville.syrj...@linux.intel.com> > > > > Let's make rate_to_index() return the highest rate available that's > > less than or equal to the rate requested by the caller. The function > > can then be used to filter out rates higher than a certain maximum > > rate. > > > > Cc: Ander Conselvan de Oliveira <ander.conselvan.de.olive...@intel.com> > > Cc: Jim Bride <jim.br...@linux.intel.com> > > Cc: Manasi D Navare <manasi.d.nav...@intel.com> > > Cc: Durgadoss R <durgados...@intel.com> > > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> > > Not sure what this will be used for, couldn't figure it out looking at > later patches?
Ah, forgot to mention that part. It seemed like there was a hunk in the upfront link training patch that could use this, rather than hand rolling yet another way of looking through the rate array. > -Daneil > > > --- > > drivers/gpu/drm/i915/intel_dp.c | 7 +++++-- > > 1 file changed, 5 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_dp.c > > b/drivers/gpu/drm/i915/intel_dp.c > > index 40cae6202160..08d95352ffd6 100644 > > --- a/drivers/gpu/drm/i915/intel_dp.c > > +++ b/drivers/gpu/drm/i915/intel_dp.c > > @@ -1431,10 +1431,13 @@ static int rate_to_index(int find, const int *rates) > > int i = 0; > > > > for (i = 0; i < DP_MAX_SUPPORTED_RATES; ++i) > > - if (find == rates[i]) > > + if (find < rates[i]) > > break; > > > > - return i; > > + if (WARN_ON(i == 0)) > > + return 0; > > + > > + return i - 1; > > } > > > > int > > -- > > 2.7.4 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx