Merge remote-tracking branch 'origin/released'

Conflicts:
	pixman/pixman/pixman-mmx.c
	xorg-server/Xext/shm.c
	xorg-server/Xext/syncsrv.h
	xorg-server/Xext/xvmain.c
	xorg-server/Xi/exevents.c
	xorg-server/Xi/opendev.c
	xorg-server/composite/compalloc.c
	xorg-server/composite/compoverlay.c
	xorg-server/dix/colormap.c
	xorg-server/dix/devices.c
	xorg-server/dix/dispatch.c
	xorg-server/dix/dixfonts.c
	xorg-server/dix/eventconvert.c
	xorg-server/dix/events.c
	xorg-server/dix/gc.c
	xorg-server/dix/getevents.c
	xorg-server/dix/main.c
	xorg-server/dix/privates.c
	xorg-server/dix/registry.c
	xorg-server/dix/resource.c
	xorg-server/exa/exa_accel.c
	xorg-server/exa/exa_migration_classic.c
	xorg-server/exa/exa_unaccel.c
	xorg-server/fb/fb.h
	xorg-server/fb/fbcopy.c
	xorg-server/fb/fbpixmap.c
	xorg-server/glx/dispatch.h
	xorg-server/glx/glapi.h
	xorg-server/glx/glapi_gentable.c
	xorg-server/glx/glapitable.h
	xorg-server/glx/glprocs.h
	xorg-server/glx/glxcmds.c
	xorg-server/glx/glxcmdsswap.c
	xorg-server/glx/glxdricommon.c
	xorg-server/glx/glxdriswrast.c
	xorg-server/glx/glxext.c
	xorg-server/glx/indirect_dispatch.c
	xorg-server/glx/indirect_dispatch.h
	xorg-server/glx/indirect_dispatch_swap.c
	xorg-server/glx/indirect_size.h
	xorg-server/glx/indirect_size_get.h
	xorg-server/glx/indirect_table.c
	xorg-server/glx/indirect_util.c
	xorg-server/glx/rensize.c
	xorg-server/glx/single2swap.c
	xorg-server/glx/singlepix.c
	xorg-server/glx/singlepixswap.c
	xorg-server/glx/singlesize.c
	xorg-server/hw/dmx/dmxinit.c
	xorg-server/hw/kdrive/ephyr/ephyr.c
	xorg-server/hw/kdrive/ephyr/hostx.c
	xorg-server/hw/kdrive/ephyr/hostx.h
	xorg-server/hw/kdrive/src/kinput.c
	xorg-server/hw/xfree86/common/compiler.h
	xorg-server/hw/xwin/InitInput.c
	xorg-server/hw/xwin/InitOutput.c
	xorg-server/hw/xwin/ddraw.h
	xorg-server/hw/xwin/glx/glwrap.c
	xorg-server/hw/xwin/glx/indirect.c
	xorg-server/hw/xwin/glx/wgl_ext_api.h
	xorg-server/hw/xwin/glx/winpriv.c
	xorg-server/hw/xwin/win.h
	xorg-server/hw/xwin/winallpriv.c
	xorg-server/hw/xwin/winauth.c
	xorg-server/hw/xwin/winclipboard.h
	xorg-server/hw/xwin/winclipboardinit.c
	xorg-server/hw/xwin/winclipboardthread.c
	xorg-server/hw/xwin/winclipboardunicode.c
	xorg-server/hw/xwin/winclipboardwndproc.c
	xorg-server/hw/xwin/winclipboardwrappers.c
	xorg-server/hw/xwin/winclipboardxevents.c
	xorg-server/hw/xwin/wincmap.c
	xorg-server/hw/xwin/winconfig.c
	xorg-server/hw/xwin/wincreatewnd.c
	xorg-server/hw/xwin/wincursor.c
	xorg-server/hw/xwin/windialogs.c
	xorg-server/hw/xwin/winengine.c
	xorg-server/hw/xwin/winerror.c
	xorg-server/hw/xwin/wingc.c
	xorg-server/hw/xwin/wingetsp.c
	xorg-server/hw/xwin/winkeybd.c
	xorg-server/hw/xwin/winkeybd.h
	xorg-server/hw/xwin/winlayouts.h
	xorg-server/hw/xwin/winmisc.c
	xorg-server/hw/xwin/winmonitors.c
	xorg-server/hw/xwin/winmouse.c
	xorg-server/hw/xwin/winmsg.c
	xorg-server/hw/xwin/winmsg.h
	xorg-server/hw/xwin/winmultiwindowclass.c
	xorg-server/hw/xwin/winmultiwindowicons.c
	xorg-server/hw/xwin/winmultiwindowshape.c
	xorg-server/hw/xwin/winmultiwindowwindow.c
	xorg-server/hw/xwin/winmultiwindowwm.c
	xorg-server/hw/xwin/winmultiwindowwndproc.c
	xorg-server/hw/xwin/winnativegdi.c
	xorg-server/hw/xwin/winpfbdd.c
	xorg-server/hw/xwin/winpixmap.c
	xorg-server/hw/xwin/winpolyline.c
	xorg-server/hw/xwin/winprefs.c
	xorg-server/hw/xwin/winprocarg.c
	xorg-server/hw/xwin/winregistry.c
	xorg-server/hw/xwin/winscrinit.c
	xorg-server/hw/xwin/winsetsp.c
	xorg-server/hw/xwin/winshaddd.c
	xorg-server/hw/xwin/winshadddnl.c
	xorg-server/hw/xwin/winshadgdi.c
	xorg-server/hw/xwin/wintrayicon.c
	xorg-server/hw/xwin/winwin32rootless.c
	xorg-server/hw/xwin/winwin32rootlesswindow.c
	xorg-server/hw/xwin/winwin32rootlesswndproc.c
	xorg-server/hw/xwin/winwindow.c
	xorg-server/hw/xwin/winwindow.h
	xorg-server/hw/xwin/winwindowswm.c
	xorg-server/hw/xwin/winwndproc.c
	xorg-server/include/callback.h
	xorg-server/include/dixstruct.h
	xorg-server/include/misc.h
	xorg-server/include/os.h
	xorg-server/include/scrnintstr.h
	xorg-server/mi/micmap.c
	xorg-server/mi/miinitext.c
	xorg-server/mi/mioverlay.c
	xorg-server/mi/misprite.c
	xorg-server/mi/mivaltree.c
	xorg-server/mi/miwindow.c
	xorg-server/miext/damage/damage.c
	xorg-server/miext/rootless/rootlessGC.c
	xorg-server/miext/rootless/rootlessWindow.c
	xorg-server/os/WaitFor.c
	xorg-server/os/access.c
	xorg-server/os/connection.c
	xorg-server/os/io.c
	xorg-server/os/log.c
	xorg-server/os/osinit.c
	xorg-server/os/utils.c
	xorg-server/os/xdmcp.c
	xorg-server/os/xprintf.c
	xorg-server/os/xstrans.c
	xorg-server/render/mipict.c
	xorg-server/xkb/xkbActions.c
	xorg-server/xkb/xkbInit.c
	xorg-server/xkeyboard-config/compat/default.in

1 files changed, 675 insertions, 664 deletions

diff --git a/xorg-server/hw/xfree86/common/xf86Helper.c b/xorg-server/hw/xfree86/common/xf86Helper.c
index 248ccf47e..8c948cf6f 100644
--- a/xorg-server/hw/xfree86/common/xf86Helper.c
+++ b/xorg-server/hw/xfree86/common/xf86Helper.c
@@ -64,7 +64,6 @@
 
 static int xf86ScrnInfoPrivateCount = 0;
 
-
 /* Add a pointer to a new DriverRec to xf86DriverList */
 
 void
@@ -72,22 +71,21 @@ xf86AddDriver(DriverPtr driver, pointer module, int flags)
 {
     /* Don't add null entries */
     if (!driver)
-	return;
+        return;
 
     if (xf86DriverList == NULL)
-	xf86NumDrivers = 0;
+        xf86NumDrivers = 0;
 
     xf86NumDrivers++;
     xf86DriverList = xnfrealloc(xf86DriverList,
-				xf86NumDrivers * sizeof(DriverPtr));
+                                xf86NumDrivers * sizeof(DriverPtr));
     xf86DriverList[xf86NumDrivers - 1] = xnfalloc(sizeof(DriverRec));
     if (flags & HaveDriverFuncs)
-	*xf86DriverList[xf86NumDrivers - 1] = *driver;
+        *xf86DriverList[xf86NumDrivers - 1] = *driver;
     else {
-	(void) memset( xf86DriverList[xf86NumDrivers - 1], 0,
-		       sizeof( DriverRec ) );
-	(void) memcpy( xf86DriverList[xf86NumDrivers - 1], driver,
-		       sizeof(DriverRec1));
+        (void) memset(xf86DriverList[xf86NumDrivers - 1], 0, sizeof(DriverRec));
+        (void) memcpy(xf86DriverList[xf86NumDrivers - 1], driver,
+                      sizeof(DriverRec1));
 
     }
     xf86DriverList[xf86NumDrivers - 1]->module = module;
@@ -98,11 +96,11 @@ void
 xf86DeleteDriver(int drvIndex)
 {
     if (xf86DriverList[drvIndex]
-	&& (!xf86DriverHasEntities(xf86DriverList[drvIndex]))) {
-	if (xf86DriverList[drvIndex]->module)
-	    UnloadModule(xf86DriverList[drvIndex]->module);
-	free(xf86DriverList[drvIndex]);
-	xf86DriverList[drvIndex] = NULL;
+        && (!xf86DriverHasEntities(xf86DriverList[drvIndex]))) {
+        if (xf86DriverList[drvIndex]->module)
+            UnloadModule(xf86DriverList[drvIndex]->module);
+        free(xf86DriverList[drvIndex]);
+        xf86DriverList[drvIndex] = NULL;
     }
 }
 
@@ -113,16 +111,17 @@ xf86AddInputDriver(InputDriverPtr driver, pointer module, int flags)
 {
     /* Don't add null entries */
     if (!driver)
-	return;
+        return;
 
     if (xf86InputDriverList == NULL)
-	xf86NumInputDrivers = 0;
+        xf86NumInputDrivers = 0;
 
     xf86NumInputDrivers++;
     xf86InputDriverList = xnfrealloc(xf86InputDriverList,
-				xf86NumInputDrivers * sizeof(InputDriverPtr));
+                                     xf86NumInputDrivers *
+                                     sizeof(InputDriverPtr));
     xf86InputDriverList[xf86NumInputDrivers - 1] =
-				xnfalloc(sizeof(InputDriverRec));
+        xnfalloc(sizeof(InputDriverRec));
     *xf86InputDriverList[xf86NumInputDrivers - 1] = *driver;
     xf86InputDriverList[xf86NumInputDrivers - 1]->module = module;
 }
@@ -131,7 +130,7 @@ void
 xf86DeleteInputDriver(int drvIndex)
 {
     if (xf86InputDriverList[drvIndex] && xf86InputDriverList[drvIndex]->module)
-	UnloadModule(xf86InputDriverList[drvIndex]->module);
+        UnloadModule(xf86InputDriverList[drvIndex]->module);
     free(xf86InputDriverList[drvIndex]);
     xf86InputDriverList[drvIndex] = NULL;
 }
