[PATCH] try the newport driver on mips

Mark Kettenis mark.kettenis at xs4all.nl
Sun Nov 7 23:30:37 PST 2010


> From: Matt Turner <mattst88 at gmail.com>
> Date: Sun, 7 Nov 2010 23:57:34 -0500
> 
> On Sat, Mar 6, 2010 at 1:03 PM, Matt Turner <mattst88 at gmail.com> wrote:
> > On Sat, Mar 6, 2010 at 3:23 AM, Brice Goglin <brice.goglin at gmail.com> wrote:
> >> *From:* "Bernhard R. Link" <brlink at debian.org>
> >> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=566545
> >>
> >> Attached patch should tell the autodection code that mips machines can
> >> have newport cards so they get tested and used.
> >>
> >> (Untested, but it looks trivally enough to fix the first of the three
> >> problems in http://bugs.debian.org/566490).
> >>
> >> diff --git a/hw/xfree86/common/xf86AutoConfig.c b/hw/xfree86/common/xf86AutoConfig.c
> >> index a6199b0..ff6107e 100644
> >> --- a/hw/xfree86/common/xf86AutoConfig.c
> >> +++ b/hw/xfree86/common/xf86AutoConfig.c
> >> @@ -507,6 +507,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
> >>     }
> >
> > Seems like a good idea.
> >
> > Reviewed-by: Matt Turner <mattst88 at gmail.com>
> >
> 
> Wow, this was forgotten.
> 
> Can you pull this, Keith?

Actually, I don't think this is a good idea.  Unless I'm mistaken, the
newport driver only supports one particular range of SGI graphics
hardware.  There are quite a few other SGI graphics boards out there
and other non-SGI mips hardware platforms.

Isn't it possible to do something a little bit smarter here?

(I'm actually thinking of removing the "sunffb" case for __sparc__,
since it causes endless amounts of pain for me because probing of
non-PCI hardware tends to be more broken with every release of the X
server).


More information about the xorg-devel mailing list