summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--src/mesa/pipe/draw/draw_context.h10
-rw-r--r--src/mesa/pipe/draw/draw_flatshade.c4
-rw-r--r--src/mesa/pipe/draw/draw_vertex.c10
-rw-r--r--src/mesa/pipe/draw/draw_vertex.h51
-rw-r--r--src/mesa/pipe/i915simple/i915_state_derived.c4
-rw-r--r--src/mesa/pipe/softpipe/sp_prim_setup.c6
-rw-r--r--src/mesa/pipe/softpipe/sp_state_derived.c4
7 files changed, 51 insertions, 38 deletions
diff --git a/src/mesa/pipe/draw/draw_context.h b/src/mesa/pipe/draw/draw_context.h
index 03679848b55..7c4f1a12974 100644
--- a/src/mesa/pipe/draw/draw_context.h
+++ b/src/mesa/pipe/draw/draw_context.h
@@ -89,16 +89,6 @@ void draw_set_setup_state( struct draw_context *draw,
void draw_set_setup_stage( struct draw_context *draw,
struct draw_stage *stage );
-void draw_set_vertex_attributes( struct draw_context *draw,
- const uint *attrs, const uint *interp_mode,
- unsigned nr_attrs );
-
-void draw_set_twoside_attributes(struct draw_context *draw,
- uint front0, uint back0,
- uint front1, uint back1);
-
-void draw_compute_vertex_size(struct vertex_info *vinfo);
-
unsigned draw_prim_info( unsigned prim, unsigned *first, unsigned *incr );
unsigned draw_trim( unsigned count, unsigned first, unsigned incr );
diff --git a/src/mesa/pipe/draw/draw_flatshade.c b/src/mesa/pipe/draw/draw_flatshade.c
index ae7bd245544..2ba0d5820e9 100644
--- a/src/mesa/pipe/draw/draw_flatshade.c
+++ b/src/mesa/pipe/draw/draw_flatshade.c
@@ -56,12 +56,12 @@ static INLINE void copy_colors( struct draw_stage *stage,
const struct vertex_header *src )
{
const uint num_attribs = stage->draw->vertex_info.num_attribs;
- const uint *interp_mode = stage->draw->vertex_info.interp_mode;
+ const interp_mode *interp = stage->draw->vertex_info.interp_mode;
uint i;
/* Look for constant/flat attribs and duplicate from src to dst vertex */
for (i = 1; i < num_attribs - 2; i++) {
- if (interp_mode[i + 2] == INTERP_CONSTANT) {
+ if (interp[i + 2] == INTERP_CONSTANT) {
copy_attr( i, dst, src );
}
}
diff --git a/src/mesa/pipe/draw/draw_vertex.c b/src/mesa/pipe/draw/draw_vertex.c
index 64bc3232077..1c7e1d8662b 100644
--- a/src/mesa/pipe/draw/draw_vertex.c
+++ b/src/mesa/pipe/draw/draw_vertex.c
@@ -45,8 +45,8 @@
static INLINE void
-emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr, uint format,
- uint interp)
+emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr,
+ attrib_format format, interp_mode interp)
{
const uint n = vinfo->num_attribs;
vinfo->attr_mask |= (1 << vfAttr);
@@ -59,7 +59,6 @@ emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr, uint format,
vinfo->interp_mode[n] = interp;
vinfo->format[n] = format;
vinfo->num_attribs++;
-
}
@@ -89,6 +88,7 @@ draw_compute_vertex_size(struct vertex_info *vinfo)
vinfo->size += 3;
break;
case FORMAT_4F:
+ case FORMAT_4F_VIEWPORT:
vinfo->size += 4;
break;
default:
@@ -104,7 +104,7 @@ draw_compute_vertex_size(struct vertex_info *vinfo)
void
draw_set_vertex_attributes( struct draw_context *draw,
const uint *slot_to_vf_attr,
- const uint *interp_mode,
+ const interp_mode *interps,
unsigned nr_attrs )
{
struct vertex_info *vinfo = &draw->vertex_info;
@@ -125,7 +125,7 @@ draw_set_vertex_attributes( struct draw_context *draw,
* Remaining attribs (color, texcoords, etc)
*/
for (i = 1; i < nr_attrs; i++) {
- emit_vertex_attr(vinfo, slot_to_vf_attr[i], FORMAT_4F, interp_mode[i]);
+ emit_vertex_attr(vinfo, slot_to_vf_attr[i], FORMAT_4F, interps[i]);
}
draw_compute_vertex_size(vinfo);
diff --git a/src/mesa/pipe/draw/draw_vertex.h b/src/mesa/pipe/draw/draw_vertex.h
index f696475510b..391058af265 100644
--- a/src/mesa/pipe/draw/draw_vertex.h
+++ b/src/mesa/pipe/draw/draw_vertex.h
@@ -38,21 +38,34 @@
#define MAX_VERT_ATTRIBS 12 /* OK? */
-#define FORMAT_OMIT 0
-#define FORMAT_1F 1
-#define FORMAT_2F 2
-#define FORMAT_3F 3
-#define FORMAT_4F 4
-#define FORMAT_4F_VIEWPORT 4
-#define FORMAT_4UB 5
+
+struct draw_context;
+
+
+
+/**
+ * Vertex attribute format
+ */
+typedef enum {
+ FORMAT_OMIT,
+ FORMAT_1F,
+ FORMAT_2F,
+ FORMAT_3F,
+ FORMAT_4F,
+ FORMAT_4F_VIEWPORT,
+ FORMAT_4UB
+} attrib_format;
-enum interp_mode {
+/**
+ * Attribute interpolation mode
+ */
+typedef enum {
INTERP_NONE, /**< never interpolate vertex header info */
INTERP_CONSTANT,
INTERP_LINEAR,
INTERP_PERSPECTIVE
-};
+} interp_mode;
@@ -63,8 +76,8 @@ struct vertex_info
uint attr_mask; /**< mask of VF_ATTR_ bits */
uint slot_to_attrib[MAX_VERT_ATTRIBS];
uint attrib_to_slot[TGSI_ATTRIB_MAX];
- uint interp_mode[MAX_VERT_ATTRIBS];
- uint format[MAX_VERT_ATTRIBS]; /**< FORMAT_x */
+ interp_mode interp_mode[MAX_VERT_ATTRIBS];
+ attrib_format format[MAX_VERT_ATTRIBS]; /**< FORMAT_x */
uint size; /**< total vertex size in dwords */
};
@@ -75,10 +88,11 @@ struct vertex_info
* \return slot in which the attribute was added
*/
static INLINE uint
-draw_emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr, uint format,
- uint interp)
+draw_emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr,
+ attrib_format format, interp_mode interp)
{
const uint n = vinfo->num_attribs;
+ assert(n < MAX_VERT_ATTRIBS);
vinfo->attr_mask |= (1 << vfAttr);
vinfo->slot_to_attrib[n] = vfAttr;
vinfo->format[n] = format;
@@ -88,8 +102,17 @@ draw_emit_vertex_attr(struct vertex_info *vinfo, uint vfAttr, uint format,
}
+extern void draw_set_vertex_attributes( struct draw_context *draw,
+ const uint *attrs,
+ const interp_mode *interps,
+ unsigned nr_attrs );
+
+extern void draw_set_twoside_attributes(struct draw_context *draw,
+ uint front0, uint back0,
+ uint front1, uint back1);
+
+extern void draw_compute_vertex_size(struct vertex_info *vinfo);
-struct draw_context;
extern int draw_vertex_cache_check_space( struct draw_context *draw,
unsigned nr_verts );
diff --git a/src/mesa/pipe/i915simple/i915_state_derived.c b/src/mesa/pipe/i915simple/i915_state_derived.c
index 4347dadbd0b..792bb93b170 100644
--- a/src/mesa/pipe/i915simple/i915_state_derived.c
+++ b/src/mesa/pipe/i915simple/i915_state_derived.c
@@ -42,8 +42,8 @@
*/
static void calculate_vertex_layout( struct i915_context *i915 )
{
- const unsigned inputsRead = i915->fs.inputs_read;
- const uint colorInterp
+ const uint inputsRead = i915->fs.inputs_read;
+ const interp_mode colorInterp
= i915->setup.flatshade ? INTERP_CONSTANT : INTERP_LINEAR;
struct vertex_info *vinfo = &i915->current.vertex_info;
uint front0 = 0, back0 = 0, front1 = 0, back1 = 0;
diff --git a/src/mesa/pipe/softpipe/sp_prim_setup.c b/src/mesa/pipe/softpipe/sp_prim_setup.c
index e7b58e53140..83d317c36f7 100644
--- a/src/mesa/pipe/softpipe/sp_prim_setup.c
+++ b/src/mesa/pipe/softpipe/sp_prim_setup.c
@@ -38,9 +38,9 @@
#include "sp_quad.h"
#include "sp_prim_setup.h"
#include "pipe/draw/draw_private.h"
+#include "pipe/draw/draw_vertex.h"
#include "pipe/p_util.h"
-#include "pipe/draw/draw_vertex.h"
/**
@@ -461,7 +461,7 @@ static void tri_persp_coeff( struct setup_stage *setup,
*/
static void setup_tri_coefficients( struct setup_stage *setup )
{
- const enum interp_mode *interp = setup->softpipe->vertex_info.interp_mode;
+ const interp_mode *interp = setup->softpipe->vertex_info.interp_mode;
unsigned slot, j;
/* z and w are done by linear interpolation:
@@ -680,7 +680,7 @@ line_persp_coeff(struct setup_stage *setup, unsigned slot, unsigned i)
static INLINE void
setup_line_coefficients(struct setup_stage *setup, struct prim_header *prim)
{
- const enum interp_mode *interp = setup->softpipe->vertex_info.interp_mode;
+ const interp_mode *interp = setup->softpipe->vertex_info.interp_mode;
unsigned slot, j;
/* use setup->vmin, vmax to point to vertices */
diff --git a/src/mesa/pipe/softpipe/sp_state_derived.c b/src/mesa/pipe/softpipe/sp_state_derived.c
index 4880870e6e4..e08ed50a70e 100644
--- a/src/mesa/pipe/softpipe/sp_state_derived.c
+++ b/src/mesa/pipe/softpipe/sp_state_derived.c
@@ -43,8 +43,8 @@
*/
static void calculate_vertex_layout( struct softpipe_context *softpipe )
{
- const unsigned inputsRead = softpipe->fs.inputs_read;
- const uint colorInterp
+ const uint inputsRead = softpipe->fs.inputs_read;
+ const interp_mode colorInterp
= softpipe->setup.flatshade ? INTERP_CONSTANT : INTERP_LINEAR;
struct vertex_info *vinfo = &softpipe->vertex_info;
uint front0 = 0, back0 = 0, front1 = 0, back1 = 0;