- Fix nasty depth span bug. Drawable offset was not being added to pixel

    coords.
- Remove unneeded mask parameters from clear ioctl.
- Use correct subpixel offsets, fixes most glean bugs.
- Remove 32-bit depth buffer support. Only use 16 or 24-bit depth buffers.

diff --git a/linux-core/r128_drv.c b/linux-core/r128_drv.c
index 89b6869..b70e0f2 100644 (file)
--- a/linux-core/r128_drv.c
+++ b/linux-core/r128_drv.c
@@ -37,10 +37,10 @@
 
 #define R128_NAME              "r128"
 #define R128_DESC              "ATI Rage 128"
-#define R128_DATE              "20001201"
+#define R128_DATE              "20001212"
 #define R128_MAJOR             2
 #define R128_MINOR             1
-#define R128_PATCHLEVEL                0
+#define R128_PATCHLEVEL                2
 
 static drm_device_t    r128_device;
 drm_ctx_t              r128_res_ctx;

diff --git a/linux/r128_cce.c b/linux/r128_cce.c
index 0b1576a..d978d53 100644 (file)
--- a/linux/r128_cce.c
+++ b/linux/r128_cce.c
@@ -114,7 +114,7 @@ int R128_READ_PLL(drm_device_t *dev, int addr)
        return R128_READ(R128_CLOCK_CNTL_DATA);
 }
 
-
+#if 0
 static void r128_status( drm_r128_private_t *dev_priv )
 {
        printk( "GUI_STAT           = 0x%08x\n",
@@ -130,6 +130,7 @@ static void r128_status( drm_r128_private_t *dev_priv )
        printk( "PM4_BUFFER_CNTL    = 0x%08x\n",
                (unsigned int)R128_READ( R128_PM4_BUFFER_CNTL ) );
 }
+#endif
 
 
 /* ================================================================
@@ -701,6 +702,7 @@ int r128_engine_reset( struct inode *inode, struct file *filp,
 #define R128_BUFFER_USED       0xffffffff
 #define R128_BUFFER_FREE       0
 
+#if 0
 static int r128_freelist_init( drm_device_t *dev )
 {
        drm_device_dma_t *dma = dev->dma;
@@ -746,6 +748,7 @@ static int r128_freelist_init( drm_device_t *dev )
        return 0;
 
 }
+#endif
 
 drm_buf_t *r128_freelist_get( drm_device_t *dev )
 {
@@ -834,6 +837,7 @@ void r128_update_ring_snapshot( drm_r128_private_t *dev_priv )
                ring->space += ring->size;
 }
 
+#if 0
 static int r128_verify_command( drm_r128_private_t *dev_priv,
                                u32 cmd, int *size )
 {
@@ -974,6 +978,7 @@ static int r128_submit_packet_ring_insecure( drm_r128_private_t *dev_priv,
 #endif
        return 0;
 }
+#endif
 
 /* Internal packet submission routine.  This uses the insecure versions
  * of the packet submission functions, and thus should only be used for

diff --git a/linux/r128_drm.h b/linux/r128_drm.h
index b81d2fc..68a55d5 100644 (file)
--- a/linux/r128_drm.h
+++ b/linux/r128_drm.h
@@ -219,8 +219,6 @@ typedef struct drm_r128_clear {
        int x, y, w, h;
        unsigned int clear_color;
        unsigned int clear_depth;
-       unsigned int color_mask;
-       unsigned int depth_mask;
 } drm_r128_clear_t;
 
 typedef struct drm_r128_vertex {

diff --git a/linux/r128_drv.c b/linux/r128_drv.c
index 89b6869..b70e0f2 100644 (file)
--- a/linux/r128_drv.c
+++ b/linux/r128_drv.c
@@ -37,10 +37,10 @@
 
 #define R128_NAME              "r128"
 #define R128_DESC              "ATI Rage 128"
-#define R128_DATE              "20001201"
+#define R128_DATE              "20001212"
 #define R128_MAJOR             2
 #define R128_MINOR             1
-#define R128_PATCHLEVEL                0
+#define R128_PATCHLEVEL                2
 
 static drm_device_t    r128_device;
 drm_ctx_t              r128_res_ctx;

diff --git a/linux/r128_state.c b/linux/r128_state.c
index faad25a..3f52c14 100644 (file)
--- a/linux/r128_state.c
+++ b/linux/r128_state.c
@@ -297,10 +297,11 @@ static void r128_clear_box( drm_r128_private_t *dev_priv,
                color = ((r << 16) | (g << 8) | b);
                break;
        case 32:
-       default:
                fb_bpp = R128_GMC_DST_32BPP;
                color = (((0xff) << 24) | (r << 16) | (g <<  8) | b);
                break;
+       default:
+               return;
        }
 
        offset = dev_priv->back_offset;
@@ -362,9 +363,7 @@ static void r128_cce_dispatch_clear( drm_device_t *dev,
                                     unsigned int flags,
                                     int cx, int cy, int cw, int ch,
                                     unsigned int clear_color,
-                                    unsigned int clear_depth,
-                                    unsigned int color_mask,
-                                    unsigned int depth_mask )
+                                    unsigned int clear_depth )
 {
        drm_r128_private_t *dev_priv = dev->dev_private;
        drm_r128_sarea_t *sarea_priv = dev_priv->sarea_priv;
@@ -381,21 +380,17 @@ static void r128_cce_dispatch_clear( drm_device_t *dev,
        case 16:
                fb_bpp = R128_GMC_DST_16BPP;
                break;
-       case 24:
-               fb_bpp = R128_GMC_DST_24BPP;
-               break;
        case 32:
-       default:
                fb_bpp = R128_GMC_DST_32BPP;
                break;
+       default:
+               return;
        }
        switch ( dev_priv->depth_bpp ) {
        case 16:
                depth_bpp = R128_GMC_DST_16BPP;
                break;
        case 24:
-               depth_bpp = R128_GMC_DST_32BPP;
-               break;
        case 32:
                depth_bpp = R128_GMC_DST_32BPP;
                break;
@@ -417,7 +412,7 @@ static void r128_cce_dispatch_clear( drm_device_t *dev,
                        BEGIN_RING( 2 );
 
                        OUT_RING( CCE_PACKET0( R128_DP_WRITE_MASK, 0 ) );
-                       OUT_RING( color_mask );
+                       OUT_RING( sarea_priv->context_state.plane_3d_mask_c );
 
                        ADVANCE_RING();
                }
@@ -465,10 +460,7 @@ static void r128_cce_dispatch_clear( drm_device_t *dev,
                }
 
                if ( flags & R128_DEPTH ) {
-                       BEGIN_RING( 8 );
-
-                       OUT_RING( CCE_PACKET0( R128_DP_WRITE_MASK, 0 ) );
-                       OUT_RING( depth_mask );
+                       BEGIN_RING( 6 );
 
                        OUT_RING( CCE_PACKET3( R128_CNTL_PAINT_MULTI, 4 ) );
                        OUT_RING( R128_GMC_DST_PITCH_OFFSET_CNTL
@@ -477,7 +469,8 @@ static void r128_cce_dispatch_clear( drm_device_t *dev,
                                  | R128_GMC_SRC_DATATYPE_COLOR
                                  | R128_ROP3_P
                                  | R128_GMC_CLR_CMP_CNTL_DIS
-                                 | R128_GMC_AUX_CLIP_DIS );
+                                 | R128_GMC_AUX_CLIP_DIS
+                                 | R128_GMC_WR_MSK_DIS );
 
                        OUT_RING( dev_priv->depth_pitch_offset_c );
                        OUT_RING( clear_depth );
@@ -513,9 +506,6 @@ static void r128_cce_dispatch_swap( drm_device_t *dev )
        case 16:
                fb_bpp = R128_GMC_DST_16BPP;
                break;
-       case 24:
-               fb_bpp = R128_GMC_DST_24BPP;
-               break;
        case 32:
        default:
                fb_bpp = R128_GMC_DST_32BPP;
@@ -542,21 +532,12 @@ static void r128_cce_dispatch_swap( drm_device_t *dev )
                          | R128_GMC_AUX_CLIP_DIS
                          | R128_GMC_WR_MSK_DIS );
 
-#if 1
                OUT_RING( dev_priv->back_pitch_offset_c );
                OUT_RING( dev_priv->front_pitch_offset_c );
 
                OUT_RING( (x << 16) | y );
                OUT_RING( (x << 16) | y );
                OUT_RING( (w << 16) | h );
-#else
-               OUT_RING( dev_priv->depth_pitch_offset_c /*& ~R128_DST_TILE*/ );
-               OUT_RING( dev_priv->front_pitch_offset_c );
-
-               OUT_RING( (0 << 16) | 0 );
-               OUT_RING( (0 << 16) | 0 );
-               OUT_RING( (800 << 16) | 600 );
-#endif
 
                ADVANCE_RING();
        }
