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