@@ -142,9 +141,9 @@ xf86LookupInputDriver(const char *name)
     int i;
 
     for (i = 0; i < xf86NumInputDrivers; i++) {
-       if (xf86InputDriverList[i] && xf86InputDriverList[i]->driverName &&
-           xf86NameCmp(name, xf86InputDriverList[i]->driverName) == 0)
-           return xf86InputDriverList[i];
+        if (xf86InputDriverList[i] && xf86InputDriverList[i]->driverName &&
+            xf86NameCmp(name, xf86InputDriverList[i]->driverName) == 0)
+            return xf86InputDriverList[i];
     }
     return NULL;
 }
@@ -170,15 +169,15 @@ xf86AllocateScreen(DriverPtr drv, int flags)
     int i;
 
     if (xf86Screens == NULL)
-	xf86NumScreens = 0;
+        xf86NumScreens = 0;
 
     i = xf86NumScreens++;
     xf86Screens = xnfrealloc(xf86Screens, xf86NumScreens * sizeof(ScrnInfoPtr));
     xf86Screens[i] = xnfcalloc(sizeof(ScrnInfoRec), 1);
-    xf86Screens[i]->scrnIndex = i;	/* Changes when a screen is removed */
-    xf86Screens[i]->origIndex = i;	/* This never changes */
+    xf86Screens[i]->scrnIndex = i;      /* Changes when a screen is removed */
+    xf86Screens[i]->origIndex = i;      /* This never changes */
     xf86Screens[i]->privates = xnfcalloc(sizeof(DevUnion),
-					 xf86ScrnInfoPrivateCount);
+                                         xf86ScrnInfoPrivateCount);
     /*
      * EnableDisableFBAccess now gets initialized in InitOutput()
      * xf86Screens[i]->EnableDisableFBAccess = xf86EnableDisableFBAccess;
@@ -193,7 +192,6 @@ xf86AllocateScreen(DriverPtr drv, int flags)
     return xf86Screens[i];
 }
 
-
 /*
  * Remove an entry from xf86Screens.  Ideally it should free all allocated
  * data.  To do this properly may require a driver hook.
@@ -207,31 +205,31 @@ xf86DeleteScreen(int scrnIndex, int flags)
 
     /* First check if the screen is valid */
     if (xf86NumScreens == 0 || xf86Screens == NULL)
-	return;
+        return;
 
     if (scrnIndex > xf86NumScreens - 1)
-	return;
+        return;
 
     if (!(pScrn = xf86Screens[scrnIndex]))
-	return;
+        return;
 
     /* If a FreeScreen function is defined, call it here */
     if (pScrn->FreeScreen != NULL)
-	pScrn->FreeScreen(scrnIndex, 0);
+        pScrn->FreeScreen(scrnIndex, 0);
 
     while (pScrn->modes)
-	xf86DeleteMode(&pScrn->modes, pScrn->modes);
+        xf86DeleteMode(&pScrn->modes, pScrn->modes);
 
     while (pScrn->modePool)
-	xf86DeleteMode(&pScrn->modePool, pScrn->modePool);
+        xf86DeleteMode(&pScrn->modePool, pScrn->modePool);
 
     xf86OptionListFree(pScrn->options);
 
     if (pScrn->module)
-	UnloadModule(pScrn->module);
+        UnloadModule(pScrn->module);
 
     if (pScrn->drv)
-	pScrn->drv->refCount--;
+        pScrn->drv->refCount--;
 
     free(pScrn->privates);
 
@@ -244,9 +242,9 @@ xf86DeleteScreen(int scrnIndex, int flags)
     xf86NumScreens--;
 
     for (i = scrnIndex; i < xf86NumScreens; i++) {
-	xf86Screens[i] = xf86Screens[i + 1];
-	xf86Screens[i]->scrnIndex = i;
-	/* Also need to take care of the screen layout settings */
+        xf86Screens[i] = xf86Screens[i + 1];
+        xf86Screens[i]->scrnIndex = i;
+        /* Also need to take care of the screen layout settings */
     }
 }
 
@@ -263,12 +261,12 @@ xf86AllocateScrnInfoPrivateIndex(void)
 
     idx = xf86ScrnInfoPrivateCount++;
     for (i = 0; i < xf86NumScreens; i++) {
-	pScr = xf86Screens[i];
-	nprivs = xnfrealloc(pScr->privates,
-			    xf86ScrnInfoPrivateCount * sizeof(DevUnion));
-	/* Zero the new private */
-	memset(&nprivs[idx], 0, sizeof(DevUnion));
-	pScr->privates = nprivs;
+        pScr = xf86Screens[i];
+        nprivs = xnfrealloc(pScr->privates,
+                            xf86ScrnInfoPrivateCount * sizeof(DevUnion));
+        /* Zero the new private */
+        memset(&nprivs[idx], 0, sizeof(DevUnion));
+        pScr->privates = nprivs;
     }
     return idx;
 }
@@ -279,22 +277,22 @@ xf86AddPixFormat(ScrnInfoPtr pScrn, int depth, int bpp, int pad)
     int i;
 
     if (pScrn->numFormats >= MAXFORMATS)
-	return FALSE;
+        return FALSE;
 
     if (bpp <= 0) {
-	if (depth == 1)
-	    bpp = 1;
-	else if (depth <= 8)
-	    bpp = 8;
-	else if (depth <= 16)
-	    bpp = 16;
-	else if (depth <= 32)
-	    bpp = 32;
-	else
-	    return FALSE;
+        if (depth == 1)
+            bpp = 1;
+        else if (depth <= 8)
+            bpp = 8;
+        else if (depth <= 16)
+            bpp = 16;
+        else if (depth <= 32)
+            bpp = 32;
+        else
+            return FALSE;
     }
     if (pad <= 0)
-	pad = BITMAP_SCANLINE_PAD;
+        pad = BITMAP_SCANLINE_PAD;
 
     i = pScrn->numFormats++;
     pScrn->formats[i].depth = depth;
@@ -346,7 +344,7 @@ xf86AddPixFormat(ScrnInfoPtr pScrn, int depth, int bpp, int pad)
 
 Bool
 xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
