[PATCH] xf86AutoConfig: Try newport on mips.

Mark Kettenis mark.kettenis at xs4all.nl
Sun Feb 27 14:31:27 PST 2011


> From: Cyril Brulebois <kibi at debian.org>
> 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


More information about the xorg-devel mailing list