On Mon, 2008-11-17 at 16:46 +0800, [EMAIL PROTECTED] wrote:

> @@ -2687,14 +2725,9 @@ xf86OutputGetEDIDModes (xf86OutputPtr output)
>  _X_EXPORT xf86MonPtr
>  xf86OutputGetEDID (xf86OutputPtr output, I2CBusPtr pDDCBus)
>  {
> -    ScrnInfoPtr      scrn = output->scrn;
> -    xf86MonPtr mon;
>  
> -    mon = xf86DoEEDID(scrn->scrnIndex, pDDCBus, TRUE);
> -    if (mon)
> -        xf86DDCApplyQuirks(scrn->scrnIndex, mon);
> +    return  xf86DoEEDID(output->scrn->scrnIndex, pDDCBus, TRUE);
>  
> -    return mon;
>  }
>  
>  static char *_xf86ConnectorNames[] = {

I'm really not sure why you've moved quirk application from something we
do once when we get the EDID block, to something we do every time we
walk the detailed descriptors.  What's the motivation for that?

- ajax

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/xorg

Reply via email to