Bug 438299 - DDC fails on PowerBook
Summary: DDC fails on PowerBook
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: xorg-x11-drv-ati
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Dave Airlie
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F9KernelBlocker
TreeView+ depends on / blocked
 
Reported: 2008-03-20 09:39 UTC by David Woodhouse
Modified: 2008-04-18 06:02 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-04-18 06:02:05 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
Xorg.log from Fedora 8 (47.89 KB, text/plain)
2008-03-20 09:42 UTC, David Woodhouse 		no flags Details
Xorg.log from Fedora 9 (34.96 KB, text/plain)
2008-03-20 09:43 UTC, David Woodhouse 		no flags Details
Electra Xorg.log (37.70 KB, text/plain)
2008-03-20 12:04 UTC, David Woodhouse 		no flags Details
View All

Description David Woodhouse 2008-03-20 09:39:28 UTC 
Fedora 8 correctly reported the panel on my shinybook:
(II) RADEON(0): I2C device "DVI_DDC:ddc2" registered at address 0xA0.
(II) RADEON(0): DDC Type: 2, Detected Monitor Type: 2
(II) RADEON(0): EDID data from the display on connector: Proprietary/LVDS
----------------------
(II) RADEON(0): Manufacturer: APP  Model: 9c2c  Serial#: 0
(II) RADEON(0): Year: 2003  Week: 1
(II) RADEON(0): EDID Version: 1.3
(II) RADEON(0): Digital Display Input
(II) RADEON(0): Max H-Image Size [cm]: horiz.: 37  vert.: 23
....

Fedora 9 (xorg-x11-drv-ati-6.8.0-3.fc9) doesn't:
(II) RADEON(0): Output: LVDS, Detected Monitor Type: 0
(II) RADEON(0): Detected non-DDC Monitor Type: 2
finished output detect: 0
Comment 1 David Woodhouse 2008-03-20 09:42:36 UTC 
Created attachment 298672 [details]
Xorg.log from Fedora 8
Comment 2 David Woodhouse 2008-03-20 09:43:06 UTC 
Created attachment 298673 [details]
Xorg.log from Fedora 9
Comment 3 David Woodhouse 2008-03-20 12:04:11 UTC 
Created attachment 298688 [details]
Electra Xorg.log

I see DDC failure on an NV card too...
Comment 4 David Woodhouse 2008-03-21 01:10:32 UTC 
The DDC failure on ATI goes away, along with the crash on my G5, when I change
RADEONGetBIOSInfo() to avoid allocating a buffer of 64KiB and then immediately
asking pci_device_read_rom() to read the 128KiB BIOS ROM into it.

Why is it that we don't allocate a buffer of size info->PciInfo->rom_size,
anyway? That seems to be commented out, and we allocate RADEON_VBIOS_SIZE
(64KiB) instead.
Comment 5 David Woodhouse 2008-03-21 01:23:46 UTC 
--- a/src/radeon_bios.c
+++ b/src/radeon_bios.c
@@ -74,8 +74,8 @@ Bool RADEONGetBIOSInfo(ScrnInfoPtr pScrn, xf86Int10InfoPtr 
pInt10)
     unsigned short dptr;
 
 #ifdef XSERVER_LIBPCIACCESS
-    //info->VBIOS = xalloc(info->PciInfo->rom_size);
-    info->VBIOS = xalloc(RADEON_VBIOS_SIZE);
+    info->VBIOS = xalloc(info->PciInfo->rom_size);
+    //    info->VBIOS = xalloc(RADEON_VBIOS_SIZE);
 #else
     info->VBIOS = xalloc(RADEON_VBIOS_SIZE);
 #endif
Comment 6 Jon Stanley 2008-04-17 01:03:17 UTC 
Has this made it into rawhide yet? (just reviewing the F9 blocker)
Comment 7 Dave Airlie 2008-04-18 06:02:05 UTC 
should be fixed in rawhide this patch is in at least.
Note You need to log in before you can comment on or make changes to this bug.