On Fri, 2005-01-14 at 12:23 -0500, Michel Dänzer wrote: > On Fri, 2005-01-14 at 16:34 +0100, Jerome Glisse wrote: > > > > Anyway i wanted to ask mesa folks how to make a real > > proper patch. The fact is that INREG & OUTREG in > > server/radeon_macros.h have to do endian swapping. > > Moreover the swapping is only needed for r300, isn't it ? > > So do we need to have our own copy of radeon_macros.h > > insted of using the one from dri/radeon/ ? > > No. Try this patch, which is quite ugly though, most of this shouldn't > be in driver specific code IMHO.
Gack ! You mean that the DRI doesn't use eieio for accessing MMIO registers ? I have missed that. This is really bad. All 3 sizes, both reads and writes, need to have eieio. Ben. ------------------------------------------------------- The SF.Net email is sponsored by: Beat the post-holiday blues Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek. It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel