Index: gbefb.c
===================================================================
RCS file: /home/cvs/linux/drivers/video/gbefb.c,v
retrieving revision 1.9
diff -u -r1.9 gbefb.c
--- gbefb.c	25 Oct 2004 20:44:39 -0000	1.9
+++ gbefb.c	29 Oct 2004 08:30:11 -0000
@@ -29,6 +29,7 @@
 #endif
 #include <asm/byteorder.h>
 #include <asm/io.h>
+#include <asm/pgalloc.h>
 #include <asm/tlbflush.h>
 
 #include <video/gbe.h>
@@ -386,6 +387,9 @@
 		break;
 
 	case 1:		/* blank */
+	case 2:
+	case 3:
+	case 4:
 		gbe_turn_off();
 		break;
 
@@ -652,16 +656,20 @@
 	gbe_set_timing_info(&par->timing);
 
 	/* initialize DIDs */
+	/* Update the current visual, otherwise depth change with fbset won't work */
 	val = 0;
 	switch (bytesPerPixel) {
 	case 1:
 		SET_GBE_FIELD(WID, TYP, val, GBE_CMODE_I8);
+		info->fix.visual = FB_VISUAL_PSEUDOCOLOR;
 		break;
 	case 2:
 		SET_GBE_FIELD(WID, TYP, val, GBE_CMODE_ARGB5);
+		info->fix.visual = FB_VISUAL_TRUECOLOR;
 		break;
 	case 4:
 		SET_GBE_FIELD(WID, TYP, val, GBE_CMODE_RGB8);
+		info->fix.visual = FB_VISUAL_TRUECOLOR;
 		break;
 	}
 	SET_GBE_FIELD(WID, BUF, val, GBE_BMODE_BOTH);
@@ -924,8 +932,8 @@
 		var->yoffset = info->var.yoffset;
 	}
 
-	/* No grayscale for now */
-	var->grayscale = 0;
+	var->xoffset = 0;
+	var->yoffset = 0;
 
 	/* Memory limit */
 	line_length = var->xres_virtual * var->bits_per_pixel / 8;
@@ -1028,6 +1036,7 @@
 	return 0;
 }
 
+
 static struct fb_ops gbefb_ops = {
 	.owner		= THIS_MODULE,
 	.fb_check_var	= gbefb_check_var,
@@ -1112,6 +1121,7 @@
 	struct fb_info *info;
 	struct gbefb_par *par;
 	struct platform_device *p_dev = to_platform_device(dev);
+	unsigned long page, gbe_mem_cac;
 
 	info = framebuffer_alloc(sizeof(struct gbefb_par), &p_dev->dev);
 	if (!info)
@@ -1122,7 +1132,7 @@
 
 	if (fb_get_options("gbefb", &option))
 		return -ENODEV;
-	gbefb_setup(options);
+	gbefb_setup(option);
 #endif
 
 	if (!request_mem_region(GBE_BASE, sizeof(struct sgi_gbe), "GBE")) {
@@ -1131,6 +1141,8 @@
 		goto out_release_framebuffer;
 	}
 
+	dev->coherent_dma_mask = 0xffffffff;
+
 	gbe = (struct sgi_gbe *) ioremap(GBE_BASE, sizeof(struct sgi_gbe));
 	if (!gbe) {
 		printk(KERN_ERR "gbefb: couldn't map mmio region\n");
@@ -1140,7 +1152,7 @@
 	gbe_revision = gbe->ctrlstat & 15;
 
 	gbe_tiles.cpu =
-		dma_alloc_coherent(NULL, GBE_TLB_SIZE * sizeof(uint16_t),
+		dma_alloc_coherent(dev, GBE_TLB_SIZE * sizeof(uint16_t),
 				   &gbe_tiles.dma, GFP_KERNEL);
 	if (!gbe_tiles.cpu) {
 		printk(KERN_ERR "gbefb: couldn't allocate tiles table\n");
@@ -1149,17 +1161,35 @@
 	}
 
 	if (gbe_mem_phys) {
+		printk(KERN_ERR "gbefb: memory already allocated\n");
 		/* memory was allocated at boot time */
 		gbe_mem = ioremap_nocache(gbe_mem_phys, gbe_mem_size);
 		gbe_dma_addr = 0;
 	} else {
+		printk(KERN_ERR "gbefb: allocating memory\n");
 		/* try to allocate memory with the classical allocator
 		 * this has high chance to fail on low memory machines */
-		gbe_mem = dma_alloc_coherent(NULL, gbe_mem_size, &gbe_dma_addr,
+		gbe_mem = dma_alloc_coherent(dev, gbe_mem_size, &gbe_dma_addr,
 					     GFP_KERNEL);
 		gbe_mem_phys = (unsigned long) gbe_dma_addr;
+
 	}
 
+	/* dma_alloc_coherent gives us a "wrong" (UNCAC_ADDR) address, so doing 
+	 * SetPageReserved there has no effect. Use the CAC_ADDR address to do
+	 * SetPageReserved in the correct place. This fixed mmap access...
+	 * XXX: is the above correct ? 
+	 */
+	gbe_mem_cac = (unsigned long)CAC_ADDR(gbe_mem);
+
+	for (page = (unsigned long)gbe_mem_cac; 
+		page < PAGE_ALIGN( ((unsigned long)(gbe_mem_cac + gbe_mem_size)));
+		page += PAGE_SIZE)
+			SetPageReserved(virt_to_page(page));
+
+	/* Probably not needed but I'm to lazy to test :) */
+	flush_cache_all();
+
 #ifdef CONFIG_X86
 	mtrr_add(gbe_mem_phys, gbe_mem_size, MTRR_TYPE_WRCOMB, 1);
 #endif