On Mon, 14 Feb 2005 16:13:18 +0100, Jerome Glisse <[EMAIL PROTECTED]> wrote: > On Mon, 14 Feb 2005 10:02:07 -0500 (EST), Vladimir Dergachev > <[EMAIL PROTECTED]> wrote: > > > > > > On Mon, 14 Feb 2005, Jerome Glisse wrote: > > > > >> On Mon, 14 Feb 2005 17:17:05 +1100, Paul Mackerras <[EMAIL PROTECTED]> > > >> wrote: > > >>> Vladimir Dergachev writes: > > >>> > > >>>> I am a bit puzzled with your patch - do you really need to hard code > > >>>> format field in r300_state.c ? > > > > > > Ok i think i understand, Mesa tell us that the format is > > > MESA_FORMAT_RGBA8888_REV on big endiand and > > > MESA_FORMAT_RGBA8888 on little endian. Thus if i change > > > {0xffffff01, 0}, > > > to > > > {R300_EASY_TX_FORMAT(Z, Y, X, W, W8Z8Y8X8), 0}, > > > > > > it works. Vlad does this lead to some strange things on x86 ? > > > > The reason these entries are as they are is because I (and probably no one > > else yet) was not able to trigger them with any test program. > > > > Of course, it must be possible to debug them by writing a custom program, > > but I did not have time for that yet.. > > Yes but my understanding is that mesa will never give you > MESA_FORMAT_RGBA8888_REV on x86 because this tell you > that data are in reverse order (see mesa texformat). So i think > that all odd entry in this table are for REV order (which we will > found on big endian) and all even entry are for "normal" order > x86. I think that you can ask mesa to give you texture in REV > order on x86 but don't know how. > > I am really want to commit this simple patch, maybe someone > with a better understand of mesa & who know radeon driver > could tell us if i am wrong. More over if you look at all table > (r200 table or table0 in r300) you will see that allmost all > odd entry have the _REV suffix. > > So every even entry for you x86 and odd for us ppc ;) > > By the way with which program do you test : > {0xffffff11, 0}, > ->{R300_EASY_TX_FORMAT(Y, Y, Y, X, Y8X8), 0}, > {0xffffff13, 0}, > > the 0xffffff13 would be > {0xffffff11, 0}, > {R300_EASY_TX_FORMAT(Y, Y, Y, X, Y8X8), 0}, > {R300_EASY_TX_FORMAT(Y, Y, X, Y, Y8X8), 0}, > > But testing if the result is good is better :)
I commited a patch for big endian, i will do a test prog so we can test texture format on x86 & ppc. Jerome Glisse ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel