r600g: use a helper to add an initialized atom

Instead of writing to rctx->atoms directly use a helper to take
advantage of assert checks.

Signed-off-by: Marek Olšák <marek.olsak@amd.com>

diff --git a/src/gallium/drivers/r600/evergreen_state.c b/src/gallium/drivers/r600/evergreen_state.c
index bb5338304ea6f4d579fa7b257f1449ccc9b720f2..f7a76f6cac73b499e702a390489f5a116b4d98da 100644 (file)
--- a/src/gallium/drivers/r600/evergreen_state.c
+++ b/src/gallium/drivers/r600/evergreen_state.c
@@ -3470,8 +3470,8 @@ void evergreen_init_state_functions(struct r600_context *rctx)
        }
        r600_init_atom(rctx, &rctx->stencil_ref.atom, id++, r600_emit_stencil_ref, 4);
        r600_init_atom(rctx, &rctx->vertex_fetch_shader.atom, id++, evergreen_emit_vertex_fetch_shader, 5);
-       rctx->atoms[id++] = &rctx->b.streamout.begin_atom;
-       rctx->atoms[id++] = &rctx->b.streamout.enable_atom;
+       r600_add_atom(rctx, &rctx->b.streamout.begin_atom, id++);
+       r600_add_atom(rctx, &rctx->b.streamout.enable_atom, id++);
        r600_init_atom(rctx, &rctx->vertex_shader.atom, id++, r600_emit_shader, 23);
        r600_init_atom(rctx, &rctx->pixel_shader.atom, id++, r600_emit_shader, 0);
        r600_init_atom(rctx, &rctx->geometry_shader.atom, id++, r600_emit_shader, 0);

diff --git a/src/gallium/drivers/r600/r600_pipe.h b/src/gallium/drivers/r600/r600_pipe.h
index 3653e8fd0af8dbb1c6cb8f4f37edd08af95fb357..8b612698b13f30d57b4d0160e234089818367e28 100644 (file)
--- a/src/gallium/drivers/r600/r600_pipe.h
+++ b/src/gallium/drivers/r600/r600_pipe.h
@@ -671,6 +671,7 @@ void r600_emit_clip_misc_state(struct r600_context *rctx, struct r600_atom *atom
 void r600_emit_stencil_ref(struct r600_context *rctx, struct r600_atom *atom);
 void r600_emit_viewport_state(struct r600_context *rctx, struct r600_atom *atom);
 void r600_emit_shader(struct r600_context *rctx, struct r600_atom *a);
+void r600_add_atom(struct r600_context *rctx, struct r600_atom *atom, unsigned id);
 void r600_init_atom(struct r600_context *rctx, struct r600_atom *atom, unsigned id,
                    void (*emit)(struct r600_context *ctx, struct r600_atom *state),
                    unsigned num_dw);

diff --git a/src/gallium/drivers/r600/r600_state.c b/src/gallium/drivers/r600/r600_state.c
index 7fb03b4cff6542065a005eb0f7ce10bcd061f368..848818842aacfdadd6a566545ffb7824007c7721 100644 (file)
--- a/src/gallium/drivers/r600/r600_state.c
+++ b/src/gallium/drivers/r600/r600_state.c
@@ -3074,8 +3074,8 @@ void r600_init_state_functions(struct r600_context *rctx)
        r600_init_atom(rctx, &rctx->config_state.atom, id++, r600_emit_config_state, 3);
        r600_init_atom(rctx, &rctx->stencil_ref.atom, id++, r600_emit_stencil_ref, 4);
        r600_init_atom(rctx, &rctx->vertex_fetch_shader.atom, id++, r600_emit_vertex_fetch_shader, 5);
-       rctx->atoms[id++] = &rctx->b.streamout.begin_atom;
-       rctx->atoms[id++] = &rctx->b.streamout.enable_atom;
+       r600_add_atom(rctx, &rctx->b.streamout.begin_atom, id++);
+       r600_add_atom(rctx, &rctx->b.streamout.enable_atom, id++);
        r600_init_atom(rctx, &rctx->vertex_shader.atom, id++, r600_emit_shader, 23);
        r600_init_atom(rctx, &rctx->pixel_shader.atom, id++, r600_emit_shader, 0);
        r600_init_atom(rctx, &rctx->geometry_shader.atom, id++, r600_emit_shader, 0);

diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c
index 2d654dc5eb73f789762fbbce6cac42f9e0b8704c..a4238a2fec9f93db4b2bc7ccf55a32a86ba49760 100644 (file)
--- a/src/gallium/drivers/r600/r600_state_common.c
+++ b/src/gallium/drivers/r600/r600_state_common.c
@@ -47,18 +47,25 @@ void r600_release_command_buffer(struct r600_command_buffer *cb)
        FREE(cb->buf);
 }
 
+void r600_add_atom(struct r600_context *rctx,
+                  struct r600_atom *atom,
+                  unsigned id)
+{
+       assert(id < R600_NUM_ATOMS);
+       assert(rctx->atoms[id] == NULL);
+       rctx->atoms[id] = atom;
+       atom->dirty = false;
+}
+
 void r600_init_atom(struct r600_context *rctx,
                    struct r600_atom *atom,
                    unsigned id,
                    void (*emit)(struct r600_context *ctx, struct r600_atom *state),
                    unsigned num_dw)
 {
-       assert(id < R600_NUM_ATOMS);
-       assert(rctx->atoms[id] == NULL);
-       rctx->atoms[id] = atom;
        atom->emit = (void*)emit;
        atom->num_dw = num_dw;
-       atom->dirty = false;
+       r600_add_atom(rctx, atom, id);
 }
 
 void r600_emit_cso_state(struct r600_context *rctx, struct r600_atom *atom)