Thread (5 messages) flat view 5 messages, 2 authors, 2004-05-19

Re: vga16fb broke

From: James Simmons <hidden>
Date: 2004-05-19 00:18:50
Also in: lkml

quoted hunk ↗ jump to hunk
I have pondered your email at length and have failed to understand it.

I _think_ you're saying that we need to do this, which will fix x86:
--- 25/drivers/video/vga16fb.c~vga16fb-fix	Tue May 18 17:10:14 2004
+++ 25-akpm/drivers/video/vga16fb.c	Tue May 18 17:10:39 2004
@@ -1347,7 +1347,7 @@ int __init vga16fb_init(void)
 
 	/* XXX share VGA_FB_PHYS and I/O region with vgacon and others */
 
-	vga16fb.screen_base = ioremap(VGA_MAP_MEM(VGA_FB_PHYS), VGA_FB_PHYS_LEN);
+	vga16fb.screen_base = VGA_MAP_MEM(VGA_FB_PHYS);
 	if (!vga16fb.screen_base) {
 		printk(KERN_ERR "vga16fb: unable to map device\n");
 		ret = -ENOMEM;
This will make the driver on all platforms. 
_
and that ARM and others need to teach their VGA_MAP_MEM() to do an internal
ioremap().

Or do you mean something else?  Please be more clear?
I like to see the VGA_MAP_MEM hack go away and be replaced with ioremap.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help