[PATCH 10/10] xfree86: remove PCI dependency from InitOutput

Adam Jackson ajax at nwnk.net
Fri May 7 07:39:40 PDT 2010


On Fri, 2010-05-07 at 15:44 +0300, Tiago Vignatti wrote:

> +Bool
> +xf86CallDriverProbe( DriverPtr drv, Bool detect_only )
> +{
> +    Bool     foundScreen = FALSE;
> +
> +    if ( drv->PciProbe != NULL ) {
> +    if ( xf86DoConfigure && xf86DoConfigurePass1 ) {
> +        assert( detect_only );
> +        foundScreen = xf86PciAddMatchingDev(drv);
> +    }
> +    else {
> +        assert( ! detect_only );
> +        foundScreen = xf86PciProbeDev(drv);
> +    }
> +    }
> +
> +    if ( ! foundScreen && (drv->Probe != NULL) ) {
> +    xf86Msg( X_WARNING, "Falling back to old probe method for %s\n",
> +         drv->driverName );
> +    foundScreen = (*drv->Probe)( drv, (detect_only) ? PROBE_DETECT
> +                     : PROBE_DEFAULT );
> +    }
> +
> +    return foundScreen;
> +}

You're missing some indent levels here.  In fact most of this patch
seems bizarrely outdented in places.  I think your editor is doing
something strange.

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100507/fb8dac86/attachment.pgp>


More information about the xorg-devel mailing list