Commit a4d3d612 authored by Ezequiel Garcia's avatar Ezequiel Garcia Committed by Mauro Carvalho Chehab

media: v4l2-mem2mem: Rename v4l2_m2m_buf_copy_data to v4l2_m2m_buf_copy_metadata

The v4l2_m2m_buf_copy_data helper is used to copy the buffer
metadata, such as its timestamp and its flags.

Therefore, the v4l2_m2m_buf_copy_metadata name is more clear
and avoids confusion with a payload data copy.
Signed-off-by: default avatarEzequiel Garcia <ezequiel@collabora.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
[hverkuil-cisco@xs4all.nl: also fix cedrus_dec.c]
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent c2d88e7d
...@@ -199,7 +199,7 @@ static int device_process(struct vicodec_ctx *ctx, ...@@ -199,7 +199,7 @@ static int device_process(struct vicodec_ctx *ctx,
dst_vb->sequence = q_dst->sequence++; dst_vb->sequence = q_dst->sequence++;
dst_vb->flags &= ~V4L2_BUF_FLAG_LAST; dst_vb->flags &= ~V4L2_BUF_FLAG_LAST;
v4l2_m2m_buf_copy_data(src_vb, dst_vb, !ctx->is_enc); v4l2_m2m_buf_copy_metadata(src_vb, dst_vb, !ctx->is_enc);
return 0; return 0;
} }
...@@ -414,7 +414,7 @@ static void set_last_buffer(struct vb2_v4l2_buffer *dst_buf, ...@@ -414,7 +414,7 @@ static void set_last_buffer(struct vb2_v4l2_buffer *dst_buf,
vb2_set_plane_payload(&dst_buf->vb2_buf, 0, 0); vb2_set_plane_payload(&dst_buf->vb2_buf, 0, 0);
dst_buf->sequence = q_dst->sequence++; dst_buf->sequence = q_dst->sequence++;
v4l2_m2m_buf_copy_data(src_buf, dst_buf, !ctx->is_enc); v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, !ctx->is_enc);
dst_buf->flags |= V4L2_BUF_FLAG_LAST; dst_buf->flags |= V4L2_BUF_FLAG_LAST;
v4l2_m2m_buf_done(dst_buf, VB2_BUF_STATE_DONE); v4l2_m2m_buf_done(dst_buf, VB2_BUF_STATE_DONE);
} }
......
...@@ -432,7 +432,7 @@ static int device_process(struct vim2m_ctx *ctx, ...@@ -432,7 +432,7 @@ static int device_process(struct vim2m_ctx *ctx,
out_vb->sequence = get_q_data(ctx, out_vb->sequence = get_q_data(ctx,
V4L2_BUF_TYPE_VIDEO_CAPTURE)->sequence++; V4L2_BUF_TYPE_VIDEO_CAPTURE)->sequence++;
in_vb->sequence = q_data_in->sequence++; in_vb->sequence = q_data_in->sequence++;
v4l2_m2m_buf_copy_data(in_vb, out_vb, true); v4l2_m2m_buf_copy_metadata(in_vb, out_vb, true);
if (ctx->mode & MEM2MEM_VFLIP) { if (ctx->mode & MEM2MEM_VFLIP) {
start = height - 1; start = height - 1;
......
...@@ -975,9 +975,9 @@ void v4l2_m2m_buf_queue(struct v4l2_m2m_ctx *m2m_ctx, ...@@ -975,9 +975,9 @@ void v4l2_m2m_buf_queue(struct v4l2_m2m_ctx *m2m_ctx,
} }
EXPORT_SYMBOL_GPL(v4l2_m2m_buf_queue); EXPORT_SYMBOL_GPL(v4l2_m2m_buf_queue);
void v4l2_m2m_buf_copy_data(const struct vb2_v4l2_buffer *out_vb, void v4l2_m2m_buf_copy_metadata(const struct vb2_v4l2_buffer *out_vb,
struct vb2_v4l2_buffer *cap_vb, struct vb2_v4l2_buffer *cap_vb,
bool copy_frame_flags) bool copy_frame_flags)
{ {
u32 mask = V4L2_BUF_FLAG_TIMECODE | V4L2_BUF_FLAG_TSTAMP_SRC_MASK; u32 mask = V4L2_BUF_FLAG_TIMECODE | V4L2_BUF_FLAG_TSTAMP_SRC_MASK;
...@@ -993,7 +993,7 @@ void v4l2_m2m_buf_copy_data(const struct vb2_v4l2_buffer *out_vb, ...@@ -993,7 +993,7 @@ void v4l2_m2m_buf_copy_data(const struct vb2_v4l2_buffer *out_vb,
cap_vb->flags &= ~mask; cap_vb->flags &= ~mask;
cap_vb->flags |= out_vb->flags & mask; cap_vb->flags |= out_vb->flags & mask;
} }
EXPORT_SYMBOL_GPL(v4l2_m2m_buf_copy_data); EXPORT_SYMBOL_GPL(v4l2_m2m_buf_copy_metadata);
void v4l2_m2m_request_queue(struct media_request *req) void v4l2_m2m_request_queue(struct media_request *req)
{ {
......
...@@ -50,7 +50,7 @@ void cedrus_device_run(void *priv) ...@@ -50,7 +50,7 @@ void cedrus_device_run(void *priv)
break; break;
} }
v4l2_m2m_buf_copy_data(run.src, run.dst, true); v4l2_m2m_buf_copy_metadata(run.src, run.dst, true);
dev->dec_ops[ctx->current_codec]->setup(ctx, &run); dev->dec_ops[ctx->current_codec]->setup(ctx, &run);
......
...@@ -623,11 +623,11 @@ v4l2_m2m_dst_buf_remove_by_idx(struct v4l2_m2m_ctx *m2m_ctx, unsigned int idx) ...@@ -623,11 +623,11 @@ v4l2_m2m_dst_buf_remove_by_idx(struct v4l2_m2m_ctx *m2m_ctx, unsigned int idx)
} }
/** /**
* v4l2_m2m_buf_copy_data() - copy buffer data from the output buffer to the * v4l2_m2m_buf_copy_metadata() - copy buffer metadata from
* capture buffer * the output buffer to the capture buffer
* *
* @out_vb: the output buffer that is the source of the data. * @out_vb: the output buffer that is the source of the metadata.
* @cap_vb: the capture buffer that will receive the data. * @cap_vb: the capture buffer that will receive the metadata.
* @copy_frame_flags: copy the KEY/B/PFRAME flags as well. * @copy_frame_flags: copy the KEY/B/PFRAME flags as well.
* *
* This helper function copies the timestamp, timecode (if the TIMECODE * This helper function copies the timestamp, timecode (if the TIMECODE
...@@ -638,9 +638,9 @@ v4l2_m2m_dst_buf_remove_by_idx(struct v4l2_m2m_ctx *m2m_ctx, unsigned int idx) ...@@ -638,9 +638,9 @@ v4l2_m2m_dst_buf_remove_by_idx(struct v4l2_m2m_ctx *m2m_ctx, unsigned int idx)
* flags are not copied. This is typically needed for encoders that * flags are not copied. This is typically needed for encoders that
* set this bits explicitly. * set this bits explicitly.
*/ */
void v4l2_m2m_buf_copy_data(const struct vb2_v4l2_buffer *out_vb, void v4l2_m2m_buf_copy_metadata(const struct vb2_v4l2_buffer *out_vb,
struct vb2_v4l2_buffer *cap_vb, struct vb2_v4l2_buffer *cap_vb,
bool copy_frame_flags); bool copy_frame_flags);
/* v4l2 request helper */ /* v4l2 request helper */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment