[Mesa-dev] [PATCH 2/3] radeonsi: implement vertex color clamping

[Michel Dänzer]

On 11.10.2015 10:17, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
> 
> This is only supported in the compatibility profile (without GS and tess).

[...]

> diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c
> index 3aafe8a..a81060a 100644
> --- a/src/gallium/drivers/radeonsi/si_state.c
> +++ b/src/gallium/drivers/radeonsi/si_state.c
> @@ -760,6 +760,8 @@ static void *si_create_rs_state(struct pipe_context *ctx,
>  				   state->fill_back != PIPE_POLYGON_MODE_FILL) |
>  		S_028814_POLYMODE_FRONT_PTYPE(si_translate_fill(state->fill_front)) |
>  		S_028814_POLYMODE_BACK_PTYPE(si_translate_fill(state->fill_back)));
> +	si_pm4_set_reg(pm4, R_00B130_SPI_SHADER_USER_DATA_VS_0 +
> +		            SI_SGPR_VS_STATE_BITS * 4, state->clamp_vertex_color);

The second line added here seems to have a few too many spaces of
indentation. With that fixed, this series is

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer