On Thu, Jan 14, 2010 at 07:02:20PM +0100, Jerome Glisse wrote: > X is accepting such video mode, do the same. Pointed out by Joshua Roys > on IRC. Fix https://bugzilla.redhat.com/show_bug.cgi?id=540024 > > Signed-off-by: Jerome Glisse <[email protected]> > --- > drivers/gpu/drm/drm_edid.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > index 5c9f798..6d66383 100644 > --- a/drivers/gpu/drm/drm_edid.c > +++ b/drivers/gpu/drm/drm_edid.c > @@ -634,7 +634,6 @@ static struct drm_display_mode *drm_mode_detailed(struct > drm_device *dev, > } > if (!(pt->misc & DRM_EDID_PT_SEPARATE_SYNC)) { > printk(KERN_WARNING "integrated sync not supported\n"); > - return NULL; > }
I suppose the patch title should be 'accept composite sync'. Perhaps the error message could say composite sync too since then people would know what it's trying to say. At least I've never heard the term integrated sync before. -- Ville Syrjälä [email protected] http://www.sci.fi/~syrjala/ ------------------------------------------------------------------------------ Throughout its 18-year history, RSA Conference consistently attracts the world's best and brightest in the field, creating opportunities for Conference attendees to learn about information security's most important issues through interactions with peers, luminaries and emerging and established companies. http://p.sf.net/sfu/rsaconf-dev2dev -- _______________________________________________ Dri-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dri-devel