-		int depth24flags)
+                int depth24flags)
 {
     int i;
     DispPtr disp;
@@ -360,174 +358,182 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
     scrp->depthFrom = X_DEFAULT;
 
     if (xf86FbBpp > 0) {
-	scrp->bitsPerPixel = xf86FbBpp;
-	scrp->bitsPerPixelFrom = X_CMDLINE;
+        scrp->bitsPerPixel = xf86FbBpp;
+        scrp->bitsPerPixelFrom = X_CMDLINE;
     }
 
     if (xf86Depth > 0) {
-	scrp->depth = xf86Depth;
-	scrp->depthFrom = X_CMDLINE;
+        scrp->depth = xf86Depth;
+        scrp->depthFrom = X_CMDLINE;
     }
 
     if (xf86FbBpp < 0 && xf86Depth < 0) {
-	if (scrp->confScreen->defaultfbbpp > 0) {
-	    scrp->bitsPerPixel = scrp->confScreen->defaultfbbpp;
-	    scrp->bitsPerPixelFrom = X_CONFIG;
-	}
-	if (scrp->confScreen->defaultdepth > 0) {
-	    scrp->depth = scrp->confScreen->defaultdepth;
-	    scrp->depthFrom = X_CONFIG;
-	}
-
-	if (scrp->confScreen->defaultfbbpp <= 0 &&
-	    scrp->confScreen->defaultdepth <= 0) {
-	    /*
-	     * Check for DefaultDepth and DefaultFbBpp options in the
-	     * Device sections.
-	     */
-	    int i;
-	    GDevPtr device;
-	    Bool found = FALSE;
-
-	    for (i = 0; i < scrp->numEntities; i++) {
-		device = xf86GetDevFromEntity(scrp->entityList[i],
-					      scrp->entityInstanceList[i]);
-		if (device && device->options) {
-		    if (xf86FindOption(device->options, "DefaultDepth")) {
-			scrp->depth = xf86SetIntOption(device->options,
-						       "DefaultDepth", -1);
-			scrp->depthFrom = X_CONFIG;
-			found = TRUE;
-		    }
-		    if (xf86FindOption(device->options, "DefaultFbBpp")) {
-			scrp->bitsPerPixel = xf86SetIntOption(device->options,
-							      "DefaultFbBpp",
-							      -1);
-			scrp->bitsPerPixelFrom = X_CONFIG;
-			found = TRUE;
-		    }
-		}
-		if (found)
-		    break;
-	    }
-	}
+        if (scrp->confScreen->defaultfbbpp > 0) {
+            scrp->bitsPerPixel = scrp->confScreen->defaultfbbpp;
+            scrp->bitsPerPixelFrom = X_CONFIG;
+        }
+        if (scrp->confScreen->defaultdepth > 0) {
+            scrp->depth = scrp->confScreen->defaultdepth;
+            scrp->depthFrom = X_CONFIG;
+        }
+
+        if (scrp->confScreen->defaultfbbpp <= 0 &&
+            scrp->confScreen->defaultdepth <= 0) {
+            /*
+             * Check for DefaultDepth and DefaultFbBpp options in the
+             * Device sections.
+             */
+            int i;
+            GDevPtr device;
+            Bool found = FALSE;
+
+            for (i = 0; i < scrp->numEntities; i++) {
+                device = xf86GetDevFromEntity(scrp->entityList[i],
+                                              scrp->entityInstanceList[i]);
+                if (device && device->options) {
+                    if (xf86FindOption(device->options, "DefaultDepth")) {
+                        scrp->depth = xf86SetIntOption(device->options,
+                                                       "DefaultDepth", -1);
+                        scrp->depthFrom = X_CONFIG;
+                        found = TRUE;
+                    }
+                    if (xf86FindOption(device->options, "DefaultFbBpp")) {
+                        scrp->bitsPerPixel = xf86SetIntOption(device->options,
+                                                              "DefaultFbBpp",
+                                                              -1);
+                        scrp->bitsPerPixelFrom = X_CONFIG;
+                        found = TRUE;
+                    }
+                }
+                if (found)
+                    break;
+            }
+        }
     }
 
     /* If none of these is set, pick a default */
     if (scrp->bitsPerPixel < 0 && scrp->depth < 0) {
         if (fbbpp > 0 || depth > 0) {
-	    if (fbbpp > 0)
-		scrp->bitsPerPixel = fbbpp;
-	    if (depth > 0)
-		scrp->depth = depth;
-	} else {
-	    scrp->depth = GLOBAL_DEFAULT_DEPTH;
-	}
+            if (fbbpp > 0)
+                scrp->bitsPerPixel = fbbpp;
+            if (depth > 0)
+                scrp->depth = depth;
+        }
+        else {
+            scrp->depth = GLOBAL_DEFAULT_DEPTH;
+        }
     }
 
     /* If any are not given, determine a default for the others */
 
     if (scrp->bitsPerPixel < 0) {
-	/* The depth must be set */
-	if (scrp->depth > -1) {
-	    if (scrp->depth == 1)
-		scrp->bitsPerPixel = 1;
-	    else if (scrp->depth <= 4)
-		scrp->bitsPerPixel = 4;
-	    else if (scrp->depth <= 8)
-		scrp->bitsPerPixel = 8;
-	    else if (scrp->depth <= 16)
-		scrp->bitsPerPixel = 16;
-	    else if (scrp->depth <= 24) {
-		/*
-		 * Figure out if a choice is possible based on the depth24
-		 * and pix24 flags.
-		 */
-		/* Check pix24 first */
-		if (pix24 != Pix24DontCare) {
-		    if (pix24 == Pix24Use32) {
-			if (DO_PIX32(depth24flags)) {
-			    if (CHOOSE24FOR32(depth24flags))
-				scrp->bitsPerPixel = 24;
-			    else
-				scrp->bitsPerPixel = 32;
-			} else {
-			    nomatch = TRUE;
-			}
-		    } else if (pix24 == Pix24Use24) {
-			if (DO_PIX24(depth24flags)) {
-			    if (CHOOSE32FOR24(depth24flags))
-				scrp->bitsPerPixel = 32;
-			    else
-				scrp->bitsPerPixel = 24;
-			} else {
-			    nomatch = TRUE;
-			}
-		    }
-		} else {
-		    if (DO_PIX32(depth24flags)) {
-			if (CHOOSE24FOR32(depth24flags))
-			    scrp->bitsPerPixel = 24;
-			else
-			    scrp->bitsPerPixel = 32;
-		    } else if (DO_PIX24(depth24flags)) {
-			if (CHOOSE32FOR24(depth24flags))
-			    scrp->bitsPerPixel = 32;
-			else
-			    scrp->bitsPerPixel = 24;
-		    }
-		}
-	    } else if (scrp->depth <= 32)
-		scrp->bitsPerPixel = 32;
-	    else {
-		xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-			   "Specified depth (%d) is greater than 32\n",
-			   scrp->depth);
-		return FALSE;
-	    }
-	} else {
-	    xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-			"xf86SetDepthBpp: internal error: depth and fbbpp"
-			" are both not set\n");
-	    return FALSE;
-	}
-	if (scrp->bitsPerPixel < 0) {
-	    if (nomatch)
-		xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-			"Driver can't support depth 24 pixmap format (%d)\n",
-			PIX24TOBPP(pix24));
-	    else if ((depth24flags & (Support24bppFb | Support32bppFb)) ==
-		     NoDepth24Support)
-		xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-			"Driver can't support depth 24\n");
-	    else
-		xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-			"Can't find fbbpp for depth 24\n");
-	    return FALSE;
-	}
-	scrp->bitsPerPixelFrom = X_PROBED;
+        /* The depth must be set */
+        if (scrp->depth > -1) {
+            if (scrp->depth == 1)
+                scrp->bitsPerPixel = 1;
+            else if (scrp->depth <= 4)
+                scrp->bitsPerPixel = 4;
+            else if (scrp->depth <= 8)
+                scrp->bitsPerPixel = 8;
+            else if (scrp->depth <= 16)
+                scrp->bitsPerPixel = 16;
+            else if (scrp->depth <= 24) {
+                /*
+                 * Figure out if a choice is possible based on the depth24
+                 * and pix24 flags.
+                 */
+                /* Check pix24 first */
+                if (pix24 != Pix24DontCare) {
+                    if (pix24 == Pix24Use32) {
+                        if (DO_PIX32(depth24flags)) {
+                            if (CHOOSE24FOR32(depth24flags))
+                                scrp->bitsPerPixel = 24;
+                            else
+                                scrp->bitsPerPixel = 32;
+                        }
+                        else {
+                            nomatch = TRUE;
+                        }
+                    }
+                    else if (pix24 == Pix24Use24) {
+                        if (DO_PIX24(depth24flags)) {
+                            if (CHOOSE32FOR24(depth24flags))
+                                scrp->bitsPerPixel = 32;
+                            else
+                                scrp->bitsPerPixel = 24;
+                        }
+                        else {
+                            nomatch = TRUE;
+                        }
+                    }
+                }
+                else {
+                    if (DO_PIX32(depth24flags)) {
+                        if (CHOOSE24FOR32(depth24flags))
+                            scrp->bitsPerPixel = 24;
+                        else
+                            scrp->bitsPerPixel = 32;
+                    }
+                    else if (DO_PIX24(depth24flags)) {
+                        if (CHOOSE32FOR24(depth24flags))
+                            scrp->bitsPerPixel = 32;
+                        else
+                            scrp->bitsPerPixel = 24;
+                    }
+                }
+            }
+            else if (scrp->depth <= 32)
+                scrp->bitsPerPixel = 32;
+            else {
+                xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                           "Specified depth (%d) is greater than 32\n",
+                           scrp->depth);
+                return FALSE;
+            }
+        }
+        else {
+            xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                       "xf86SetDepthBpp: internal error: depth and fbbpp"
+                       " are both not set\n");
+            return FALSE;
+        }
+        if (scrp->bitsPerPixel < 0) {
+            if (nomatch)
+                xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                           "Driver can't support depth 24 pixmap format (%d)\n",
+                           PIX24TOBPP(pix24));
+            else if ((depth24flags & (Support24bppFb | Support32bppFb)) ==
+                     NoDepth24Support)
+                xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                           "Driver can't support depth 24\n");
+            else
+                xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                           "Can't find fbbpp for depth 24\n");
+            return FALSE;
+        }
+        scrp->bitsPerPixelFrom = X_PROBED;
     }
 
     if (scrp->depth <= 0) {
-	/* bitsPerPixel is already set */
-	switch (scrp->bitsPerPixel) {
-	case 32:
-	    scrp->depth = 24;
-	    break;
-	default:
-	    /* 1, 4, 8, 16 and 24 */
-	    scrp->depth = scrp->bitsPerPixel;
-	    break;
-	}
-	scrp->depthFrom = X_PROBED;
+        /* bitsPerPixel is already set */
+        switch (scrp->bitsPerPixel) {
+        case 32:
+            scrp->depth = 24;
+            break;
+        default:
+            /* 1, 4, 8, 16 and 24 */
+            scrp->depth = scrp->bitsPerPixel;
+            break;
+        }
+        scrp->depthFrom = X_PROBED;
     }
 
     /* Sanity checks */
     if (scrp->depth < 1 || scrp->depth > 32) {
-	xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		   "Specified depth (%d) is not in the range 1-32\n",
-		    scrp->depth);
-	return FALSE;
+        xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                   "Specified depth (%d) is not in the range 1-32\n",
+                   scrp->depth);
+        return FALSE;
     }
     switch (scrp->bitsPerPixel) {
     case 1:
@@ -536,26 +542,26 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
     case 16:
     case 24:
     case 32:
-	break;
+        break;
     default:
-	xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		   "Specified fbbpp (%d) is not a permitted value\n",
-		   scrp->bitsPerPixel);
-	return FALSE;
+        xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                   "Specified fbbpp (%d) is not a permitted value\n",
+                   scrp->bitsPerPixel);
+        return FALSE;
     }
     if (scrp->depth > scrp->bitsPerPixel) {
-	xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		   "Specified depth (%d) is greater than the fbbpp (%d)\n",
-		   scrp->depth, scrp->bitsPerPixel);
-	return FALSE;
+        xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                   "Specified depth (%d) is greater than the fbbpp (%d)\n",
+                   scrp->depth, scrp->bitsPerPixel);
+        return FALSE;
     }
 
     /* set scrp->pixmap24 if the driver isn't flexible */
     if (scrp->bitsPerPixel == 24 && !DO_PIX32FOR24(depth24flags)) {
-	scrp->pixmap24 = Pix24Use24;
+        scrp->pixmap24 = Pix24Use24;
     }
     if (scrp->bitsPerPixel == 32 && !DO_PIX24FOR32(depth24flags)) {
-	scrp->pixmap24 = Pix24Use32;
+        scrp->pixmap24 = Pix24Use32;
     }
 
     /*
@@ -563,13 +569,13 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
      * scrp->display with it.
      */
     for (i = 0, disp = scrp->confScreen->displays;
-	 i < scrp->confScreen->numdisplays; i++, disp++) {
-	if ((disp->depth == scrp->depth && disp->fbbpp == scrp->bitsPerPixel)
-	    || (disp->depth == scrp->depth && disp->fbbpp <= 0)
-	    || (disp->fbbpp == scrp->bitsPerPixel && disp->depth <= 0)) {
-	    scrp->display = disp;
-	    break;
-	}
+         i < scrp->confScreen->numdisplays; i++, disp++) {
+        if ((disp->depth == scrp->depth && disp->fbbpp == scrp->bitsPerPixel)
+            || (disp->depth == scrp->depth && disp->fbbpp <= 0)
+            || (disp->fbbpp == scrp->bitsPerPixel && disp->depth <= 0)) {
+            scrp->display = disp;
+            break;
+        }
     }
 
     /*
@@ -577,40 +583,40 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
      * depth or fbbpp specified.
      */
     if (i == scrp->confScreen->numdisplays) {
-	for (i = 0, disp = scrp->confScreen->displays;
-	     i < scrp->confScreen->numdisplays; i++, disp++) {
-	    if (disp->depth <= 0 && disp->fbbpp <= 0) {
-		scrp->display = disp;
-		break;
-	    }
-	}
+        for (i = 0, disp = scrp->confScreen->displays;
+             i < scrp->confScreen->numdisplays; i++, disp++) {
+            if (disp->depth <= 0 && disp->fbbpp <= 0) {
+                scrp->display = disp;
+                break;
+            }
+        }
     }
 
     /*
      * If all else fails, create a default one.
      */
     if (i == scrp->confScreen->numdisplays) {
-	scrp->confScreen->numdisplays++;
-	scrp->confScreen->displays =
-		xnfrealloc(scrp->confScreen->displays,
-			   scrp->confScreen->numdisplays * sizeof(DispRec));
-	xf86DrvMsg(scrp->scrnIndex, X_INFO,
-		   "Creating default Display subsection in Screen section\n"
-		   "\t\"%s\" for depth/fbbpp %d/%d\n",
-		   scrp->confScreen->id, scrp->depth, scrp->bitsPerPixel);
-	memset(&scrp->confScreen->displays[i], 0, sizeof(DispRec));
-	scrp->confScreen->displays[i].blackColour.red = -1;
-	scrp->confScreen->displays[i].blackColour.green = -1;
-	scrp->confScreen->displays[i].blackColour.blue = -1;
-	scrp->confScreen->displays[i].whiteColour.red = -1;
-	scrp->confScreen->displays[i].whiteColour.green = -1;
-	scrp->confScreen->displays[i].whiteColour.blue = -1;
-	scrp->confScreen->displays[i].defaultVisual = -1;
-	scrp->confScreen->displays[i].modes = xnfalloc(sizeof(char *));
-	scrp->confScreen->displays[i].modes[0] = NULL;
-	scrp->confScreen->displays[i].depth = depth;
-	scrp->confScreen->displays[i].fbbpp = fbbpp;
-	scrp->display = &scrp->confScreen->displays[i];
+        scrp->confScreen->numdisplays++;
+        scrp->confScreen->displays =
+            xnfrealloc(scrp->confScreen->displays,
+                       scrp->confScreen->numdisplays * sizeof(DispRec));
+        xf86DrvMsg(scrp->scrnIndex, X_INFO,
+                   "Creating default Display subsection in Screen section\n"
+                   "\t\"%s\" for depth/fbbpp %d/%d\n",
+                   scrp->confScreen->id, scrp->depth, scrp->bitsPerPixel);
+        memset(&scrp->confScreen->displays[i], 0, sizeof(DispRec));
+        scrp->confScreen->displays[i].blackColour.red = -1;
+        scrp->confScreen->displays[i].blackColour.green = -1;
+        scrp->confScreen->displays[i].blackColour.blue = -1;
+        scrp->confScreen->displays[i].whiteColour.red = -1;
+        scrp->confScreen->displays[i].whiteColour.green = -1;
+        scrp->confScreen->displays[i].whiteColour.blue = -1;
+        scrp->confScreen->displays[i].defaultVisual = -1;
+        scrp->confScreen->displays[i].modes = xnfalloc(sizeof(char *));
+        scrp->confScreen->displays[i].modes[0] = NULL;
+        scrp->confScreen->displays[i].depth = depth;
+        scrp->confScreen->displays[i].fbbpp = fbbpp;
+        scrp->display = &scrp->confScreen->displays[i];
     }
 
     /*
@@ -621,12 +627,13 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
     scrp->imageByteOrder = IMAGE_BYTE_ORDER;
     scrp->bitmapScanlinePad = BITMAP_SCANLINE_PAD;
     if (scrp->depth < 8) {
-	/* Planar modes need these settings */
-	scrp->bitmapScanlineUnit = 8;
-	scrp->bitmapBitOrder = MSBFirst;
-    } else {
-	scrp->bitmapScanlineUnit = BITMAP_SCANLINE_UNIT;
-	scrp->bitmapBitOrder = BITMAP_BIT_ORDER;
+        /* Planar modes need these settings */
+        scrp->bitmapScanlineUnit = 8;
+        scrp->bitmapBitOrder = MSBFirst;
+    }
+    else {
+        scrp->bitmapScanlineUnit = BITMAP_SCANLINE_UNIT;
+        scrp->bitmapBitOrder = BITMAP_BIT_ORDER;
     }
 
     /*
@@ -640,14 +647,14 @@ xf86SetDepthBpp(ScrnInfoPtr scrp, int depth, int dummy, int fbbpp,
     case 15:
     case 16:
     case 24:
-	/* Common depths.  Nothing to do for them */
-	break;
+        /* Common depths.  Nothing to do for them */
+        break;
     default:
-	if (!xf86AddPixFormat(scrp, scrp->depth, 0, 0)) {
-	    xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		       "Can't add pixmap format for depth %d\n", scrp->depth);
-	    return FALSE;
-	}
+        if (!xf86AddPixFormat(scrp, scrp->depth, 0, 0)) {
+            xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                       "Can't add pixmap format for depth %d\n", scrp->depth);
+            return FALSE;
+        }
     }
 
     /* Initialise the framebuffer format for this screen */
@@ -682,87 +689,91 @@ xf86SetWeight(ScrnInfoPtr scrp, rgb weight, rgb mask)
     scrp->weight.blue = 0;
 
     if (xf86Weight.red > 0 && xf86Weight.green > 0 && xf86Weight.blue > 0) {
-	scrp->weight = xf86Weight;
-	weightFrom = X_CMDLINE;
-    } else if (scrp->display->weight.red > 0 && scrp->display->weight.green > 0
-	       && scrp->display->weight.blue > 0) {
-	scrp->weight = scrp->display->weight;
-	weightFrom = X_CONFIG;
-    } else if (weight.red > 0 && weight.green > 0 && weight.blue > 0) {
-	scrp->weight = weight;
-    } else {
-	switch (scrp->depth) {
-	case 1:
-	case 4:
-	case 8:
-	    scrp->weight.red = scrp->weight.green =
-		scrp->weight.blue = scrp->rgbBits;
-	    break;
-	case 15:
-	    scrp->weight.red = scrp->weight.green = scrp->weight.blue = 5;
-	    break;
-	case 16:
-	    scrp->weight.red = scrp->weight.blue = 5;
-	    scrp->weight.green = 6;
-	    break;
-	case 18:
-	    scrp->weight.red = scrp->weight.green = scrp->weight.blue = 6;
-	    break;
-	case 24:
-	    scrp->weight.red = scrp->weight.green = scrp->weight.blue = 8;
-	    break;
-	case 30:
-	    scrp->weight.red = scrp->weight.green = scrp->weight.blue = 10;
-	    break;
-	}
+        scrp->weight = xf86Weight;
+        weightFrom = X_CMDLINE;
+    }
+    else if (scrp->display->weight.red > 0 && scrp->display->weight.green > 0
+             && scrp->display->weight.blue > 0) {
+        scrp->weight = scrp->display->weight;
+        weightFrom = X_CONFIG;
+    }
+    else if (weight.red > 0 && weight.green > 0 && weight.blue > 0) {
+        scrp->weight = weight;
+    }
+    else {
+        switch (scrp->depth) {
+        case 1:
+        case 4:
+        case 8:
+            scrp->weight.red = scrp->weight.green =
+                scrp->weight.blue = scrp->rgbBits;
+            break;
+        case 15:
+            scrp->weight.red = scrp->weight.green = scrp->weight.blue = 5;
+            break;
+        case 16:
+            scrp->weight.red = scrp->weight.blue = 5;
+            scrp->weight.green = 6;
+            break;
+        case 18:
+            scrp->weight.red = scrp->weight.green = scrp->weight.blue = 6;
+            break;
+        case 24:
+            scrp->weight.red = scrp->weight.green = scrp->weight.blue = 8;
+            break;
+        case 30:
+            scrp->weight.red = scrp->weight.green = scrp->weight.blue = 10;
+            break;
+        }
     }
 
     if (scrp->weight.red)
-	xf86DrvMsg(scrp->scrnIndex, weightFrom, "RGB weight %d%d%d\n",
-		   (int)scrp->weight.red, (int)scrp->weight.green,
-		   (int)scrp->weight.blue);
+        xf86DrvMsg(scrp->scrnIndex, weightFrom, "RGB weight %d%d%d\n",
+                   (int) scrp->weight.red, (int) scrp->weight.green,
+                   (int) scrp->weight.blue);
 
     if (scrp->depth > MAX_PSEUDO_DEPTH &&
-	(scrp->depth != scrp->weight.red + scrp->weight.green +
-			scrp->weight.blue)) {
-	xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		   "Weight given (%d%d%d) is inconsistent with the "
-		   "depth (%d)\n",
-		   (int)scrp->weight.red, (int)scrp->weight.green,
-		   (int)scrp->weight.blue, scrp->depth);
-	return FALSE;
+        (scrp->depth != scrp->weight.red + scrp->weight.green +
+         scrp->weight.blue)) {
+        xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                   "Weight given (%d%d%d) is inconsistent with the "
+                   "depth (%d)\n",
+                   (int) scrp->weight.red, (int) scrp->weight.green,
+                   (int) scrp->weight.blue, scrp->depth);
+        return FALSE;
     }
     if (scrp->depth > MAX_PSEUDO_DEPTH && scrp->weight.red) {
-	/*
-	 * XXX Does this even mean anything for TrueColor visuals?
-	 * If not, we shouldn't even be setting it here.  However, this
-	 * matches the behaviour of 3.x versions of XFree86.
-	 */
-	scrp->rgbBits = scrp->weight.red;
-	if (scrp->weight.green > scrp->rgbBits)
-	    scrp->rgbBits = scrp->weight.green;
-	if (scrp->weight.blue > scrp->rgbBits)
-	    scrp->rgbBits = scrp->weight.blue;
+        /*
+         * XXX Does this even mean anything for TrueColor visuals?
+         * If not, we shouldn't even be setting it here.  However, this
+         * matches the behaviour of 3.x versions of XFree86.
+         */
+        scrp->rgbBits = scrp->weight.red;
+        if (scrp->weight.green > scrp->rgbBits)
+            scrp->rgbBits = scrp->weight.green;
+        if (scrp->weight.blue > scrp->rgbBits)
+            scrp->rgbBits = scrp->weight.blue;
     }
 
     /* Set the mask and offsets */
     if (mask.red == 0 || mask.green == 0 || mask.blue == 0) {
-	/* Default to a setting common to PC hardware */
-	scrp->offset.red = scrp->weight.green + scrp->weight.blue;
-	scrp->offset.green = scrp->weight.blue;
-	scrp->offset.blue = 0;
-	scrp->mask.red = ((1 << scrp->weight.red) - 1) << scrp->offset.red;
-	scrp->mask.green = ((1 << scrp->weight.green) - 1)
-				<< scrp->offset.green;
-	scrp->mask.blue = (1 << scrp->weight.blue) - 1;
-    } else {
-	/* Initialise to the values passed */
-	scrp->mask.red = mask.red;
-	scrp->mask.green = mask.green;
-	scrp->mask.blue = mask.blue;
-	scrp->offset.red = ffs(mask.red);
-	scrp->offset.green = ffs(mask.green);
-	scrp->offset.blue = ffs(mask.blue);
+        /* Default to a setting common to PC hardware */
+        scrp->offset.red = scrp->weight.green + scrp->weight.blue;
+        scrp->offset.green = scrp->weight.blue;
+        scrp->offset.blue = 0;
+        scrp->mask.red = ((1 << scrp->weight.red) - 1) << scrp->offset.red;
+        scrp->mask.green = ((1 << scrp->weight.green) - 1)
+            << scrp->offset.green;
+        scrp->mask.blue = (1 << scrp->weight.blue) - 1;
+    }
+    else {
+        /* Initialise to the values passed */
+        scrp->mask.red = mask.red;
+        scrp->mask.green = mask.green;
+        scrp->mask.blue = mask.blue;
+        scrp->offset.red = ffs(mask.red);
+        scrp->offset.green = ffs(mask.green);
+        scrp->offset.blue = ffs(mask.blue);
     }
     return TRUE;
 }
@@ -773,22 +784,25 @@ xf86SetDefaultVisual(ScrnInfoPtr scrp, int visual)
     MessageType visualFrom = X_DEFAULT;
 
     if (defaultColorVisualClass >= 0) {
-	scrp->defaultVisual = defaultColorVisualClass;
-	visualFrom = X_CMDLINE;
-    } else if (scrp->display->defaultVisual >= 0) {
-	scrp->defaultVisual = scrp->display->defaultVisual;
-	visualFrom = X_CONFIG;
-    } else if (visual >= 0) {
-	scrp->defaultVisual = visual;
-    } else {
-	if (scrp->depth == 1)
-	    scrp->defaultVisual = StaticGray;
-	else if (scrp->depth == 4)
-	    scrp->defaultVisual = StaticColor;
-	else if (scrp->depth <= MAX_PSEUDO_DEPTH)
-	    scrp->defaultVisual = PseudoColor;
-	else
-	    scrp->defaultVisual = TrueColor;
+        scrp->defaultVisual = defaultColorVisualClass;
+        visualFrom = X_CMDLINE;
+    }
+    else if (scrp->display->defaultVisual >= 0) {
+        scrp->defaultVisual = scrp->display->defaultVisual;
+        visualFrom = X_CONFIG;
+    }
+    else if (visual >= 0) {
+        scrp->defaultVisual = visual;
+    }
+    else {
+        if (scrp->depth == 1)
+            scrp->defaultVisual = StaticGray;
+        else if (scrp->depth == 4)
+            scrp->defaultVisual = StaticColor;
+        else if (scrp->depth <= MAX_PSEUDO_DEPTH)
+            scrp->defaultVisual = PseudoColor;
+        else
+            scrp->defaultVisual = TrueColor;
     }
     switch (scrp->defaultVisual) {
     case StaticGray:
@@ -797,14 +811,14 @@ xf86SetDefaultVisual(ScrnInfoPtr scrp, int visual)
     case PseudoColor:
     case TrueColor:
     case DirectColor:
-	xf86DrvMsg(scrp->scrnIndex, visualFrom, "Default visual is %s\n",
-		   xf86VisualNames[scrp->defaultVisual]);
-	    return TRUE;
+        xf86DrvMsg(scrp->scrnIndex, visualFrom, "Default visual is %s\n",
+                   xf86VisualNames[scrp->defaultVisual]);
+        return TRUE;
     default:
 
-	xf86DrvMsg(scrp->scrnIndex, X_ERROR,
-		   "Invalid default visual class (%d)\n", scrp->defaultVisual);
-	return FALSE;
+        xf86DrvMsg(scrp->scrnIndex, X_ERROR,
+                   "Invalid default visual class (%d)\n", scrp->defaultVisual);
+        return FALSE;
     }
 }
 