@@ -925,6 +906,9 @@ static int r128_cce_dispatch_blit( drm_device_t *dev,
 
 /* ================================================================
  * Tiled depth buffer management
+ *
+ * FIXME: These should all set the destination write mask for when we
+ * have hardware stencil support.
  */
 
 static int r128_cce_dispatch_write_span( drm_device_t *dev,
@@ -946,8 +930,6 @@ static int r128_cce_dispatch_write_span( drm_device_t *dev,
                depth_bpp = R128_GMC_DST_16BPP;
                break;
        case 24:
-               depth_bpp = R128_GMC_DST_32BPP;
-               break;
        case 32:
                depth_bpp = R128_GMC_DST_32BPP;
                break;
@@ -1057,8 +1039,6 @@ static int r128_cce_dispatch_write_pixels( drm_device_t *dev,
                depth_bpp = R128_GMC_DST_16BPP;
                break;
        case 24:
-               depth_bpp = R128_GMC_DST_32BPP;
-               break;
        case 32:
                depth_bpp = R128_GMC_DST_32BPP;
                break;
@@ -1190,8 +1170,6 @@ static int r128_cce_dispatch_read_span( drm_device_t *dev,
                depth_bpp = R128_GMC_DST_16BPP;
                break;
        case 24:
-               depth_bpp = R128_GMC_DST_32BPP;
-               break;
        case 32:
                depth_bpp = R128_GMC_DST_32BPP;
                break;
@@ -1249,8 +1227,6 @@ static int r128_cce_dispatch_read_pixels( drm_device_t *dev,
                depth_bpp = R128_GMC_DST_16BPP;
                break;
        case 24:
-               depth_bpp = R128_GMC_DST_32BPP;
-               break;
        case 32:
                depth_bpp = R128_GMC_DST_32BPP;
                break;
@@ -1367,8 +1343,7 @@ int r128_cce_clear( struct inode *inode, struct file *filp,
 
        r128_cce_dispatch_clear( dev, clear.flags,
                                 clear.x, clear.y, clear.w, clear.h,
-                                clear.clear_color, clear.clear_depth,
-                                clear.color_mask, clear.depth_mask );
+                                clear.clear_color, clear.clear_depth );
 
        /* Make sure we restore the 3D state next time.
         */