Subject: re: CVS commit: xsrc/xfree/xc/programs/Xserver/hw/xfree86/drivers/ati
To: Emmanuel Dreyfus <manu@netbsd.org>
From: matthew green <mrg@eterna.com.au>
List: source-changes
Date: 06/28/2004 13:02:20
   On Mon, Jun 28, 2004 at 02:19:47AM +1000, matthew green wrote:
   > this is gross and wrong.  the radeon driver has worked on apple laptops
   > for quite a while.  
   
   There are already many nasty hacks in this driver, anyway...

sure, but this one is has potential to break other systems.

   Does the change break on other Apple laptops?

do you not know that already?  ie, i'd hope you had wider testing of
both new & old (ie, already working) hardware before commiting tis.
   
   > please back this out and bring it up on tech-x11 or fix it properly...
   
   There is already a thread about it on tech-x11. I'd be happy if you
   have a better fix for that problem.

yeah i'd like it to be an XF86Config option that you have to set, or
at least derived someway and only enabled where it is known needed,
or something.

i don't particularly understand the problem itself - i'm objecting
to it being applied with a hammer to all powerpc platforms when it's
(a) a complete hack (please clean up the magic number at least) and
(b) it's already working in older systems...


thanks.


.mrg.