@@ -818,45 +832,50 @@ Bool
 xf86SetGamma(ScrnInfoPtr scrp, Gamma gamma)
 {
     MessageType from = X_DEFAULT;
+
 #if 0
-    xf86MonPtr DDC = (xf86MonPtr)(scrp->monitor->DDC);
+    xf86MonPtr DDC = (xf86MonPtr) (scrp->monitor->DDC);
 #endif
     if (TEST_GAMMA(xf86Gamma)) {
-	from = X_CMDLINE;
-	scrp->gamma.red = SET_GAMMA(xf86Gamma.red);
-	scrp->gamma.green = SET_GAMMA(xf86Gamma.green);
-	scrp->gamma.blue = SET_GAMMA(xf86Gamma.blue);
-    } else if (TEST_GAMMA(scrp->monitor->gamma)) {
-	from = X_CONFIG;
-	scrp->gamma.red = SET_GAMMA(scrp->monitor->gamma.red);
-	scrp->gamma.green = SET_GAMMA(scrp->monitor->gamma.green);
-	scrp->gamma.blue = SET_GAMMA(scrp->monitor->gamma.blue);
+        from = X_CMDLINE;
+        scrp->gamma.red = SET_GAMMA(xf86Gamma.red);
+        scrp->gamma.green = SET_GAMMA(xf86Gamma.green);
+        scrp->gamma.blue = SET_GAMMA(xf86Gamma.blue);
+    }
+    else if (TEST_GAMMA(scrp->monitor->gamma)) {
+        from = X_CONFIG;
+        scrp->gamma.red = SET_GAMMA(scrp->monitor->gamma.red);
+        scrp->gamma.green = SET_GAMMA(scrp->monitor->gamma.green);
+        scrp->gamma.blue = SET_GAMMA(scrp->monitor->gamma.blue);
 #if 0
-    } else if ( DDC && DDC->features.gamma > GAMMA_ZERO ) {
+    }
+    else if (DDC && DDC->features.gamma > GAMMA_ZERO) {
         from = X_PROBED;
-	scrp->gamma.red = SET_GAMMA(DDC->features.gamma);
-	scrp->gamma.green = SET_GAMMA(DDC->features.gamma);
-	scrp->gamma.blue = SET_GAMMA(DDC->features.gamma);
-	/* EDID structure version 2 gives optional seperate red, green & blue gamma values
-	 * in bytes 0x57-0x59 */
+        scrp->gamma.red = SET_GAMMA(DDC->features.gamma);
+        scrp->gamma.green = SET_GAMMA(DDC->features.gamma);
+        scrp->gamma.blue = SET_GAMMA(DDC->features.gamma);
+        /* EDID structure version 2 gives optional seperate red, green & blue gamma values
+         * in bytes 0x57-0x59 */
 #endif
-    } else if (TEST_GAMMA(gamma)) {
-	scrp->gamma.red = SET_GAMMA(gamma.red);
-	scrp->gamma.green = SET_GAMMA(gamma.green);
-	scrp->gamma.blue = SET_GAMMA(gamma.blue);
-    } else {
-	scrp->gamma.red = 1.0;
-	scrp->gamma.green = 1.0;
-	scrp->gamma.blue = 1.0;
+    }
+    else if (TEST_GAMMA(gamma)) {
+        scrp->gamma.red = SET_GAMMA(gamma.red);
+        scrp->gamma.green = SET_GAMMA(gamma.green);
+        scrp->gamma.blue = SET_GAMMA(gamma.blue);
+    }
+    else {
+        scrp->gamma.red = 1.0;
+        scrp->gamma.green = 1.0;
+        scrp->gamma.blue = 1.0;
     }
     /* Pretend we succeeded if we support better a gamma system.
      * This avoids a confusing message.
      */
     if (xf86_crtc_supports_gamma(scrp))
-	return TRUE;
+        return TRUE;
     xf86DrvMsg(scrp->scrnIndex, from,
-	       "Using gamma correction (%.1f, %.1f, %.1f)\n",
-	       scrp->gamma.red, scrp->gamma.green, scrp->gamma.blue);
+               "Using gamma correction (%.1f, %.1f, %.1f)\n",
+               scrp->gamma.red, scrp->gamma.green, scrp->gamma.blue);
 
     return TRUE;
 }
@@ -864,7 +883,6 @@ xf86SetGamma(ScrnInfoPtr scrp, Gamma gamma)
 #undef TEST_GAMMA
 #undef SET_GAMMA
 
-
 /*
  * Set the DPI from the command line option.  XXX should allow it to be
  * calculated from the widthmm/heightmm values.
@@ -877,7 +895,7 @@ void
 xf86SetDpi(ScrnInfoPtr pScrn, int x, int y)
 {
     MessageType from = X_DEFAULT;
-    xf86MonPtr DDC = (xf86MonPtr)(pScrn->monitor->DDC);
+    xf86MonPtr DDC = (xf86MonPtr) (pScrn->monitor->DDC);
     int ddcWidthmm, ddcHeightmm;
     int widthErr, heightErr;
 
@@ -885,100 +903,107 @@ xf86SetDpi(ScrnInfoPtr pScrn, int x, int y)
     pScrn->widthmm = pScrn->monitor->widthmm;
     pScrn->heightmm = pScrn->monitor->heightmm;
 
-    if (DDC && (DDC->features.hsize > 0 && DDC->features.vsize > 0) ) {
-      /* DDC gives display size in mm for individual modes,
-       * but cm for monitor
-       */
-      ddcWidthmm = DDC->features.hsize * 10; /* 10mm in 1cm */
-      ddcHeightmm = DDC->features.vsize * 10; /* 10mm in 1cm */
-    } else {
-      ddcWidthmm = ddcHeightmm = 0;
+    if (DDC && (DDC->features.hsize > 0 && DDC->features.vsize > 0)) {
+        /* DDC gives display size in mm for individual modes,
+         * but cm for monitor
+         */
+        ddcWidthmm = DDC->features.hsize * 10;  /* 10mm in 1cm */
+        ddcHeightmm = DDC->features.vsize * 10; /* 10mm in 1cm */
+    }
+    else {
+        ddcWidthmm = ddcHeightmm = 0;
     }
 
     if (monitorResolution > 0) {
-	pScrn->xDpi = monitorResolution;
-	pScrn->yDpi = monitorResolution;
-	from = X_CMDLINE;
-    } else if (pScrn->widthmm > 0 || pScrn->heightmm > 0) {
-	from = X_CONFIG;
-	if (pScrn->widthmm > 0) {
-	   pScrn->xDpi =
-		(int)((double)pScrn->virtualX * MMPERINCH / pScrn->widthmm);
-	}
-	if (pScrn->heightmm > 0) {
-	   pScrn->yDpi =
-		(int)((double)pScrn->virtualY * MMPERINCH / pScrn->heightmm);
-	}
-	if (pScrn->xDpi > 0 && pScrn->yDpi <= 0)
-	    pScrn->yDpi = pScrn->xDpi;
-	if (pScrn->yDpi > 0 && pScrn->xDpi <= 0)
-	    pScrn->xDpi = pScrn->yDpi;
-	xf86DrvMsg(pScrn->scrnIndex, from, "Display dimensions: (%d, %d) mm\n",
-		   pScrn->widthmm, pScrn->heightmm);
-
-	/* Warn if config and probe disagree about display size */
-	if ( ddcWidthmm && ddcHeightmm ) {
-	  if (pScrn->widthmm > 0) {
-	    widthErr  = abs(ddcWidthmm  - pScrn->widthmm);
-	  } else {
-	    widthErr  = 0;
-	  }
-	  if (pScrn->heightmm > 0) {
-	    heightErr = abs(ddcHeightmm - pScrn->heightmm);
-	  } else {
-	    heightErr = 0;
-	  }
-	  if (widthErr>10 || heightErr>10) {
-	    /* Should include config file name for monitor here */
-	    xf86DrvMsg(pScrn->scrnIndex, X_WARNING,
-		       "Probed monitor is %dx%d mm, using Displaysize %dx%d mm\n",
-		       ddcWidthmm,ddcHeightmm, pScrn->widthmm,pScrn->heightmm);
-	  }
-	}
-    } else if ( ddcWidthmm && ddcHeightmm ) {
-	from = X_PROBED;
-	xf86DrvMsg(pScrn->scrnIndex, from, "Display dimensions: (%d, %d) mm\n",
-		   ddcWidthmm, ddcHeightmm );
-	pScrn->widthmm = ddcWidthmm;
-	pScrn->heightmm = ddcHeightmm;
-	if (pScrn->widthmm > 0) {
-	   pScrn->xDpi =
-		(int)((double)pScrn->virtualX * MMPERINCH / pScrn->widthmm);
-	}
-	if (pScrn->heightmm > 0) {
-	   pScrn->yDpi =
-		(int)((double)pScrn->virtualY * MMPERINCH / pScrn->heightmm);
-	}
-	if (pScrn->xDpi > 0 && pScrn->yDpi <= 0)
-	    pScrn->yDpi = pScrn->xDpi;
-	if (pScrn->yDpi > 0 && pScrn->xDpi <= 0)
-	    pScrn->xDpi = pScrn->yDpi;
-    } else {
-	if (x > 0)
-	    pScrn->xDpi = x;
-	else
-	    pScrn->xDpi = DEFAULT_DPI;
-	if (y > 0)
-	    pScrn->yDpi = y;
-	else
-	    pScrn->yDpi = DEFAULT_DPI;
+        pScrn->xDpi = monitorResolution;
+        pScrn->yDpi = monitorResolution;
+        from = X_CMDLINE;
+    }
+    else if (pScrn->widthmm > 0 || pScrn->heightmm > 0) {
+        from = X_CONFIG;
+        if (pScrn->widthmm > 0) {
+            pScrn->xDpi =
+                (int) ((double) pScrn->virtualX * MMPERINCH / pScrn->widthmm);
+        }
+        if (pScrn->heightmm > 0) {
+            pScrn->yDpi =
+                (int) ((double) pScrn->virtualY * MMPERINCH / pScrn->heightmm);
+        }
+        if (pScrn->xDpi > 0 && pScrn->yDpi <= 0)
+            pScrn->yDpi = pScrn->xDpi;
+        if (pScrn->yDpi > 0 && pScrn->xDpi <= 0)
+            pScrn->xDpi = pScrn->yDpi;
+        xf86DrvMsg(pScrn->scrnIndex, from, "Display dimensions: (%d, %d) mm\n",
+                   pScrn->widthmm, pScrn->heightmm);
+
+        /* Warn if config and probe disagree about display size */
+        if (ddcWidthmm && ddcHeightmm) {
+            if (pScrn->widthmm > 0) {
+                widthErr = abs(ddcWidthmm - pScrn->widthmm);
+            }
+            else {
+                widthErr = 0;
+            }
+            if (pScrn->heightmm > 0) {
+                heightErr = abs(ddcHeightmm - pScrn->heightmm);
+            }
+            else {
+                heightErr = 0;
+            }
+            if (widthErr > 10 || heightErr > 10) {
+                /* Should include config file name for monitor here */
+                xf86DrvMsg(pScrn->scrnIndex, X_WARNING,
+                           "Probed monitor is %dx%d mm, using Displaysize %dx%d mm\n",
+                           ddcWidthmm, ddcHeightmm, pScrn->widthmm,
+                           pScrn->heightmm);
+            }
+        }
+    }
+    else if (ddcWidthmm && ddcHeightmm) {
+        from = X_PROBED;
+        xf86DrvMsg(pScrn->scrnIndex, from, "Display dimensions: (%d, %d) mm\n",
+                   ddcWidthmm, ddcHeightmm);
+        pScrn->widthmm = ddcWidthmm;
+        pScrn->heightmm = ddcHeightmm;
+        if (pScrn->widthmm > 0) {
+            pScrn->xDpi =
+                (int) ((double) pScrn->virtualX * MMPERINCH / pScrn->widthmm);
+        }
+        if (pScrn->heightmm > 0) {
+            pScrn->yDpi =
+                (int) ((double) pScrn->virtualY * MMPERINCH / pScrn->heightmm);
+        }
+        if (pScrn->xDpi > 0 && pScrn->yDpi <= 0)
+            pScrn->yDpi = pScrn->xDpi;
+        if (pScrn->yDpi > 0 && pScrn->xDpi <= 0)
+            pScrn->xDpi = pScrn->yDpi;
+    }
+    else {
+        if (x > 0)
+            pScrn->xDpi = x;
+        else
+            pScrn->xDpi = DEFAULT_DPI;
+        if (y > 0)
+            pScrn->yDpi = y;
+        else
+            pScrn->yDpi = DEFAULT_DPI;
     }
     xf86DrvMsg(pScrn->scrnIndex, from, "DPI set to (%d, %d)\n",
-	       pScrn->xDpi, pScrn->yDpi);
+               pScrn->xDpi, pScrn->yDpi);
 }
 
 #undef MMPERINCH
 
