Michel Dänzer wrote:
> On Mon, 2002-07-01 at 21:28, Tim Smith wrote:
> 
> 
>>-     BEGIN_RING( 4 );
>>-     OUT_RING( CP_PACKET0( RADEON_RE_TOP_LEFT, 0 ) );
>>-     OUT_RING( (box->y1 << 16) | box->x1 );
>>-     OUT_RING( CP_PACKET0( RADEON_RE_WIDTH_HEIGHT, 0 ) );
>>-/*   OUT_RING( ((box->y2 - 1) << 16) | (box->x2 - 1) );*/
>>-     OUT_RING( (box->y2 << 16) | box->x2 );
>>-     ADVANCE_RING();
>>+        BEGIN_RING( 3 );
>>+        OUT_RING( CP_PACKET3( RADEON_CCE_SET_SCISSORS, 1 ));
>>+        OUT_RING( (box->y1 << 16) | box->x1 );
>>+        OUT_RING( (box->y2 << 16) | box->x2 );
>>+        ADVANCE_RING();
>>
> 
> Add Jacek's fix to use the commented out variant with -1, and it's
> perfect! :) Or does the command account for that? Either way, great job.
> 
> Also, 'CCE' makes me think of a Rage128, the Radeon engine is usually
> called 'CP', and the existing commands in the file are simply called
> RADEON_CNTL_* .

Michel, if you want to make those changes & commit this, that's fine with me.

Tim, how interested are you in cvs access?

Keith



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to