Created attachment 18398 [details] [review]
Patch that fixes -ati for the Powerbook3,3

Forwarding this Ubuntu bug:
https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-ati/+bug/220887

Logs / Config Files:
http://launchpadlibrarian.net/13833567/xorg.conf
http://launchpadlibrarian.net/13833579/Xorg.0.log.old-Radeon-no-connected-devices-found.txt
http://launchpadlibrarian.net/13833584/Xorg.0.log-working-wrong-resolution-with-ext-lcd.txt


On a Powerbook3,3, X crashes with error "(EE) RADEON(0): No connected devices found!"  When an external LCD is plugged into the VGA out, X comes up on the laptop panel, but at the wrong resolution, and the external monitor remains black.

"It turns out that setting the MacModel option to "powerbook-internal" and specifying horizontal and vertical refresh settings (grabbed from Ubuntu 7.1) in xorg.conf gets my powerbook3,3 up and running, although there is an ugly ~5 seconds as X is starting during which the screen is filled with green lines that refresh like an old television. I created a patch to the radeon driver that makes the configuration for powerbook3,3's built-in lcd the same as as for powerbook-internal models. The external output was also getting configured incorrectly by the radeon driver (it is a VGA output, but it was getting configured as DVI), so I changed that in the patch too. Rebooting with a VGA monitor attached sort of works. The driver picks a reasonable resolution and color depth for the VGA monitor, displays the default desktop background, but mirrors the GNOME toolbars and desktop icons from the laptop's lcd on the upper left 1152x768 pixels of the screen (that is the resolution of the built-in lcd). Don't know how to fix that problem.

The patch is against Ubuntu's latest release of xserver-xorg-video-ati for Hardy."


It looks like the change for BiosConnector[0] is already in git, however the changes for BiosConnector[1] weren't; is this patch sane?