On Mon, 2002-07-08 at 20:17, Tim Smith wrote: > On Monday 08 Jul 2002 12:49 am, Michel Dänzer scribed numinously:" > > > The scratch register values need to be read with DRM_READ32(), which > > accounts both for endianness and memory barriers. So it would be > > > > u32 done_age = DRM_READ32(&dev_priv->scratch[1]); > > That's good to know; I'll file that a little closer to my forebrain. I'd > noticed the macros before but not taken enough notice. I thought the card > took care of that when it wrote the value back (I believe it can) but maybe > not.
It can, but that would mean extra code to set the control registers according to endianness and wouldn't really buy us anything as reading little endian data is free with a decent big endian CPU and the memory barriers would still have to be dealt with. -- Earthling Michel Dänzer (MrCooper)/ Debian GNU/Linux (powerpc) developer XFree86 and DRI project member / CS student, Free Software enthusiast ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Oh, it's good to be a geek. http://thinkgeek.com/sf _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel