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

Attachment: 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

Reply via email to