This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: videobuf2-core: clear memory related fields in 
__vb2_plane_dmabuf_put()
Author:  Yunke Cao <yun...@chromium.org>
Date:    Wed Aug 14 11:06:40 2024 +0900

Clear vb2_plane's memory related fields in __vb2_plane_dmabuf_put(),
including bytesused, length, fd and data_offset.

Remove the duplicated code in __prepare_dmabuf().

Signed-off-by: Yunke Cao <yun...@chromium.org>
Acked-by: Tomasz Figa <tf...@chromium.org>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/common/videobuf2/videobuf2-core.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

---

diff --git a/drivers/media/common/videobuf2/videobuf2-core.c 
b/drivers/media/common/videobuf2/videobuf2-core.c
index 0217392fcc0d..4d232b08f950 100644
--- a/drivers/media/common/videobuf2/videobuf2-core.c
+++ b/drivers/media/common/videobuf2/videobuf2-core.c
@@ -311,6 +311,10 @@ static void __vb2_plane_dmabuf_put(struct vb2_buffer *vb, 
struct vb2_plane *p)
        p->mem_priv = NULL;
        p->dbuf = NULL;
        p->dbuf_mapped = 0;
+       p->bytesused = 0;
+       p->length = 0;
+       p->m.fd = 0;
+       p->data_offset = 0;
 }
 
 /*
@@ -1420,10 +1424,6 @@ static int __prepare_dmabuf(struct vb2_buffer *vb)
 
                /* Release previously acquired memory if present */
                __vb2_plane_dmabuf_put(vb, &vb->planes[plane]);
-               vb->planes[plane].bytesused = 0;
-               vb->planes[plane].length = 0;
-               vb->planes[plane].m.fd = 0;
-               vb->planes[plane].data_offset = 0;
 
                /* Acquire each plane's memory */
                mem_priv = call_ptr_memop(attach_dmabuf,

Reply via email to