-
 void
 xf86SetBlackWhitePixels(ScreenPtr pScreen)
 {
     if (xf86FlipPixels) {
-	pScreen->whitePixel = 0;
-	pScreen->blackPixel = 1;
-    } else {
-	pScreen->whitePixel = 1;
-	pScreen->blackPixel = 0;
+        pScreen->whitePixel = 0;
+        pScreen->blackPixel = 1;
+    }
+    else {
+        pScreen->whitePixel = 1;
+        pScreen->blackPixel = 0;
     }
 }
 
@@ -1009,21 +1034,19 @@ xf86EnableDisableFBAccess(int scrnIndex, Bool enable)
     PixmapPtr pspix;
 
     pspix = (*pScreen->GetScreenPixmap) (pScreen);
-    if (enable)
-    {
-	/*
-	 * Restore all of the clip lists on the screen
-	 */
-	if (!xf86Resetting)
-	    SetRootClip (pScreen, TRUE);
+    if (enable) {
+        /*
+         * Restore all of the clip lists on the screen
+         */
+        if (!xf86Resetting)
+            SetRootClip(pScreen, TRUE);
 
     }
-    else
-    {
-	/*
-	 * Empty all of the clip lists on the screen
-	 */
-	SetRootClip (pScreen, FALSE);
+    else {
+        /*
+         * Empty all of the clip lists on the screen
+         */
+        SetRootClip(pScreen, FALSE);
     }
 }
 
@@ -1031,21 +1054,21 @@ xf86EnableDisableFBAccess(int scrnIndex, Bool enable)
    (<type>) <screen name>(<screen index>): <message> */
 void
 xf86VDrvMsgVerb(int scrnIndex, MessageType type, int verb, const char *format,
-		va_list args)
+                va_list args)
 {
     /* Prefix the scrnIndex name to the format string. */
     if (scrnIndex >= 0 && scrnIndex < xf86NumScreens &&
-	xf86Screens[scrnIndex]->name)
-	LogHdrMessageVerb(type, verb, format, args, "%s(%d): ",
-	    xf86Screens[scrnIndex]->name, scrnIndex);
+        xf86Screens[scrnIndex]->name)
+        LogHdrMessageVerb(type, verb, format, args, "%s(%d): ",
+                          xf86Screens[scrnIndex]->name, scrnIndex);
     else
-	LogVMessageVerb(type, verb, format, args);
+        LogVMessageVerb(type, verb, format, args);
 }
 
 /* Print driver messages, with verbose level specified directly */
 void
 xf86DrvMsgVerb(int scrnIndex, MessageType type, int verb, const char *format,
-	       ...)
+               ...)
 {
     va_list ap;
 
@@ -1069,26 +1092,26 @@ xf86DrvMsg(int scrnIndex, MessageType type, const char *format, ...)
    (<type>) <driver>: <device name>: <message> */
 void
 xf86VIDrvMsgVerb(InputInfoPtr dev, MessageType type, int verb,
-		 const char *format, va_list args)
+                 const char *format, va_list args)
 {
     const char *driverName = NULL;
     const char *deviceName = NULL;
 
     /* Prefix driver and device names to formatted message. */
     if (dev) {
-	deviceName = dev->name;
-	if (dev->drv)
-	    driverName = dev->drv->driverName;
+        deviceName = dev->name;
+        if (dev->drv)
+            driverName = dev->drv->driverName;
     }
 
     LogHdrMessageVerb(type, verb, format, args, "%s: %s: ", driverName,
-	deviceName);
+                      deviceName);
 }
 
 /* Print input driver message, with verbose level specified directly */
 void
-xf86IDrvMsgVerb(InputInfoPtr dev, MessageType type, int verb, const char *format,
-	       ...)
+xf86IDrvMsgVerb(InputInfoPtr dev, MessageType type, int verb,
+                const char *format, ...)
 {
     va_list ap;
 
@@ -1108,7 +1131,6 @@ xf86IDrvMsg(InputInfoPtr dev, MessageType type, const char *format, ...)
     va_end(ap);
 }
 
-
 /* Print non-driver messages with verbose level specified directly */
 void
 xf86MsgVerb(MessageType type, int verb, const char *format, ...)
@@ -1139,7 +1161,7 @@ xf86ErrorFVerb(int verb, const char *format, ...)
 
     va_start(ap, format);
     if (xf86Verbose >= verb || xf86LogVerbose >= verb)
-	LogVWrite(verb, format, ap);
+        LogVWrite(verb, format, ap);
     va_end(ap);
 }
 
@@ -1151,11 +1173,10 @@ xf86ErrorF(const char *format, ...)
 
     va_start(ap, format);
     if (xf86Verbose >= 1 || xf86LogVerbose >= 1)
-	LogVWrite(1, format, ap);
+        LogVWrite(1, format, ap);
     va_end(ap);
 }
 
-
 void
 xf86LogInit(void)
 {
@@ -1166,10 +1187,10 @@ xf86LogInit(void)
 
     /* Get the log file name */
     if (xf86LogFileFrom == X_DEFAULT) {
-	/* Append the display number and ".log" */
-	if (asprintf(&lf, "%s%%s" LOGSUFFIX, xf86LogFile) == -1)
-	    FatalError("Cannot allocate space for the log file name\n");
-	xf86LogFile = lf;
+        /* Append the display number and ".log" */
+        if (asprintf(&lf, "%s%%s" LOGSUFFIX, xf86LogFile) == -1)
+            FatalError("Cannot allocate space for the log file name\n");
+        xf86LogFile = lf;
     }
 
     xf86LogFile = LogInit(xf86LogFile, LOGOLDSUFFIX);
@@ -1190,7 +1211,6 @@ xf86CloseLog(enum ExitCode error)
     LogClose(error);
 }
 
-
 /*
  * Drivers can use these for using their own SymTabRecs.
  */
@@ -1200,13 +1220,12 @@ xf86TokenToString(SymTabPtr table, int token)
 {
     int i;
 
-    for (i = 0; table[i].token >= 0 && table[i].token != token; i++)
-	;
+    for (i = 0; table[i].token >= 0 && table[i].token != token; i++);
 
     if (table[i].token < 0)
-	return NULL;
+        return NULL;
     else
-	return table[i].name;
+        return table[i].name;
 }
 
 int
@@ -1215,10 +1234,9 @@ xf86StringToToken(SymTabPtr table, const char *string)
     int i;
 
     if (string == NULL)
-	return -1;
+        return -1;
 
-    for (i = 0; table[i].token >= 0 && xf86NameCmp(string, table[i].name); i++)
-	;
+    for (i = 0; table[i].token >= 0 && xf86NameCmp(string, table[i].name); i++);
 
     return table[i].token;
 }
@@ -1232,17 +1250,16 @@ xf86ShowClocks(ScrnInfoPtr scrp, MessageType from)
     int j;
 
     xf86DrvMsg(scrp->scrnIndex, from, "Pixel clocks available:");
-    for (j=0; j < scrp->numClocks; j++) {
-	if ((j % 4) == 0) {
-	    xf86ErrorF("\n");
-	    xf86DrvMsg(scrp->scrnIndex, from, "pixel clocks:");
-	}
-	xf86ErrorF(" %7.3f", (double)scrp->clock[j] / 1000.0);
+    for (j = 0; j < scrp->numClocks; j++) {
+        if ((j % 4) == 0) {
+            xf86ErrorF("\n");
+            xf86DrvMsg(scrp->scrnIndex, from, "pixel clocks:");
+        }
+        xf86ErrorF(" %7.3f", (double) scrp->clock[j] / 1000.0);
     }
     xf86ErrorF("\n");
 }
 
