PATCH: cyber2010 framebuffer on ARM Netwinder fix...

[Woody Suwalski <woodys@xandros.com>]

[-- Attachment #1: Type: text/plain, Size: 1049 bytes --]

Martin Michlmayr wrote:
> * Stuart Anderson <anderson@netsweng.com> [2007-01-05 09:40]:
>   
>> shark w/o any changes to the kernel. I dug a bit further, both in the
>> driver, and in the HW spec for the shark, and discovered that the video
>> chip on the shark is connected via the VL bus, not the PCI bus. The
>> shark does have a VL-PCI bridge, but there doesn't seem to be anything
>> connected to the PCI side of it (which matches what lspci says). The
>> function containing the patch in question doesn't appear to even run on
>> the shark (there is a VL version that is #ifdef SHARK'd), so I'd have
>> to say the patch would have not impact on the shark.
>>     
>
> OK, good news.  Thanks for checking.  Woody, can you submit the patch
> (with proper intentation) to linux-fbdev-devel@lists.sourceforge.net
>   

As suggested - I am sending this patch to fbdev-devel....

The Netwinder machines with Cyber2010 crash badly when starting Xserver.
The workaround is to disable pci burst option for this revision of video 
chip.

Thanks, Woody


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: netwinder_cyber2010_burst.patch --]
[-- Type: text/x-patch; name="netwinder_cyber2010_burst.patch", Size: 1271 bytes --]

The Netwinder machines with Cyber2010 crash badly when starting Xserver.
The workaround is to disable pci burst option for this revision of video chip.

Signed-off-by: Woody Suwalski <woodys@xandros.com>

---

--- a/drivers/video/cyber2000fb.c	2006-11-29 16:57:37.000000000 -0500
+++ b/drivers/video/cyber2000fb.c	2007-01-09 12:20:01.000000000 -0500
@@ -1539,16 +1539,24 @@ static int cyberpro_pci_enable_mmio(stru
 	/*
 	 * Allow the CyberPro to accept PCI burst accesses
 	 */
-	val = cyber2000_grphr(EXT_BUS_CTL, cfb);
-	if (!(val & EXT_BUS_CTL_PCIBURST_WRITE)) {
-		printk(KERN_INFO "%s: enabling PCI bursts\n", cfb->fb.fix.id);
-
-		val |= EXT_BUS_CTL_PCIBURST_WRITE;
-
-		if (cfb->id == ID_CYBERPRO_5000)
-			val |= EXT_BUS_CTL_PCIBURST_READ;
-
-		cyber2000_grphw(EXT_BUS_CTL, val, cfb);
+	if (cfb->id == ID_CYBERPRO_2010)
+	{
+		printk(KERN_INFO "%s: NOT enabling PCI bursts\n", cfb->fb.fix.id);
+	}
+	else
+	{
+		val = cyber2000_grphr(EXT_BUS_CTL, cfb);
+		if (!(val & EXT_BUS_CTL_PCIBURST_WRITE)) {
+			printk(KERN_INFO "%s: enabling PCI bursts\n",
+				cfb->fb.fix.id);
+	
+			val |= EXT_BUS_CTL_PCIBURST_WRITE;
+	
+			if (cfb->id == ID_CYBERPRO_5000)
+				val |= EXT_BUS_CTL_PCIBURST_READ;
+	
+			cyber2000_grphw(EXT_BUS_CTL, val, cfb);
+		}
 	}
 
 	return 0;