Re: pkg/35416: agp driver hangs on ALi/ULi M1689 chipset

[Gary Duzan <gary%duzan.org@localhost>]

In Message <20080405131758.GA476761%medusa.sis.pasteur.fr@localhost> ,
   Nicolas Joly <njoly%pasteur.fr@localhost> wrote:

=>On Sat, Apr 05, 2008 at 11:45:01AM +0000, Martin Husemann wrote:
=>>  On Sat, Apr 05, 2008 at 07:23:40AM -0400, Gary Duzan wrote:
=>>  >    I believe so. However, the problem isn't the size, it is that
=>>  > the wrong driver attaches, and I assume the aperture size is being
=>>  > pulled from the wrong place.
=>>  
=>>  The check for zero size should be added there, however the right fix is
=>>  to add an agp_match function in agp.c:
=>[...]
=>
=>Can you try this patch ?

   It certainly looks better:

agp0 at pchb1: 1 Miscellaneous Control unit(s) found.
agp0: aperture at 0xdc000000, size 0x4000000

Unfortunately, I can't test the functionality since I also have
a jmide issue which is keeping me from completing a boot. I'll
work on that a bit more and post my findings on current-users
and/or send a PR.

   Thanks.

                                        Gary Duzan