-
 /*
  * This prints out the driver identify message, including the names of
  * the supported chipsets.
@@ -1258,40 +1275,40 @@ xf86PrintChipsets(const char *drvname, const char *drvmsg, SymTabPtr chips)
     len = 6 + strlen(drvname) + 2 + strlen(drvmsg) + 2;
     xf86Msg(X_INFO, "%s: %s:", drvname, drvmsg);
     for (i = 0; chips[i].name != NULL; i++) {
-	if (i != 0) {
-	    xf86ErrorF(",");
-	    len++;
-	}
-	if (len + 2 + strlen(chips[i].name) < 78) {
-	    xf86ErrorF(" ");
-	    len++;
-	} else {
-	    xf86ErrorF("\n\t");
-	    len = 8;
-	}
-	xf86ErrorF("%s", chips[i].name);
-	len += strlen(chips[i].name);
+        if (i != 0) {
+            xf86ErrorF(",");
+            len++;
+        }
+        if (len + 2 + strlen(chips[i].name) < 78) {
+            xf86ErrorF(" ");
+            len++;
+        }
+        else {
+            xf86ErrorF("\n\t");
+            len = 8;
+        }
+        xf86ErrorF("%s", chips[i].name);
+        len += strlen(chips[i].name);
     }
     xf86ErrorF("\n");
 }
 
-
 int
-xf86MatchDevice(const char *drivername, GDevPtr **sectlist)
+xf86MatchDevice(const char *drivername, GDevPtr ** sectlist)
 {
-    GDevPtr       gdp, *pgdp = NULL;
+    GDevPtr gdp, *pgdp = NULL;
     confScreenPtr screensecptr;
-    int i,j;
+    int i, j;
 
     if (sectlist)
-	*sectlist = NULL;
+        *sectlist = NULL;
 
     /*
      * This can happen when running Xorg -showopts and a module like ati
      * or vmware tries to load its submodules when xf86ConfigLayout is empty
      */
     if (!xf86ConfigLayout.screens)
-	return 0;
+        return 0;
 
     /*
      * This is a very important function that matches the device sections
@@ -1314,11 +1331,11 @@ xf86MatchDevice(const char *drivername, GDevPtr **sectlist)
      * first we need to loop over all the Screens sections to get to all
      * 'active' device sections
      */
-    for (j=0; xf86ConfigLayout.screens[j].screen != NULL; j++) {
+    for (j = 0; xf86ConfigLayout.screens[j].screen != NULL; j++) {
         screensecptr = xf86ConfigLayout.screens[j].screen;
         if ((screensecptr->device->driver != NULL)
-            && (xf86NameCmp( screensecptr->device->driver,drivername) == 0)
-            && (! screensecptr->device->claimed)) {
+            && (xf86NameCmp(screensecptr->device->driver, drivername) == 0)
+            && (!screensecptr->device->claimed)) {
             /*
              * we have a matching driver that wasn't claimed, yet
              */
@@ -1330,14 +1347,14 @@ xf86MatchDevice(const char *drivername, GDevPtr **sectlist)
     /* Then handle the inactive devices */
     j = 0;
     while (xf86ConfigLayout.inactives[j].identifier) {
-	gdp = &xf86ConfigLayout.inactives[j];
-	if (gdp->driver && !gdp->claimed &&
-	    !xf86NameCmp(gdp->driver,drivername)) {
-	    /* we have a matching driver that wasn't claimed yet */
-	    pgdp = xnfrealloc(pgdp, (i + 2) * sizeof(GDevPtr));
-	    pgdp[i++] = gdp;
-	}
-	j++;
+        gdp = &xf86ConfigLayout.inactives[j];
+        if (gdp->driver && !gdp->claimed &&
+            !xf86NameCmp(gdp->driver, drivername)) {
+            /* we have a matching driver that wasn't claimed yet */
+            pgdp = xnfrealloc(pgdp, (i + 2) * sizeof(GDevPtr));
+            pgdp[i++] = gdp;
+        }
+        j++;
     }
 
     /*
@@ -1347,9 +1364,9 @@ xf86MatchDevice(const char *drivername, GDevPtr **sectlist)
         pgdp[i] = NULL;
 
     if (sectlist)
-	*sectlist = pgdp;
+        *sectlist = pgdp;
     else
-	free(pgdp);
+        free(pgdp);
     return i;
 }
 
@@ -1357,12 +1374,11 @@ const char *
 xf86GetVisualName(int visual)
 {
     if (visual < 0 || visual > DirectColor)
-	return NULL;
+        return NULL;
 
     return xf86VisualNames[visual];
 }
 
-
 int
 xf86GetVerbosity(void)
 {
@@ -1375,84 +1391,72 @@ xf86GetPix24(void)
     return xf86Info.pixmap24;
 }
 
-
 int
 xf86GetDepth(void)
 {
     return xf86Depth;
 }
 
-
 rgb
 xf86GetWeight(void)
 {
     return xf86Weight;
 }
 
-
 Gamma
 xf86GetGamma(void)
 {
     return xf86Gamma;
 }
 
-
 Bool
 xf86GetFlipPixels(void)
 {
     return xf86FlipPixels;
 }
 
-
 const char *
 xf86GetServerName(void)
 {
     return xf86ServerName;
 }
 
-
 Bool
 xf86ServerIsExiting(void)
 {
     return (dispatchException & DE_TERMINATE) == DE_TERMINATE;
 }
 
-
 Bool
 xf86ServerIsResetting(void)
 {
     return xf86Resetting;
 }
 
-
 Bool
 xf86ServerIsInitialising(void)
 {
     return xf86Initialising;
 }
 
-
 Bool
 xf86ServerIsOnlyDetecting(void)
 {
     return xf86DoConfigure;
 }
 
-
 Bool
 xf86CaughtSignal(void)
 {
     return xf86Info.caughtSignal;
 }
 
-
 Bool
 xf86GetVidModeAllowNonLocal(void)
 {
     return xf86Info.vidModeAllowNonLocal;
 }
 
-
 Bool
 xf86GetVidModeEnabled(void)
 {
@@ -1465,21 +1469,18 @@ xf86GetModInDevAllowNonLocal(void)
     return xf86Info.miscModInDevAllowNonLocal;
 }
 
-
 Bool
 xf86GetModInDevEnabled(void)
 {
     return xf86Info.miscModInDevEnabled;
 }
 
-
 Bool
 xf86GetAllowMouseOpenFail(void)
 {
     return xf86Info.allowMouseOpenFail;
 }
 
-
 void
 xf86DisableRandR(void)
 {
@@ -1490,7 +1491,7 @@ xf86DisableRandR(void)
 CARD32
 xf86GetModuleVersion(pointer module)
 {
-    return (CARD32)LoaderGetModuleVersion(module);
+    return (CARD32) LoaderGetModuleVersion(module);
 }
 
 pointer
@@ -1500,9 +1501,9 @@ xf86LoadDrvSubModule(DriverPtr drv, const char *name)
     int errmaj = 0, errmin = 0;
 
     ret = LoadSubModule(drv->module, name, NULL, NULL, NULL, NULL,
-			&errmaj, &errmin);
+                        &errmaj, &errmin);
     if (!ret)
-	LoaderErrorMsg(NULL, name, errmaj, errmin);
+        LoaderErrorMsg(NULL, name, errmaj, errmin);
     return ret;
 }
 
@@ -1513,9 +1514,9 @@ xf86LoadSubModule(ScrnInfoPtr pScrn, const char *name)
     int errmaj = 0, errmin = 0;
 
     ret = LoadSubModule(pScrn->module, name, NULL, NULL, NULL, NULL,
-			&errmaj, &errmin);
+                        &errmaj, &errmin);
     if (!ret)
-	LoaderErrorMsg(pScrn->name, name, errmaj, errmin);
+        LoaderErrorMsg(pScrn->name, name, errmaj, errmin);
     return ret;
 }
 
@@ -1530,22 +1531,22 @@ xf86LoadOneModule(char *name, pointer opt)
     pointer mod;
 
     if (!name)
-	return NULL;
+        return NULL;
 
     /* Normalise the module name */
     Name = xf86NormalizeName(name);
 
     /* Skip empty names */
     if (Name == NULL)
-	return NULL;
+        return NULL;
     if (*Name == '\0') {
-	free(Name);
-	return NULL;
+        free(Name);
+        return NULL;
     }
 
     mod = LoadModule(Name, NULL, NULL, NULL, opt, NULL, &errmaj, &errmin);
     if (!mod)
-	LoaderErrorMsg(NULL, Name, errmaj, errmin);
+        LoaderErrorMsg(NULL, Name, errmaj, errmin);
     free(Name);
     return mod;
 }
@@ -1563,12 +1564,12 @@ xf86LoaderCheckSymbol(const char *name)
 }
 
 typedef enum {
-   OPTION_BACKING_STORE
+    OPTION_BACKING_STORE
 } BSOpts;
 
 static const OptionInfoRec BSOptions[] = {
-   { OPTION_BACKING_STORE, "BackingStore", OPTV_BOOLEAN, {0}, FALSE },
-   { -1,                   NULL,           OPTV_NONE,    {0}, FALSE }
+    {OPTION_BACKING_STORE, "BackingStore", OPTV_BOOLEAN, {0}, FALSE},
+    {-1, NULL, OPTV_NONE, {0}, FALSE}
 };
 
 void
@@ -1580,39 +1581,40 @@ xf86SetBackingStore(ScreenPtr pScreen)
     OptionInfoPtr options;
 
     options = xnfalloc(sizeof(BSOptions));
-    (void)memcpy(options, BSOptions, sizeof(BSOptions));
+    (void) memcpy(options, BSOptions, sizeof(BSOptions));
     xf86ProcessOptions(pScrn->scrnIndex, pScrn->options, options);
 
     /* check for commandline option here */
     if (xf86bsEnableFlag) {
-	from = X_CMDLINE;
-	useBS = TRUE;
-    } else if (xf86bsDisableFlag) {
-	from = X_CMDLINE;
-	useBS = FALSE;
-    } else {
-	if (xf86GetOptValBool(options, OPTION_BACKING_STORE, &useBS))
-	    from = X_CONFIG;
+        from = X_CMDLINE;
+        useBS = TRUE;
+    }
+    else if (xf86bsDisableFlag) {
+        from = X_CMDLINE;
+        useBS = FALSE;
+    }
+    else {
+        if (xf86GetOptValBool(options, OPTION_BACKING_STORE, &useBS))
+            from = X_CONFIG;
     }
     free(options);
     pScreen->backingStoreSupport = useBS ? Always : NotUseful;
     if (serverGeneration == 1)
-	xf86DrvMsg(pScreen->myNum, from, "Backing store %s\n",
-		   useBS ? "enabled" : "disabled");
+        xf86DrvMsg(pScreen->myNum, from, "Backing store %s\n",
+                   useBS ? "enabled" : "disabled");
 }
 
-
 typedef enum {
-   OPTION_SILKEN_MOUSE
+    OPTION_SILKEN_MOUSE
 } SMOpts;
 
 static const OptionInfoRec SMOptions[] = {
-   { OPTION_SILKEN_MOUSE, "SilkenMouse",   OPTV_BOOLEAN, {0}, FALSE },
-   { -1,                   NULL,           OPTV_NONE,    {0}, FALSE }
+    {OPTION_SILKEN_MOUSE, "SilkenMouse", OPTV_BOOLEAN, {0}, FALSE},
+    {-1, NULL, OPTV_NONE, {0}, FALSE}
 };
 
 void
