Am Mittwoch, den 12.12.2018, 14:45 +0100 schrieb Christian Gmeiner: > There is no need to have an extra ctx paramter as all the other > parameters carry all the needed information. > > Signed-off-by: Christian Gmeiner <[email protected]>
Reviewed-by: Lucas Stach <[email protected]> > --- > src/gallium/drivers/etnaviv/etnaviv_shader.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/src/gallium/drivers/etnaviv/etnaviv_shader.c > b/src/gallium/drivers/etnaviv/etnaviv_shader.c > index 27c735b83b..d2d736bdee 100644 > --- a/src/gallium/drivers/etnaviv/etnaviv_shader.c > +++ b/src/gallium/drivers/etnaviv/etnaviv_shader.c > @@ -230,8 +230,7 @@ etna_shader_link(struct etna_context *ctx) > } > > static bool > -etna_shader_update_vs_inputs(struct etna_context *ctx, > - struct compiled_shader_state *cs, > +etna_shader_update_vs_inputs(struct compiled_shader_state *cs, > const struct etna_shader_variant *vs, > const struct compiled_vertex_elements_state > *ves) > { > @@ -246,7 +245,7 @@ etna_shader_update_vs_inputs(struct etna_context *ctx, > num_vs_inputs = MAX2(ves->num_elements, vs->infile.num_reg); > if (num_vs_inputs != ves->num_elements) { > BUG("Number of elements %u does not match the number of VS inputs %zu", > - ctx->vertex_elements->num_elements, > ctx->shader.vs->infile.num_reg); > + ves->num_elements, vs->infile.num_reg); > return false; > } > > @@ -312,7 +311,7 @@ dump_shader_info(struct etna_shader_variant *v, struct > pipe_debug_callback *debu > bool > etna_shader_update_vertex(struct etna_context *ctx) > { > - return etna_shader_update_vs_inputs(ctx, &ctx->shader_state, > ctx->shader.vs, > + return etna_shader_update_vs_inputs(&ctx->shader_state, ctx->shader.vs, > ctx->vertex_elements); > } > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
