etnaviv: remove imm_ prefix from etna_shader_uniform_info members

This prefix is meaningless.

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8062>
This commit is contained in:
Christian Gmeiner 2020-11-18 15:54:16 +01:00 committed by Marge Bot
parent fefbafb6ff
commit a5f4be4fcd
7 changed files with 47 additions and 47 deletions

View file

@ -183,13 +183,13 @@ copy_uniform_state_to_shader(struct etna_shader_variant *sobj, uint64_t *consts,
{
struct etna_shader_uniform_info *uinfo = &sobj->uniforms;
uinfo->imm_count = count * 4;
uinfo->imm_data = MALLOC(uinfo->imm_count * sizeof(*uinfo->imm_data));
uinfo->imm_contents = MALLOC(uinfo->imm_count * sizeof(*uinfo->imm_contents));
uinfo->count = count * 4;
uinfo->data = MALLOC(uinfo->count * sizeof(*uinfo->data));
uinfo->contents = MALLOC(uinfo->count * sizeof(*uinfo->contents));
for (unsigned i = 0; i < uinfo->imm_count; i++) {
uinfo->imm_data[i] = consts[i];
uinfo->imm_contents[i] = consts[i] >> 32;
for (unsigned i = 0; i < uinfo->count; i++) {
uinfo->data[i] = consts[i];
uinfo->contents[i] = consts[i] >> 32;
}
etna_set_shader_uniforms_dirty_flags(sobj);
@ -992,9 +992,9 @@ etna_compile_check_limits(struct etna_shader_variant *v)
return false;
}
if (v->uniforms.imm_count / 4 > max_uniforms) {
if (v->uniforms.count / 4 > max_uniforms) {
DBG("Number of uniforms (%d) exceeds maximum %d",
v->uniforms.imm_count / 4, max_uniforms);
v->uniforms.count / 4, max_uniforms);
return false;
}

View file

@ -2290,17 +2290,17 @@ copy_uniform_state_to_shader(struct etna_compile *c, struct etna_shader_variant
uint32_t count = c->imm_base + c->imm_size;
struct etna_shader_uniform_info *uinfo = &sobj->uniforms;
uinfo->imm_count = count;
uinfo->count = count;
uinfo->imm_data = malloc(count * sizeof(*c->imm_data));
uinfo->data = malloc(count * sizeof(*c->imm_data));
for (unsigned i = 0; i < c->imm_base; i++)
uinfo->imm_data[i] = i;
memcpy(&uinfo->imm_data[c->imm_base], c->imm_data, c->imm_size * sizeof(*c->imm_data));
uinfo->data[i] = i;
memcpy(&uinfo->data[c->imm_base], c->imm_data, c->imm_size * sizeof(*c->imm_data));
uinfo->imm_contents = malloc(count * sizeof(*c->imm_contents));
uinfo->contents = malloc(count * sizeof(*c->imm_contents));
for (unsigned i = 0; i < c->imm_base; i++)
uinfo->imm_contents[i] = ETNA_UNIFORM_UNIFORM;
memcpy(&uinfo->imm_contents[c->imm_base], c->imm_contents, c->imm_size * sizeof(*c->imm_contents));
uinfo->contents[i] = ETNA_UNIFORM_UNIFORM;
memcpy(&uinfo->contents[c->imm_base], c->imm_contents, c->imm_size * sizeof(*c->imm_contents));
etna_set_shader_uniforms_dirty_flags(sobj);
}

View file

@ -101,9 +101,9 @@ enum etna_uniform_contents {
};
struct etna_shader_uniform_info {
enum etna_uniform_contents *imm_contents;
uint32_t *imm_data;
uint32_t imm_count;
enum etna_uniform_contents *contents;
uint32_t *data;
uint32_t count;
};
struct etna_context {

View file

@ -102,25 +102,25 @@ retrieve_variant(struct blob_reader *blob, struct etna_shader_variant *v)
v->code = malloc(4 * v->code_size);
blob_copy_bytes(blob, v->code, 4 * v->code_size);
blob_copy_bytes(blob, &v->uniforms.imm_count, sizeof(v->uniforms.imm_count));
v->uniforms.imm_contents = malloc(v->uniforms.imm_count * sizeof(v->uniforms.imm_contents));
v->uniforms.imm_data = malloc(v->uniforms.imm_count * sizeof(v->uniforms.imm_data));
blob_copy_bytes(blob, &v->uniforms.count, sizeof(v->uniforms.count));
v->uniforms.contents = malloc(v->uniforms.count * sizeof(v->uniforms.contents));
v->uniforms.data = malloc(v->uniforms.count * sizeof(v->uniforms.data));
blob_copy_bytes(blob, v->uniforms.imm_contents, v->uniforms.imm_count * sizeof(v->uniforms.imm_contents));
blob_copy_bytes(blob, v->uniforms.imm_data, v->uniforms.imm_count * sizeof(v->uniforms.imm_data));
blob_copy_bytes(blob, v->uniforms.contents, v->uniforms.count * sizeof(v->uniforms.contents));
blob_copy_bytes(blob, v->uniforms.data, v->uniforms.count * sizeof(v->uniforms.data));
}
static void
store_variant(struct blob *blob, const struct etna_shader_variant *v)
{
const uint32_t imm_count = v->uniforms.imm_count;
const uint32_t imm_count = v->uniforms.count;
blob_write_bytes(blob, VARIANT_CACHE_PTR(v), VARIANT_CACHE_SIZE);
blob_write_bytes(blob, v->code, 4 * v->code_size);
blob_write_bytes(blob, &v->uniforms.imm_count, sizeof(v->uniforms.imm_count));
blob_write_bytes(blob, v->uniforms.imm_contents, imm_count * sizeof(v->uniforms.imm_contents));
blob_write_bytes(blob, v->uniforms.imm_data, imm_count * sizeof(v->uniforms.imm_data));
blob_write_bytes(blob, &v->uniforms.count, sizeof(v->uniforms.count));
blob_write_bytes(blob, v->uniforms.contents, imm_count * sizeof(v->uniforms.contents));
blob_write_bytes(blob, v->uniforms.data, imm_count * sizeof(v->uniforms.data));
}
bool

View file

@ -105,8 +105,8 @@ required_stream_size(struct etna_context *ctx)
size += ctx->vertex_elements->num_elements + 1;
/* uniforms - worst case (2 words per uniform load) */
size += ctx->shader.vs->uniforms.imm_count * 2;
size += ctx->shader.fs->uniforms.imm_count * 2;
size += ctx->shader.vs->uniforms.count * 2;
size += ctx->shader.fs->uniforms.count * 2;
/* shader */
size += ctx->shader_state.vs_inst_mem_size + 1;

View file

@ -71,13 +71,13 @@ etna_dump_shader(const struct etna_shader_variant *shader)
printf("num loops: %i\n", shader->num_loops);
printf("num temps: %i\n", shader->num_temps);
printf("immediates:\n");
for (int idx = 0; idx < shader->uniforms.imm_count; ++idx) {
for (int idx = 0; idx < shader->uniforms.count; ++idx) {
printf(" [%i].%s = %f (0x%08x) (%d)\n",
idx / 4,
tgsi_swizzle_names[idx % 4],
*((float *)&shader->uniforms.imm_data[idx]),
shader->uniforms.imm_data[idx],
shader->uniforms.imm_contents[idx]);
*((float *)&shader->uniforms.data[idx]),
shader->uniforms.data[idx],
shader->uniforms.contents[idx]);
}
printf("inputs:\n");
for (int idx = 0; idx < shader->infile.num_reg; ++idx) {
@ -302,8 +302,8 @@ etna_destroy_shader(struct etna_shader_variant *shader)
assert(shader);
FREE(shader->code);
FREE(shader->uniforms.imm_data);
FREE(shader->uniforms.imm_contents);
FREE(shader->uniforms.data);
FREE(shader->uniforms.contents);
FREE(shader);
}
@ -389,7 +389,7 @@ dump_shader_info(struct etna_shader_variant *v, struct pipe_debug_callback *debu
etna_shader_stage(v),
v->code_size,
v->num_temps,
v->uniforms.imm_count,
v->uniforms.count,
v->num_loops);
}

View file

@ -72,16 +72,16 @@ etna_uniforms_write(const struct etna_context *ctx,
uint32_t base = frag ? screen->specs.ps_uniforms_offset : screen->specs.vs_uniforms_offset;
unsigned idx;
if (!uinfo->imm_count)
if (!uinfo->count)
return;
etna_cmd_stream_reserve(stream, align(uinfo->imm_count + 1, 2));
etna_emit_load_state(stream, base >> 2, uinfo->imm_count, 0);
etna_cmd_stream_reserve(stream, align(uinfo->count + 1, 2));
etna_emit_load_state(stream, base >> 2, uinfo->count, 0);
for (uint32_t i = 0; i < uinfo->imm_count; i++) {
uint32_t val = uinfo->imm_data[i];
for (uint32_t i = 0; i < uinfo->count; i++) {
uint32_t val = uinfo->data[i];
switch (uinfo->imm_contents[i]) {
switch (uinfo->contents[i]) {
case ETNA_UNIFORM_CONSTANT:
etna_cmd_stream_emit(stream, val);
break;
@ -94,11 +94,11 @@ etna_uniforms_write(const struct etna_context *ctx,
case ETNA_UNIFORM_TEXRECT_SCALE_X:
case ETNA_UNIFORM_TEXRECT_SCALE_Y:
etna_cmd_stream_emit(stream,
get_texrect_scale(ctx, frag, uinfo->imm_contents[i], val));
get_texrect_scale(ctx, frag, uinfo->contents[i], val));
break;
case ETNA_UNIFORM_UBO0_ADDR ... ETNA_UNIFORM_UBOMAX_ADDR:
idx = uinfo->imm_contents[i] - ETNA_UNIFORM_UBO0_ADDR;
idx = uinfo->contents[i] - ETNA_UNIFORM_UBO0_ADDR;
etna_cmd_stream_reloc(stream, &(struct etna_reloc) {
.bo = etna_resource(cb[idx].buffer)->bo,
.flags = ETNA_RELOC_READ,
@ -112,7 +112,7 @@ etna_uniforms_write(const struct etna_context *ctx,
}
}
if ((uinfo->imm_count % 2) == 0)
if ((uinfo->count % 2) == 0)
etna_cmd_stream_emit(stream, 0);
}
@ -121,8 +121,8 @@ etna_set_shader_uniforms_dirty_flags(struct etna_shader_variant *sobj)
{
uint32_t dirty = 0;
for (uint32_t i = 0; i < sobj->uniforms.imm_count; i++) {
switch (sobj->uniforms.imm_contents[i]) {
for (uint32_t i = 0; i < sobj->uniforms.count; i++) {
switch (sobj->uniforms.contents[i]) {
default:
break;