> From: Cyril Brulebois <[email protected]>
> Date: Sun, 27 Feb 2011 22:52:18 +0100
> 
> As suggested by Bernhard R. Link in Debian bug report #566545
> (http://bugs.debian.org/566545), newport is a candidate for
> autodetection on mips.

> diff --git a/hw/xfree86/common/xf86AutoConfig.c 
> b/hw/xfree86/common/xf86AutoConfig.c
> index 5c6e721..ef1aab0 100644
> --- a/hw/xfree86/common/xf86AutoConfig.c
> +++ b/hw/xfree86/common/xf86AutoConfig.c
> @@ -264,6 +264,8 @@ listPossibleVideoDrivers(char *matches[], int nmatches)
>       matches[i++] = xnfstrdup("vesa");
>  #elif defined(__sparc__) && !defined(sun)
>       matches[i++] = xnfstrdup("sunffb");
> +#elif defined(__mips__)
> +     matches[i++] = xnfstrdup("newport");
>  #endif
>      }

Not necessary a good idea.  The autoconfig code was broken some time
ago by a "all-the-world-is-PCI" believer some time ago.  The breakage
is that if you list more than a single "possible" video driver, things
break.  That's why we actually have a diff in the OpenBSD tree now
that changes the

  #elif defined(__sparc__) && !defined(sun)

just above the bit you're adding into

  #elif defined(__sparc__) && defined(__linux__)

Since we have some (OpenBSD-specific) SBus/UPA autodetection code that
can detect the hardware that needs the sunffb driver.

The problems is of course that there are a lot of different MIPS-based
platforms around, and most of them aren't an SGI Indy.

If the Linux community thinks this is still a good idea, you could do
something similar and use:

  #elif define(__mips__) && defined(__linux__)

Cheers,

Mark
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to