From 2f8352df6488476b0c1a46798eca5dd38827444b Mon Sep 17 00:00:00 2001
From: Ian Romanick <idr@umwelt.(none)>
Date: Fri, 4 Aug 2006 15:57:08 -0700
Subject: Merge SavageMapMMIO and SavageMapFB into a single, new function
 called SavageMapMem.  Eliminate the MapBaseDense cruft.

---
 src/savage_driver.c | 45 ++++++++++-----------------------------------
 src/savage_driver.h |  1 -
 2 files changed, 10 insertions(+), 36 deletions(-)

diff --git a/src/savage_driver.c b/src/savage_driver.c
index 217e625..422bcfd 100644
--- a/src/savage_driver.c
+++ b/src/savage_driver.c
@@ -64,8 +64,7 @@ static ModeStatus SavageValidMode(int index, DisplayModePtr mode,
 				  Bool verbose, int flags);
 
 void SavageDGAInit(ScreenPtr);
-static Bool SavageMapMMIO(ScrnInfoPtr pScrn);
-static Bool SavageMapFB(ScrnInfoPtr pScrn);
+static Bool SavageMapMem(ScrnInfoPtr pScrn);
 static void SavageUnmapMem(ScrnInfoPtr pScrn, int All);
 static Bool SavageModeInit(ScrnInfoPtr pScrn, DisplayModePtr mode);
 static Bool SavageCloseScreen(int scrnIndex, ScreenPtr pScreen);
@@ -1650,13 +1649,6 @@ static Bool SavagePreInit(ScrnInfoPtr pScrn, int flags)
 
     /* maybe throw in some more sanity checks here */
 
-    if (!SavageMapMMIO(pScrn)) {
-	SavageFreeRec(pScrn);
-        vbeFree(psav->pVbe);
-	psav->pVbe = NULL;
-	return FALSE;
-    }
-
     vgaCRIndex = psav->vgaIOBase + 4;
     vgaCRReg = psav->vgaIOBase + 5;
 
@@ -1853,6 +1845,13 @@ static Bool SavagePreInit(ScrnInfoPtr pScrn, int flags)
     VGAOUT8(vgaCRReg, cr66 & ~0x02);	/* clear reset flag */
     usleep(10000);
 
+    if (!SavageMapMem(pScrn)) {
+	SavageFreeRec(pScrn);
+        vbeFree(psav->pVbe);
+	psav->pVbe = NULL;
+	return FALSE;
+    }
+
     /* Set status word positions based on chip type. */
     SavageInitStatus(pScrn);
 
@@ -2811,11 +2810,11 @@ static void SavageWriteMode(ScrnInfoPtr pScrn, vgaRegPtr vgaSavePtr,
 }
 
 
-static Bool SavageMapMMIO(ScrnInfoPtr pScrn)
+static Bool SavageMapMem(ScrnInfoPtr pScrn)
 {
     SavagePtr psav;
 
-    TRACE(("SavageMapMMIO()\n"));
+    TRACE(("SavageMapMem()\n"));
 
     psav = SAVPTR(pScrn);
 
@@ -2835,12 +2834,6 @@ static Bool SavageMapMMIO(ScrnInfoPtr pScrn)
     psav->MapBase = xf86MapPciMem(pScrn->scrnIndex, VIDMEM_MMIO, psav->PciTag,
 				  psav->MmioBase,
 				  SAVAGE_NEWMMIO_REGSIZE);
-#if 0
-    psav->MapBaseDense = xf86MapPciMem(pScrn->scrnIndex, VIDMEM_MMIO_32BIT,
-				       psav->PciTag,
-				       psav->PciInfo->memBase[0],
-				       0x8000);
-#endif
     if (!psav->MapBase) {
 	xf86DrvMsg(pScrn->scrnIndex, X_ERROR,
 		   "Internal error: cound not map registers\n");
@@ -2851,16 +2844,6 @@ static Bool SavageMapMMIO(ScrnInfoPtr pScrn)
 
     SavageEnableMMIO(pScrn);
 
-    return TRUE;
-}
-
-
-static Bool SavageMapFB(ScrnInfoPtr pScrn)
-{
-    SavagePtr psav = SAVPTR(pScrn);
-
-    TRACE(("SavageMapFB()\n"));
-
     xf86DrvMsg( pScrn->scrnIndex, X_PROBED,
 	"mapping framebuffer @ 0x%lx with size 0x%x\n", 
 	psav->FrameBufferBase, psav->videoRambytes);
@@ -2951,11 +2934,6 @@ static void SavageUnmapMem(ScrnInfoPtr pScrn, int All)
 	psav->FBBase = 0;
     }
 
-#if 0
-    xf86UnMapVidMem(pScrn->scrnIndex, (pointer)psav->MapBaseDense,
-		    0x8000);
-#endif
-
     return;
 }
 
@@ -3080,9 +3058,6 @@ static Bool SavageScreenInit(int scrnIndex, ScreenPtr pScreen,
  
     SavageEnableMMIO(pScrn);
 
-    if (!SavageMapFB(pScrn))
-	return FALSE;
- 
     psav->FBStart2nd = 0;
 
     if (psav->overlayDepth) {
diff --git a/src/savage_driver.h b/src/savage_driver.h
index b99c5b5..805aa64 100644
--- a/src/savage_driver.h
+++ b/src/savage_driver.h
@@ -295,7 +295,6 @@ typedef struct _Savage {
     /* These are linear addresses. */
     unsigned char*	MapBase;
     unsigned char*	BciMem;
-    unsigned char*	MapBaseDense;
     unsigned char*	FBBase;
     unsigned char*	ApertureMap;
     unsigned char*	FBStart;
-- 
cgit v1.2.3