[Xorg-driver-geode] looking at wide mode support for GeodeLX (success)

[Martin-Éric Racine]

On Sat, Mar 1, 2008 at 7:01 PM, Bart Trojanowski <bart at jukie.net> wrote:
> Hi again,
>
>  * Bart Trojanowski <bart at jukie.net> [080229 13:35]:
>  > Earlier, Jordan presented a patch:
>  >
>  > http://lists.x.org/archives/xorg-driver-geode/2008-February/000211.html
>  >
>  > I tested it with on Gutsy (xorg-core 1.3.0) and found that it was
>  > skipping most of the modes in LXValidMode() because it was not in the
>  > Cimarron table.
>
>  I am attaching the original patch plus a small logic correction..
>
>  -    if ((pMode->type & M_T_BUILTIN) || (pMode->type && M_T_DEFAULT)) {
>  +    if ((pMode->type & M_T_BUILTIN) || (pMode->type & M_T_DEFAULT)) {
>
>  ... I read through the patch a few times and didn't spot it until I
>  started looking at the values of mode->type.
>
>  I don't have a way to test this with a PANEL connected.  But I can
>  confirm that wide modes work.  I am happy to sign off on the patch.

Committed to X.org git.

OLPC and projects with a TFT:  please test ASAP and report any issue
that might have crept in.

If nobody reports anything major by the end of this week, this will be
pushed (along with the final version of Bernie's libpciaccess port) as
xf86-video-amd 2.7.7.7 and released with X.org 7.4 (server core 1.5).

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