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_* .


-- 
Earthling Michel D�nzer (MrCooper)/ Debian GNU/Linux (powerpc) developer
XFree86 and DRI project member   /  CS student, Free Software enthusiast


-------------------------------------------------------
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