r300: Removed the R300_RS_INTERP_[0-9]_UNKNOWN (magic) defines.
authorOliver McFadden <z3ro.geek@gmail.com>
Mon, 21 May 2007 15:51:38 +0000 (15:51 +0000)
committerOliver McFadden <z3ro.geek@gmail.com>
Wed, 23 May 2007 17:44:14 +0000 (17:44 +0000)
Supposedly you need to set these values for the interpolaters to work, but they
seem to work fine without these values.

src/mesa/drivers/dri/r300/r300_reg.h
src/mesa/drivers/dri/r300/r300_state.c

index 0a31f0b..e64f509 100644 (file)
@@ -628,17 +628,11 @@ USE OR OTHER DEALINGS IN THE SOFTWARE.
  * Set INTERP_USED on all interpolators that produce data used by
  * the fragment program. INTERP_USED looks like a swizzling mask,
  * but I haven't seen it used that way.
- *
- * Note: The _UNKNOWN constants are always set in their respective
- * register. I don't know if this is necessary.
  */
 #define R300_RS_INTERP_0                    0x4310
 #define R300_RS_INTERP_1                    0x4314
-#       define R300_RS_INTERP_1_UNKNOWN          0x40
 #define R300_RS_INTERP_2                    0x4318
-#       define R300_RS_INTERP_2_UNKNOWN          0x80
 #define R300_RS_INTERP_3                    0x431C
-#       define R300_RS_INTERP_3_UNKNOWN          0xC0
 #define R300_RS_INTERP_4                    0x4320
 #define R300_RS_INTERP_5                    0x4324
 #define R300_RS_INTERP_6                    0x4328
index e8d67f9..8a85478 100644 (file)
@@ -1442,17 +1442,6 @@ union r300_outputs_written {
 static void r300SetupRSUnit(GLcontext * ctx)
 {
        r300ContextPtr r300 = R300_CONTEXT(ctx);
-       /* I'm still unsure if these are needed */
-       GLuint interp_magic[8] = {
-               0x00,
-               R300_RS_INTERP_1_UNKNOWN,
-               R300_RS_INTERP_2_UNKNOWN,
-               R300_RS_INTERP_3_UNKNOWN,
-               0x00,
-               0x00,
-               0x00,
-               0x00
-       };
        union r300_outputs_written OutputsWritten;
        GLuint InputsRead;
        int fp_reg, high_rr;
@@ -1498,8 +1487,7 @@ static void r300SetupRSUnit(GLcontext * ctx)
        for (i = 0; i < ctx->Const.MaxTextureUnits; i++) {
                r300->hw.ri.cmd[R300_RI_INTERP_0 + i] = 0
                    | R300_RS_INTERP_USED
-                   | (in_texcoords << R300_RS_INTERP_SRC_SHIFT)
-                   | interp_magic[i];
+                   | (in_texcoords << R300_RS_INTERP_SRC_SHIFT);
 
                r300->hw.rr.cmd[R300_RR_ROUTE_0 + fp_reg] = 0;
                if (InputsRead & (FRAG_BIT_TEX0 << i)) {