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