[patch] matrox fix for corruptect screens

matrox-ppc64-fix.patch (text/plain), 2.67 KB, created by Tom Gall (RETIRED) on 2005-04-05 13:14:49 UTC

diff -Naru a/drivers/video/matrox/matroxfb_accel.c b/drivers/video/matrox/matroxfb_accel.c
--- a/drivers/video/matrox/matroxfb_accel.c	2005-04-03 13:50:21 -07:00
+++ b/drivers/video/matrox/matroxfb_accel.c	2005-04-03 13:50:21 -07:00
@@ -438,13 +438,21 @@
 		} else if (step == 1) {
 			/* Special case for 1..8bit widths */
 			while (height--) {
-				mga_writel(mmio, 0, *chardata);
+#if defined(__BIG_ENDIAN)
+				fb_writel((*chardata) << 24, mmio.vaddr);
+#else
+				fb_writel(*chardata, mmio.vaddr);
+#endif
 				chardata++;
 			}
 		} else if (step == 2) {
 			/* Special case for 9..15bit widths */
 			while (height--) {
-				mga_writel(mmio, 0, *(u_int16_t*)chardata);
+#if defined(__BIG_ENDIAN)
+				fb_writel((*(u_int16_t*)chardata) << 16, mmio.vaddr);
+#else
+				fb_writel(*(u_int16_t*)chardata, mmio.vaddr);
+#endif
 				chardata += 2;
 			}
 		} else {
@@ -454,7 +462,7 @@
 				
 				for (i = 0; i < step; i += 4) {
 					/* Hope that there are at least three readable bytes beyond the end of bitmap */
-					mga_writel(mmio, 0, get_unaligned((u_int32_t*)(chardata + i)));
+					fb_writel(get_unaligned((u_int32_t*)(chardata + i)),mmio.vaddr);
 				}
 				chardata += step;
 			}
diff -Naru a/drivers/video/matrox/matroxfb_base.h b/drivers/video/matrox/matroxfb_base.h
--- a/drivers/video/matrox/matroxfb_base.h	2005-04-03 13:50:21 -07:00
+++ b/drivers/video/matrox/matroxfb_base.h	2005-04-03 13:50:21 -07:00
@@ -170,14 +170,14 @@
 
 	if ((unsigned long)src & 3) {
 		while (len >= 4) {
-			writel(get_unaligned((u32 *)src), addr);
+			fb_writel(get_unaligned((u32 *)src), addr);
 			addr++;
 			len -= 4;
 			src += 4;
 		}
 	} else {
 		while (len >= 4) {
-			writel(*(u32 *)src, addr);
+			fb_writel(*(u32 *)src, addr);
 			addr++;
 			len -= 4;
 			src += 4;
# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
#   2005/03/31 08:48:52-08:00 [e-mail] 
#   [PATCH] Fix matroxfb on big-endian hardware
#   
#   There was too much/too few byteswapping done by driver and hardware in
#   matroxfb on big endian hardware.  Change fixes mirrored/split/corrupted
#   letters seen on screen when using accelerated matroxfb mode.
#   
#   Patch was tested on Mips (by Peter) and x86-64 (by Petr).
#   
#   Signed-off-by: Peter 'p2' De Schrijver <[e-mail]>
#   Signed-off-by: Petr Vandrovec <[e-mail]>
#   Signed-off-by: Andrew Morton <[e-mail]>
#   Signed-off-by: Linus Torvalds <[e-mail]>
# 
# drivers/video/matrox/matroxfb_accel.c
#   2005/03/31 03:06:52-08:00 [e-mail] +11 -3
#   Fix matroxfb on big-endian hardware
# 
# drivers/video/matrox/matroxfb_base.h
#   2005/03/31 03:06:52-08:00 [e-mail] +2 -2
#   Fix matroxfb on big-endian hardware
#