-xf86SetSilkenMouse (ScreenPtr pScreen)
+xf86SetSilkenMouse(ScreenPtr pScreen)
 {
     Bool useSM = TRUE;
     MessageType from = X_DEFAULT;
@@ -1620,18 +1622,19 @@ xf86SetSilkenMouse (ScreenPtr pScreen)
     OptionInfoPtr options;
 
     options = xnfalloc(sizeof(SMOptions));
-    (void)memcpy(options, SMOptions, sizeof(SMOptions));
+    (void) memcpy(options, SMOptions, sizeof(SMOptions));
     xf86ProcessOptions(pScrn->scrnIndex, pScrn->options, options);
 
     /* check for commandline option here */
     /* disable if screen shares resources */
-	/* TODO VGA arb disable silken mouse */
+    /* TODO VGA arb disable silken mouse */
     if (xf86silkenMouseDisableFlag) {
         from = X_CMDLINE;
-	useSM = FALSE;
-    } else {
-	if (xf86GetOptValBool(options, OPTION_SILKEN_MOUSE, &useSM))
-	    from = X_CONFIG;
+        useSM = FALSE;
+    }
+    else {
+        if (xf86GetOptValBool(options, OPTION_SILKEN_MOUSE, &useSM))
+            from = X_CONFIG;
     }
     free(options);
     /*
@@ -1641,33 +1644,37 @@ xf86SetSilkenMouse (ScreenPtr pScreen)
      */
     pScrn->silkenMouse = useSM && xf86Info.useSIGIO && xf86SIGIOSupported();
     if (serverGeneration == 1)
-	xf86DrvMsg(pScreen->myNum, from, "Silken mouse %s\n",
-		   pScrn->silkenMouse ? "enabled" : "disabled");
+        xf86DrvMsg(pScreen->myNum, from, "Silken mouse %s\n",
+                   pScrn->silkenMouse ? "enabled" : "disabled");
 }
 
 /* Wrote this function for the PM2 Xv driver, preliminary. */
 
 pointer
 xf86FindXvOptions(int scrnIndex, int adaptor_index, char *port_name,
-		  char **adaptor_name, pointer *adaptor_options)
+                  char **adaptor_name, pointer *adaptor_options)
 {
     ScrnInfoPtr pScrn = xf86Screens[scrnIndex];
     confXvAdaptorPtr adaptor;
     int i;
 
     if (adaptor_index >= pScrn->confScreen->numxvadaptors) {
-	if (adaptor_name) *adaptor_name = NULL;
-	if (adaptor_options) *adaptor_options = NULL;
-	return NULL;
+        if (adaptor_name)
+            *adaptor_name = NULL;
+        if (adaptor_options)
+            *adaptor_options = NULL;
+        return NULL;
     }
 
     adaptor = &pScrn->confScreen->xvadaptors[adaptor_index];
-    if (adaptor_name) *adaptor_name = adaptor->identifier;
-    if (adaptor_options) *adaptor_options = adaptor->options;
+    if (adaptor_name)
+        *adaptor_name = adaptor->identifier;
+    if (adaptor_options)
+        *adaptor_options = adaptor->options;
 
     for (i = 0; i < adaptor->numports; i++)
-	if (!xf86NameCmp(adaptor->ports[i].identifier, port_name))
-	    return adaptor->ports[i].options;
+        if (!xf86NameCmp(adaptor->ports[i].identifier, port_name))
+            return adaptor->ports[i].options;
 
     return NULL;
 }
@@ -1678,39 +1685,41 @@ xf86FindXvOptions(int scrnIndex, int adaptor_index, char *port_name,
 
 static void
 xf86ConfigFbEntityInactive(EntityInfoPtr pEnt, EntityProc init,
-			   EntityProc enter, EntityProc leave, pointer private)
+                           EntityProc enter, EntityProc leave, pointer private)
 {
     ScrnInfoPtr pScrn;
 
     if ((pScrn = xf86FindScreenForEntity(pEnt->index)))
-	xf86RemoveEntityFromScreen(pScrn,pEnt->index);
-    xf86SetEntityFuncs(pEnt->index,init,enter,leave,private);
+        xf86RemoveEntityFromScreen(pScrn, pEnt->index);
+    xf86SetEntityFuncs(pEnt->index, init, enter, leave, private);
 }
 
 ScrnInfoPtr
 xf86ConfigFbEntity(ScrnInfoPtr pScrn, int scrnFlag, int entityIndex,
-		   EntityProc init, EntityProc enter, EntityProc leave,
-		   pointer private)
+                   EntityProc init, EntityProc enter, EntityProc leave,
+                   pointer private)
 {
     EntityInfoPtr pEnt = xf86GetEntityInfo(entityIndex);
-    if (!pEnt) return pScrn;
+
+    if (!pEnt)
+        return pScrn;
 
     if (!(pEnt->location.type == BUS_NONE)) {
-	free(pEnt);
-	return pScrn;
+        free(pEnt);
+        return pScrn;
     }
 
     if (!pEnt->active) {
-	xf86ConfigFbEntityInactive(pEnt, init,  enter, leave,  private);
-	free(pEnt);
-	return pScrn;
+        xf86ConfigFbEntityInactive(pEnt, init, enter, leave, private);
+        free(pEnt);
+        return pScrn;
     }
 
     if (!pScrn)
-	pScrn = xf86AllocateScreen(pEnt->driver,scrnFlag);
-    xf86AddEntityToScreen(pScrn,entityIndex);
+        pScrn = xf86AllocateScreen(pEnt->driver, scrnFlag);
+    xf86AddEntityToScreen(pScrn, entityIndex);
 
-    xf86SetEntityFuncs(entityIndex,init,enter,leave,private);
+    xf86SetEntityFuncs(entityIndex, init, enter, leave, private);
 
     free(pEnt);
     return pScrn;
@@ -1722,48 +1731,49 @@ xf86IsScreenPrimary(int scrnIndex)
     ScrnInfoPtr pScrn = xf86Screens[scrnIndex];
     int i;
 
-    for (i=0 ; i < pScrn->numEntities; i++) {
-	if (xf86IsEntityPrimary(i))
-	    return TRUE;
+    for (i = 0; i < pScrn->numEntities; i++) {
+        if (xf86IsEntityPrimary(i))
+            return TRUE;
     }
     return FALSE;
 }
 
 int
 xf86RegisterRootWindowProperty(int ScrnIndex, Atom property, Atom type,
-			       int format, unsigned long len, pointer value )
+                               int format, unsigned long len, pointer value)
 {
     RootWinPropPtr pNewProp = NULL, pRegProp;
     Bool existing = FALSE;
 
     DebugF("xf86RegisterRootWindowProperty(%d, %ld, %ld, %d, %ld, %p)\n",
-	   ScrnIndex, property, type, format, len, value);
+           ScrnIndex, property, type, format, len, value);
 
-    if (ScrnIndex<0 || ScrnIndex>=xf86NumScreens) {
-      return BadMatch;
+    if (ScrnIndex < 0 || ScrnIndex >= xf86NumScreens) {
+        return BadMatch;
     }
 
     if (xf86RegisteredPropertiesTable &&
-	xf86RegisteredPropertiesTable[ScrnIndex]) {
-      for (pNewProp = xf86RegisteredPropertiesTable[ScrnIndex];
-	   pNewProp; pNewProp = pNewProp->next) {
-	if (strcmp(pNewProp->name, NameForAtom(property)) == 0)
-	  break;
-      }
+        xf86RegisteredPropertiesTable[ScrnIndex]) {
+        for (pNewProp = xf86RegisteredPropertiesTable[ScrnIndex];
+             pNewProp; pNewProp = pNewProp->next) {
+            if (strcmp(pNewProp->name, NameForAtom(property)) == 0)
+                break;
+        }
     }
 
     if (!pNewProp) {
-      if ((pNewProp = (RootWinPropPtr)malloc(sizeof(RootWinProp))) == NULL) {
-	return BadAlloc;
-      }
-      /*
-       * We will put this property at the end of the list so that
-       * the changes are made in the order they were requested.
-       */
-      pNewProp->next = NULL;
-    } else {
-      free(pNewProp->name);
-      existing = TRUE;
+        if ((pNewProp = (RootWinPropPtr) malloc(sizeof(RootWinProp))) == NULL) {
+            return BadAlloc;
+        }
+        /*
+         * We will put this property at the end of the list so that
+         * the changes are made in the order they were requested.
+         */
+        pNewProp->next = NULL;
+    }
+    else {
+        free(pNewProp->name);
+        existing = TRUE;
     }
 
     pNewProp->name = xnfstrdup(NameForAtom(property));
@@ -1775,28 +1785,29 @@ xf86RegisterRootWindowProperty(int ScrnIndex, Atom property, Atom type,
     DebugF("new property filled\n");
 
     if (xf86RegisteredPropertiesTable == NULL) {
-      DebugF("creating xf86RegisteredPropertiesTable[] size %d\n",
-	     xf86NumScreens);
-      xf86RegisteredPropertiesTable =
-	  xnfcalloc(sizeof(RootWinProp), xf86NumScreens);
+        DebugF("creating xf86RegisteredPropertiesTable[] size %d\n",
+               xf86NumScreens);
+        xf86RegisteredPropertiesTable =
+            xnfcalloc(sizeof(RootWinProp), xf86NumScreens);
     }
 
     DebugF("xf86RegisteredPropertiesTable %p\n",
-	   (void *)xf86RegisteredPropertiesTable);
+           (void *) xf86RegisteredPropertiesTable);
     DebugF("xf86RegisteredPropertiesTable[%d] %p\n",
-	   ScrnIndex, (void *)xf86RegisteredPropertiesTable[ScrnIndex]);
+           ScrnIndex, (void *) xf86RegisteredPropertiesTable[ScrnIndex]);
 
     if (!existing) {
-      if ( xf86RegisteredPropertiesTable[ScrnIndex] == NULL) {
-	xf86RegisteredPropertiesTable[ScrnIndex] = pNewProp;
-      } else {
-	pRegProp = xf86RegisteredPropertiesTable[ScrnIndex];
-	while (pRegProp->next != NULL) {
-	  DebugF("- next %p\n", (void *)pRegProp);
-	  pRegProp = pRegProp->next;
+        if (xf86RegisteredPropertiesTable[ScrnIndex] == NULL) {
+            xf86RegisteredPropertiesTable[ScrnIndex] = pNewProp;
+        }
+        else {
+            pRegProp = xf86RegisteredPropertiesTable[ScrnIndex];
+            while (pRegProp->next != NULL) {
+                DebugF("- next %p\n", (void *) pRegProp);
+                pRegProp = pRegProp->next;
+            }
+            pRegProp->next = pNewProp;
         }
-	pRegProp->next = pNewProp;
-      }
     }
     DebugF("xf86RegisterRootWindowProperty succeeded\n");
     return Success;
@@ -1805,16 +1816,16 @@ xf86RegisterRootWindowProperty(int ScrnIndex, Atom property, Atom type,
 Bool
 xf86IsUnblank(int mode)
 {
-    switch(mode) {
+    switch (mode) {
     case SCREEN_SAVER_OFF:
     case SCREEN_SAVER_FORCER:
-	return TRUE;
+        return TRUE;
     case SCREEN_SAVER_ON:
     case SCREEN_SAVER_CYCLE:
-	return FALSE;
+        return FALSE;
     default:
-	xf86MsgVerb(X_WARNING, 0, "Unexpected save screen mode: %d\n", mode);
-	return TRUE;
+        xf86MsgVerb(X_WARNING, 0, "Unexpected save screen mode: %d\n", mode);
+        return TRUE;
     }
 }