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

media: hantro: Rename {prepare,finish}_run to {start,end}_prepare_run

hantro_prepare_run() and hantro_finish_run() are
slightly misleading, so let's rename it to something
a bit more clear.
Signed-off-by: default avatarEzequiel Garcia <ezequiel@collabora.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent 5980d402
...@@ -152,7 +152,7 @@ void hantro_watchdog(struct work_struct *work) ...@@ -152,7 +152,7 @@ void hantro_watchdog(struct work_struct *work)
} }
} }
void hantro_prepare_run(struct hantro_ctx *ctx) void hantro_start_prepare_run(struct hantro_ctx *ctx)
{ {
struct vb2_v4l2_buffer *src_buf; struct vb2_v4l2_buffer *src_buf;
...@@ -161,7 +161,7 @@ void hantro_prepare_run(struct hantro_ctx *ctx) ...@@ -161,7 +161,7 @@ void hantro_prepare_run(struct hantro_ctx *ctx)
&ctx->ctrl_handler); &ctx->ctrl_handler);
} }
void hantro_finish_run(struct hantro_ctx *ctx) void hantro_end_prepare_run(struct hantro_ctx *ctx)
{ {
struct vb2_v4l2_buffer *src_buf; struct vb2_v4l2_buffer *src_buf;
......
...@@ -288,7 +288,7 @@ void hantro_g1_h264_dec_run(struct hantro_ctx *ctx) ...@@ -288,7 +288,7 @@ void hantro_g1_h264_dec_run(struct hantro_ctx *ctx)
set_ref(ctx); set_ref(ctx);
set_buffers(ctx); set_buffers(ctx);
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
/* Start decoding! */ /* Start decoding! */
vdpu_write_relaxed(vpu, vdpu_write_relaxed(vpu,
......
...@@ -168,7 +168,7 @@ void hantro_g1_mpeg2_dec_run(struct hantro_ctx *ctx) ...@@ -168,7 +168,7 @@ void hantro_g1_mpeg2_dec_run(struct hantro_ctx *ctx)
dst_buf = hantro_get_dst_buf(ctx); dst_buf = hantro_get_dst_buf(ctx);
/* Apply request controls if any */ /* Apply request controls if any */
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
slice_params = hantro_get_ctrl(ctx, slice_params = hantro_get_ctrl(ctx,
V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS); V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS);
...@@ -244,7 +244,7 @@ void hantro_g1_mpeg2_dec_run(struct hantro_ctx *ctx) ...@@ -244,7 +244,7 @@ void hantro_g1_mpeg2_dec_run(struct hantro_ctx *ctx)
&dst_buf->vb2_buf, &dst_buf->vb2_buf,
sequence, picture, slice_params); sequence, picture, slice_params);
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
reg = G1_REG_DEC_E(1); reg = G1_REG_DEC_E(1);
vdpu_write(vpu, reg, G1_SWREG(1)); vdpu_write(vpu, reg, G1_SWREG(1));
......
...@@ -435,7 +435,7 @@ void hantro_g1_vp8_dec_run(struct hantro_ctx *ctx) ...@@ -435,7 +435,7 @@ void hantro_g1_vp8_dec_run(struct hantro_ctx *ctx)
u32 mb_width, mb_height; u32 mb_width, mb_height;
u32 reg; u32 reg;
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
hdr = hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER); hdr = hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER);
if (WARN_ON(!hdr)) if (WARN_ON(!hdr))
...@@ -496,7 +496,7 @@ void hantro_g1_vp8_dec_run(struct hantro_ctx *ctx) ...@@ -496,7 +496,7 @@ void hantro_g1_vp8_dec_run(struct hantro_ctx *ctx)
cfg_ref(ctx, hdr); cfg_ref(ctx, hdr);
cfg_buffers(ctx, hdr); cfg_buffers(ctx, hdr);
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
vdpu_write(vpu, G1_REG_INTERRUPT_DEC_E, G1_REG_INTERRUPT); vdpu_write(vpu, G1_REG_INTERRUPT_DEC_E, G1_REG_INTERRUPT);
} }
...@@ -87,7 +87,7 @@ void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx) ...@@ -87,7 +87,7 @@ void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx)
src_buf = hantro_get_src_buf(ctx); src_buf = hantro_get_src_buf(ctx);
dst_buf = hantro_get_dst_buf(ctx); dst_buf = hantro_get_dst_buf(ctx);
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
memset(&jpeg_ctx, 0, sizeof(jpeg_ctx)); memset(&jpeg_ctx, 0, sizeof(jpeg_ctx));
jpeg_ctx.buffer = vb2_plane_vaddr(&dst_buf->vb2_buf, 0); jpeg_ctx.buffer = vb2_plane_vaddr(&dst_buf->vb2_buf, 0);
...@@ -122,7 +122,7 @@ void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx) ...@@ -122,7 +122,7 @@ void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx)
| H1_REG_ENC_PIC_INTRA | H1_REG_ENC_PIC_INTRA
| H1_REG_ENC_CTRL_EN_BIT; | H1_REG_ENC_CTRL_EN_BIT;
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
vepu_write(vpu, reg, H1_REG_ENC_CTRL); vepu_write(vpu, reg, H1_REG_ENC_CTRL);
} }
...@@ -562,7 +562,7 @@ int hantro_h264_dec_prepare_run(struct hantro_ctx *ctx) ...@@ -562,7 +562,7 @@ int hantro_h264_dec_prepare_run(struct hantro_ctx *ctx)
struct hantro_h264_dec_ctrls *ctrls = &h264_ctx->ctrls; struct hantro_h264_dec_ctrls *ctrls = &h264_ctx->ctrls;
struct hantro_h264_reflist_builder reflist_builder; struct hantro_h264_reflist_builder reflist_builder;
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
ctrls->scaling = ctrls->scaling =
hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_H264_SCALING_MATRIX); hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_H264_SCALING_MATRIX);
......
...@@ -147,8 +147,8 @@ void hantro_watchdog(struct work_struct *work); ...@@ -147,8 +147,8 @@ void hantro_watchdog(struct work_struct *work);
void hantro_run(struct hantro_ctx *ctx); void hantro_run(struct hantro_ctx *ctx);
void hantro_irq_done(struct hantro_dev *vpu, unsigned int bytesused, void hantro_irq_done(struct hantro_dev *vpu, unsigned int bytesused,
enum vb2_buffer_state result); enum vb2_buffer_state result);
void hantro_prepare_run(struct hantro_ctx *ctx); void hantro_start_prepare_run(struct hantro_ctx *ctx);
void hantro_finish_run(struct hantro_ctx *ctx); void hantro_end_prepare_run(struct hantro_ctx *ctx);
void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx); void hantro_h1_jpeg_enc_run(struct hantro_ctx *ctx);
void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx); void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx);
......
...@@ -118,7 +118,7 @@ void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx) ...@@ -118,7 +118,7 @@ void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx)
src_buf = hantro_get_src_buf(ctx); src_buf = hantro_get_src_buf(ctx);
dst_buf = hantro_get_dst_buf(ctx); dst_buf = hantro_get_dst_buf(ctx);
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
memset(&jpeg_ctx, 0, sizeof(jpeg_ctx)); memset(&jpeg_ctx, 0, sizeof(jpeg_ctx));
jpeg_ctx.buffer = vb2_plane_vaddr(&dst_buf->vb2_buf, 0); jpeg_ctx.buffer = vb2_plane_vaddr(&dst_buf->vb2_buf, 0);
...@@ -156,6 +156,6 @@ void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx) ...@@ -156,6 +156,6 @@ void rk3399_vpu_jpeg_enc_run(struct hantro_ctx *ctx)
| VEPU_REG_ENCODE_ENABLE; | VEPU_REG_ENCODE_ENABLE;
/* Kick the watchdog and start encoding */ /* Kick the watchdog and start encoding */
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
vepu_write(vpu, reg, VEPU_REG_ENCODE_START); vepu_write(vpu, reg, VEPU_REG_ENCODE_START);
} }
...@@ -169,7 +169,7 @@ void rk3399_vpu_mpeg2_dec_run(struct hantro_ctx *ctx) ...@@ -169,7 +169,7 @@ void rk3399_vpu_mpeg2_dec_run(struct hantro_ctx *ctx)
src_buf = hantro_get_src_buf(ctx); src_buf = hantro_get_src_buf(ctx);
dst_buf = hantro_get_dst_buf(ctx); dst_buf = hantro_get_dst_buf(ctx);
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
slice_params = hantro_get_ctrl(ctx, slice_params = hantro_get_ctrl(ctx,
V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS); V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS);
...@@ -250,7 +250,7 @@ void rk3399_vpu_mpeg2_dec_run(struct hantro_ctx *ctx) ...@@ -250,7 +250,7 @@ void rk3399_vpu_mpeg2_dec_run(struct hantro_ctx *ctx)
sequence, picture, slice_params); sequence, picture, slice_params);
/* Kick the watchdog and start decoding */ /* Kick the watchdog and start decoding */
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
reg = vdpu_read(vpu, VDPU_SWREG(57)) | VDPU_REG_DEC_E(1); reg = vdpu_read(vpu, VDPU_SWREG(57)) | VDPU_REG_DEC_E(1);
vdpu_write(vpu, reg, VDPU_SWREG(57)); vdpu_write(vpu, reg, VDPU_SWREG(57));
......
...@@ -513,7 +513,7 @@ void rk3399_vpu_vp8_dec_run(struct hantro_ctx *ctx) ...@@ -513,7 +513,7 @@ void rk3399_vpu_vp8_dec_run(struct hantro_ctx *ctx)
u32 mb_width, mb_height; u32 mb_width, mb_height;
u32 reg; u32 reg;
hantro_prepare_run(ctx); hantro_start_prepare_run(ctx);
hdr = hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER); hdr = hantro_get_ctrl(ctx, V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER);
if (WARN_ON(!hdr)) if (WARN_ON(!hdr))
...@@ -587,7 +587,7 @@ void rk3399_vpu_vp8_dec_run(struct hantro_ctx *ctx) ...@@ -587,7 +587,7 @@ void rk3399_vpu_vp8_dec_run(struct hantro_ctx *ctx)
cfg_ref(ctx, hdr); cfg_ref(ctx, hdr);
cfg_buffers(ctx, hdr); cfg_buffers(ctx, hdr);
hantro_finish_run(ctx); hantro_end_prepare_run(ctx);
hantro_reg_write(vpu, &vp8_dec_start_dec, 1); hantro_reg_write(vpu, &vp8_dec_start_dec, 1);
} }
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