Index: programs/Xserver/hw/xfree86/drivers/newport/newport.h
===================================================================
RCS file: /cvs/xc/programs/Xserver/hw/xfree86/drivers/newport/newport.h,v
retrieving revision 1.8
diff -u -u -r1.8 newport.h
--- programs/Xserver/hw/xfree86/drivers/newport/newport.h	2001/12/19 21:31:21	1.8
+++ programs/Xserver/hw/xfree86/drivers/newport/newport.h	2002/07/15 23:23:01
@@ -51,7 +51,7 @@
 	npireg_t drawmode1;		/* REX3 drawmode1 common to all drawing operations */
 
 	/* ShadowFB stuff: */
-	pointer ShadowPtr;
+	CARD32* ShadowPtr;
 	unsigned long int ShadowPitch;
 	unsigned int Bpp;		/* Bytes per pixel */
 
Index: programs/Xserver/hw/xfree86/drivers/newport/newport_driver.c
===================================================================
RCS file: /cvs/xc/programs/Xserver/hw/xfree86/drivers/newport/newport_driver.c,v
retrieving revision 1.19
diff -u -u -r1.19 newport_driver.c
--- programs/Xserver/hw/xfree86/drivers/newport/newport_driver.c	2002/01/04 21:22:33	1.19
+++ programs/Xserver/hw/xfree86/drivers/newport/newport_driver.c	2002/07/15 23:23:03
@@ -681,7 +689,11 @@
 					/* turn on RGB mode */	
 					NPORT_DMODE1_RGBMD | 
 					/* turn on 8888 = RGBA pixel packing */
-					NPORT_DMODE1_HD32 | NPORT_DMODE1_RWPCKD; 
+					NPORT_DMODE1_HD32 | NPORT_DMODE1_RWPCKD;
+		/* After setting up XMAP9 we have to reinitialize the CMAP for
+		 * whatever reason (the docs say nothing about it). RestorePalette()
+		 * is just a lazy way to do this */
+		NewportRestorePalette( pScrn );
 	}
 	/* blank the framebuffer */
 	NewportWait(pNewportRegs);
Index: programs/Xserver/hw/xfree86/drivers/newport/newport_shadow.c
===================================================================
RCS file: /cvs/xc/programs/Xserver/hw/xfree86/drivers/newport/newport_shadow.c,v
retrieving revision 1.2
diff -u -u -r1.2 newport_shadow.c
--- programs/Xserver/hw/xfree86/drivers/newport/newport_shadow.c	2001/11/23 19:50:45	1.2
+++ programs/Xserver/hw/xfree86/drivers/newport/newport_shadow.c	2002/07/15 23:23:05
@@ -8,13 +8,13 @@
 void
 NewportRefreshArea8(ScrnInfoPtr pScrn, int num, BoxPtr pbox)
 {
-	unsigned long *base, *src;
 	int dx, dy, x;
+	CARD32 *base, *src;
 	NewportPtr pNewport = NEWPORTPTR(pScrn);
 	NewportRegsPtr pNewportRegs = pNewport->pNewportRegs;
 
-#define RA8_PIXELS	4	/* burst 4 pixels each time */
-#define RA8_PIXEL_SHIFT 2 	/* 4 Pixels on each burst   */
+#define RA8_BYTES	4	/* burst 4 pixels each time */
+#define RA8_BYTE_SHIFT  2 	/* 4 Pixels on each burst, so divide ShadowPitch by 4 */
 #define RA8_MASK        0xffc   /* move to 4 byte boundary   */
 
 	TRACE_ENTER("NewportRefreshArea8");
@@ -25,35 +25,36 @@
 	while(num--) {
 		NewportWait(pNewportRegs);
 		x = pbox->x1 & RA8_MASK;  	/* move x to 4 byte boundary */
-		base = (unsigned long*)pNewport->ShadowPtr 
-				+ (pbox->y1 * (pNewport->ShadowPitch >> RA8_PIXEL_SHIFT) ) 
-				+ ( x >> RA8_PIXEL_SHIFT);
+		base = pNewport->ShadowPtr 
+				+ (pbox->y1 * (pNewport->ShadowPitch >> RA8_BYTE_SHIFT) ) 
+				+ ( x >> RA8_BYTE_SHIFT);
 
 		pNewportRegs->set.xystarti = (x << 16) | pbox->y1;
-		pNewportRegs->set.xyendi = (pbox->x2 << 16) | pbox->y2;
+		pNewportRegs->set.xyendi = ((pbox->x2-1) << 16) | (pbox->y2-1);
 
-		for ( dy = pbox->y1; dy <= pbox->y2; dy++) {
+		for ( dy = pbox->y1; dy < pbox->y2; dy++) {
 
 			src = base;
-			for ( dx = x; dx <= pbox->x2; dx += RA8_PIXELS) {
+			for ( dx = x; dx < pbox->x2; dx += RA8_BYTES) {
 				pNewportRegs->go.hostrw0 = *src;  
 				src++;
 			}
-			base += ( pNewport->ShadowPitch >> RA8_PIXEL_SHIFT );
+			base += ( pNewport->ShadowPitch >> RA8_BYTE_SHIFT );
 		}
 		pbox++;
 	}
 	TRACE_EXIT("NewportRefreshArea8");
 }
 
+
 void
 NewportRefreshArea24(ScrnInfoPtr pScrn, int num, BoxPtr pbox)
 {
-	NewportPtr pNewport = NEWPORTPTR(pScrn);
-	NewportRegsPtr pNewportRegs = pNewport->pNewportRegs;
 	int dx, dy;
 	CARD8 *src, *base;
 	CARD32 dest;
+	NewportPtr pNewport = NEWPORTPTR(pScrn);
+	NewportRegsPtr pNewportRegs = pNewport->pNewportRegs;
 
 	TRACE_ENTER("NewportRefreshArea24");
 	NewportWait(pNewportRegs);
@@ -67,11 +68,14 @@
 		base = (CARD8*)pNewport->ShadowPtr + pbox->y1 * pNewport->ShadowPitch + pbox->x1 * 3;
 		
 		pNewportRegs->set.xystarti = (pbox->x1 << 16) | pbox->y1;
-		pNewportRegs->set.xyendi = (pbox->x2 << 16) | pbox->y2;
+		pNewportRegs->set.xyendi = ((pbox->x2-1) << 16) | (pbox->y2-1);
 		
-		for ( dy = pbox->y1; dy <= pbox->y2; dy++) {
+		for ( dy = pbox->y1; dy < pbox->y2; dy++) {
 			src = base;
-			for ( dx = pbox->x1 ;  dx <= pbox->x2 ; dx++) {
+			for ( dx = pbox->x1 ;  dx < pbox->x2 ; dx++) {
+				/* Removing these shifts by using 32bpp fb
+				 * yields < 2 percent performance gain and wastes 25% memory 
+				 */
 				dest = src[0] | src[1] << 8 | src[2] << 16;
 				pNewportRegs->go.hostrw0 = dest;	
 				src+=3;
@@ -82,3 +86,4 @@
  	}
 	TRACE_EXIT("NewportRefreshArea24");
 }
+
