On Thu, 11 Jul 2002, Kevin E Martin wrote:

> I've just checked in a large patch from ATI.  Here's the CHANGELOG
> entry:
> 
>  199. ATI patch to:
>       - Fix Dell OEM VE card support
>       - Add better clone mode support
>       - Fix large panel (>= 1600x1200) detection and initialization problems
>       - Remove "PanelSize" and "CrtScreen" options since they are no longer
>         needed with new CloneMode and improved flat panel support
>       - Add "DDCMode" option to detect and use DDC modes
>       - Add "PanelOff" option to disable panel on laptops
>       - Fix corrupted console problem
>       - Other misc fixes
>       (#A.1043, Hui Yu@ATI).
> 
> I'd like get some feedback on how it works for others with both desktop
> and laptop Radeons.  I'll start working on the other patches that have
> been sent in shortly.

This breaks DDC on my white box Radeon 7000.
Patch to fix this attached.

lspci -v -n
01:00.0 Class 0300: 1002:5159
        Subsystem: 174b:7112
        ...

http://www.pcisig.com/membership/vid_search/by_vendor_id/?vendor_id=174b&type=h
suggests that 174b is PC Partner.

http://www.pcpartner.com/product-ATI.html
suggests they make motherboards wth onchip ATI graphics. 

-- 
Dr. Andrew C. Aitchison         Computer Officer, DPMMS, Cambridge
[EMAIL PROTECTED]   http://www.dpmms.cam.ac.uk/~werdna
Index: radeon_driver.c
===================================================================
RCS file: 
/home/CVS/XFree86/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c,v
retrieving revision 1.58
diff -u -r1.58 radeon_driver.c
--- radeon_driver.c     2002/07/11 20:11:51     1.58
+++ radeon_driver.c     2002/07/12 11:46:37
@@ -1240,7 +1240,8 @@
      * It needs some special handlings for it's 2nd head to work.
      */
     info->IsDell = FALSE;
-    if (info->PciInfo->subsysCard & (1 << 12)) { /* DELL's signature */
+    if (info->PciInfo->subsysVendor == PCI_VENDOR_DELL && 
+       info->PciInfo->subsysCard & (1 << 12)) { /* DELL's signature */
        if (info->PciInfo->subsysCard & 0xb00) {
            info->IsDell = TRUE;
            info->DellType = 2; /* DVI+DVI config, this seems to be the
Index: xf86PciInfo.h
===================================================================
RCS file: /home/CVS/XFree86/xc/programs/Xserver/hw/xfree86/common/xf86PciInfo.h,v
retrieving revision 1.128
diff -u -r1.128 xf86PciInfo.h
--- xf86PciInfo.h       2002/06/30 15:55:26     1.128
+++ xf86PciInfo.h       2002/07/12 11:43:53
@@ -63,6 +63,7 @@
 #define PCI_VENDOR_AMD         0x1022
 #define PCI_VENDOR_TRIDENT     0x1023
 #define PCI_VENDOR_ALI         0x1025
+#define PCI_VENDOR_DELL                0x1028
 #define PCI_VENDOR_MATROX      0x102B
 #define PCI_VENDOR_CHIPSTECH   0x102C
 #define PCI_VENDOR_MIRO                0x1031
@@ -154,6 +155,7 @@
 #define PCI_VENDOR_NVIDIA_SGS  0x12d2
 #define PCI_VENDOR_NETGEAR     0x1385
 #define PCI_VENDOR_VMWARE      0x15AD
+#define PCI_VENDOR_PCPARTNER   0x174B
 #define PCI_VENDOR_SYMPHONY    0x1C1C
 #define PCI_VENDOR_TEKRAM_2    0x1DE1
 #define PCI_VENDOR_3DLABS      0x3D3D

Reply via email to