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
signature.asc
Description: This is a digitally signed message part
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel