On Wed, 19 Feb 2003, Ian Romanick wrote:

> Leif Delgass wrote:
> > Ian, this commit includes references to rmesa->hw.cube[], which isn't in 
> > radeon_context.h yet.  I don't see any reason not to commit the entire 
> > cube map patch, but leave the extension disabled.  I haven't had time to 
> > investigate the problems yet, but most of the code looks to be identical 
> > to r200.  At any rate, I haven't encountered any regressions as a result 
> > of having the patch applied.
> 
> D'oh!  The problem is that part of the (full) patch modifies kernel 
> code.  I don't want to get into a situation where we have funky 
> non-working kernel code floating around.  I should either back out the 
> broken stuff that I put in or commit the rest of the user-side cube_map 
> support.  Ugh.
> 
> > With the extension enabled, the cubemap demo seems to kind of work, but it 
> > seems to only use the yellow cube face.  Is that what you were seeing?
> 
> Yeah, that's exactly what I see.  Any thoughts?

I don't have docs, so I can't verify that we're not missing any relevant 
bits.  The r200 has pp_texformat_x, which has a bit for the texcoord use: 
non-projective, cubic, volume, etc.  I don't see anything like that for 
R100.  I tried modifying the texgen matrices to swap the third and fourth 
columns, but it didn't seem to have an effect.  The upload code looks like 
it's the same as r200, but I haven't tried to verify that it's working 
right.  Do you know which face the yellow one is?

-- 
Leif Delgass 
http://www.retinalburn.net



-------------------------------------------------------
This SF.net email is sponsored by: SlickEdit Inc. Develop an edge.
The most comprehensive and flexible code editor you can use.
Code faster. C/C++, C#, Java, HTML, XML, many more. FREE 30-Day Trial.
www.slickedit.com/sourceforge
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to