On 09.11.2015 06:45, Marek Olšák wrote:
> From: Marek Olšák <marek.ol...@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.daen...@amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to