On Mon, 2009-11-23 at 20:32 +0100, Rafał Miłecki wrote: > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > index cea665d..dd95edf 100644 > > --- a/drivers/gpu/drm/drm_edid.c > > +++ b/drivers/gpu/drm/drm_edid.c > > @@ -1290,6 +1290,8 @@ int drm_add_modes_noedid(struct drm_connector > > *connector, > > ptr->vdisplay > vdisplay) > > continue; > > } > > + if (drm_mode_vrefresh(ptr) > 61) > > + continue; > > mode = drm_mode_duplicate(dev, ptr); > > if (mode) { > > drm_mode_probed_add(connector, mode); > > Did you use 61 on purpose? > > Is that typo (> 60, or >= 61) or do you expect modes like 60.1Hz we > want to survive?
Mostly just paranoia, since drm_mode_vrefresh returns an int. - ajax
signature.asc
Description: This is a digitally signed message part
------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july
-- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel