[Mesa-dev] [PATCH 2/7] radeonsi: set the DISABLE_WR_CONFIRM flag on CI-VI as well

[Michel Dänzer]

On 09.11.2015 06:45, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
> 
> I missed this in commit c3e527f93d4281ad6e2ca165eaf6ff588e4faefa
>     radeonsi: only enable write confirmation on the last CP DMA packet
> ---
>  src/gallium/drivers/radeonsi/si_cp_dma.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/gallium/drivers/radeonsi/si_cp_dma.c b/src/gallium/drivers/radeonsi/si_cp_dma.c
> index 7b8c6d0..55d423a 100644
> --- a/src/gallium/drivers/radeonsi/si_cp_dma.c
> +++ b/src/gallium/drivers/radeonsi/si_cp_dma.c
> @@ -64,7 +64,7 @@ static void si_emit_cp_dma_copy_buffer(struct si_context *sctx,
>  		radeon_emit(cs, src_va >> 32);		/* SRC_ADDR_HI [31:0] */
>  		radeon_emit(cs, dst_va);		/* DST_ADDR_LO [31:0] */
>  		radeon_emit(cs, dst_va >> 32);		/* DST_ADDR_HI [31:0] */
> -		radeon_emit(cs, size | raw_wait);	/* COMMAND [29:22] | BYTE_COUNT [20:0] */
> +		radeon_emit(cs, size | wr_confirm |raw_wait);	/* COMMAND [29:22] | BYTE_COUNT [20:0] */

Missing space after "|". With that fixed, this patch 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