[Xorg-driver-geode] URGENT! Please test this patch!

[Martin-Éric Racine]

On Wed, Apr 16, 2008 at 7:49 PM, Jordan Crouse <jordan.crouse at amd.com> wrote:
>
> On 16/04/08 14:52 +0300, Martin-Éric Racine wrote:
>  > On Wed, Apr 16, 2008 at 2:03 PM, Martin-Éric Racine <q-funk at iki.fi> wrote:
>  > > On Wed, Apr 16, 2008 at 12:54 AM, Jordan Crouse <jordan.crouse at amd.com> wrote:
>  > >  > Please drop everything and test the attached patch for "native" DDC
>  > >  >  support.  If it works, you should see this in the X log:
>  > >  >
>  > >  >  (II) GEODE(0): I2C bus "CS5536 DDC BUS" initialized.
>  > >  >  (II) GEODE(0): I2C device "CS5536 DDC BUS:ddc2" registered at address 0xA0.
>  > >  >  (II) GEODE(0): I2C bus "CS5536 DDC BUS" removed.
>  > >  >
>  > >  >  Followed by the EDID data (or no data, if you happen not to have a monitor
>  > >  >  that supports DDC).
>  > >  >
>  > >  >  The patch is against xf86-video-geode-2.8.0.  If the data does not look
>  > >  >  like you expect, please e-mail me your X log.  We have already tested
>  > >  >  this against these platforms and been successful.
>  > >  >
>  > >  >  Insyde/DB800/X server 1.5
>  > >  >  Award/FIC ION/X server 1.4.0.90a (hardy)
>  > >  >  Insyde/DTRI/X server 1.4.0.90 (hardy):
>  > >
>  > >  NOT working on GeneralSoftware/ThinCan DBE62/X server 1.4.0.90
>  > >  (ubuntu/hardy), unfortunately. Log attached.
>  >
>  > As someone pointed out, it appears that X is reverting to the VESA
>  > driver, in the above log. Yet, the xorg.conf states that:
>  >
>  > Section "Device"
>  >         ### Available Driver options are:-
>  >         ### Values: <i>: integer, <f>: float, <bool>: "True"/"False",
>  >         ### <string>: "String", <freq>: "<f> Hz/kHz/MHz"
>  >         ### [arg]: arg optional
>  >       Identifier  "Card0"
>  >       Driver      "geode"
>  >       VendorName  "Advanced Micro Devices [AMD]"
>  >       BoardName   "Geode LX Video"
>  >       BusID       "PCI:0:1:1"
>  > EndSection
>  >
>  > So it appears that GEODE fails to claim the device and X tries to
>  > revert to VESA as a backup solution.
>  >
>  > As a second test, I purged the VESA driver, leaving only GEODE
>  > installed and rebooted. In that case, X freezes and it also takes the
>  > whole system down with it.
>
>  Be forwarned - the Makefile has changed, so you need to re-run
>  autogen.sh.  If you do not, then you may be missing a symbol.
>
>  If you are missing a symbol, then X will crash, and if you are using
>  VGA, then you won't be able to get back your screen.  Either run X
>  from a remote or serial terminal or use a framebuffer to avoid this
>  annoying behavior.

After running autogen.sh again and rebuilding, it finally works as
expected on my DBE62 booting off General Software BIOS. I get the
GEODE I2C messages & all.

Great work, Jordan! :)

 PS: test packages are available for Ubuntu/Hardy in my personal
package archive at

deb http://ppa.launchpad.net/q-funk/ubuntu hardy main
deb-src http://ppa.launchpad.net/q-funk/ubuntu hardy main

-- 
Martin-Éric Racine
http://q-funk.iki.fi