In the projtex demo, while the scaling of the projection is correct, the corners are in the wrong places. Need to fix.
I discovered that projected textures work well on radeon / r200 only if you send the w-coord to the card. -> tiny vertex format isn't allowed for projtex! The old template based swtcl code (which is still used by radeon driver) worked that way: tiny vertex format isn't used when texturing. The new "t_vertex" code emits the w-coord only if its really necessary: it prefers a "tiny" vertex format. You could try to just always disable that tiny vertex format in r200ChooseVertexState().
Created attachment 1609 [details] [review] fixes this projtex problem for me; needs cleanup and maybe optimization please test. feel free to cleanup/optimize/commit Question: Do we need w component for specular color? (the old code used the NOTEX_VERTEX format for that) If not, cut that piece out of the patch. unfortunately the glut menues are broken on r200 atm., but thats another problem...
Created attachment 1625 [details] [review] r200_projtex_fix2.diff.txt + cleanup + possible fixes for points and transition swtnl<->hwtcl (when testing make sure to test the sw-tcl case...)
Created attachment 1759 [details] [review] slightly different projtex fix new version of the fix, which eliminates the additional statechange. I'm not really sure this is sufficient & correct, but it works for projtex. Some of the vte / vap combinations look a little odd, though maybe it wouldn't if I'd know what those registers actually do...
your patch seems to work well at least for projtex. First I thought if (tnl->render_inputs & _TNL_BITS_TEX_ANY) might not be up to date, but at least in projtex it seems so. Someone here to try this patch with other programs that make use of projtex ?
(In reply to comment #5) > First I thought > if (tnl->render_inputs & _TNL_BITS_TEX_ANY) > might not be up to date, but at least in projtex it seems so. Thought that too first, but then I guessed if _TNL_BITS_TEX_ANY wouldn't be up to date, it would not work correctly at all, since the code would incorrectly select the tiny vertex format in cases where it shouldn't. Applied to cvs.
Mass version move, cvs -> git
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.