From: Hariprasad Kelam <hariprasad.kelam@gmail.com>
Date: Thu, 25 Jul 2019 03:06:02 +0000 (-0400)
Subject: media: staging: media: hantro: Remove call to memset after dma_alloc_coherent
X-Git-Tag: v5.15~5520^2~205
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aa0e9e72733b069a655a3fe9167f69de103830cc;p=platform%2Fkernel%2Flinux-starfive.git

media: staging: media: hantro: Remove call to memset after dma_alloc_coherent

fix below issue reported by coccicheck
/drivers/staging/media/hantro/hantro_vp8.c:149:16-34: WARNING:
dma_alloc_coherent use in aux_buf -> cpu already zeroes out memory,  so
memset is not needed

Signed-off-by: Hariprasad Kelam <hariprasad.kelam@gmail.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
---

diff --git a/drivers/staging/media/hantro/hantro_vp8.c b/drivers/staging/media/hantro/hantro_vp8.c
index 66c4533..363ddda 100644
--- a/drivers/staging/media/hantro/hantro_vp8.c
+++ b/drivers/staging/media/hantro/hantro_vp8.c
@@ -151,8 +151,6 @@ int hantro_vp8_dec_init(struct hantro_ctx *ctx)
 	if (!aux_buf->cpu)
 		return -ENOMEM;
 
-	memset(aux_buf->cpu, 0, aux_buf->size);
-
 	/*
 	 * Allocate probability table buffer,
 	 * total 1208 bytes, 4K page is far enough.