Commit 792196fd authored by Yunfei Dong's avatar Yunfei Dong Committed by Hans Verkuil

media: mediatek: vcodec: add decoder command to support stateless decoder

The supported decoder commands are different for stateless and
stateful architectures. Add stateless decoder commands to fix
the v4l2-compliance test error below.

Codec ioctls:
    VIDIOC_ENCODER_CMD returned -1 (Inappropriate ioctl for device)
    VIDIOC_TRY_ENCODER_CMD returned -1 (Inappropriate ioctl for device)
 test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
    VIDIOC_G_ENC_INDEX returned -1 (Inappropriate ioctl for device)
 test VIDIOC_G_ENC_INDEX: OK (Not Supported)
    VIDIOC_DECODER_CMD returned -1 (Invalid argument)
    VIDIOC_TRY_DECODER_CMD returned -1 (Invalid argument)
    VIDIOC_TRY_DECODER_CMD returned -1 (Invalid argument)
    fail: v4l2-test-codecs.cpp(126): ret
 test VIDIOC_(TRY_)DECODER_CMD: FAIL
Signed-off-by: default avatarYunfei Dong <yunfei.dong@mediatek.com>
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: default avatarSebastian Fricke <sebastian.fricke@collabora.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent 8f79b09b
...@@ -80,21 +80,18 @@ static struct mtk_q_data *mtk_vdec_get_q_data(struct mtk_vcodec_dec_ctx *ctx, ...@@ -80,21 +80,18 @@ static struct mtk_q_data *mtk_vdec_get_q_data(struct mtk_vcodec_dec_ctx *ctx,
return &ctx->q_data[MTK_Q_DATA_DST]; return &ctx->q_data[MTK_Q_DATA_DST];
} }
static int vidioc_try_decoder_cmd(struct file *file, void *priv, static int stateful_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
struct v4l2_decoder_cmd *cmd)
{ {
return v4l2_m2m_ioctl_try_decoder_cmd(file, priv, cmd); return v4l2_m2m_ioctl_try_decoder_cmd(file, priv, cmd);
} }
static int stateful_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
static int vidioc_decoder_cmd(struct file *file, void *priv,
struct v4l2_decoder_cmd *cmd)
{ {
struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv); struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
struct vb2_queue *src_vq, *dst_vq; struct vb2_queue *src_vq, *dst_vq;
int ret; int ret;
ret = vidioc_try_decoder_cmd(file, priv, cmd); ret = stateful_try_decoder_cmd(file, priv, cmd);
if (ret) if (ret)
return ret; return ret;
...@@ -128,6 +125,57 @@ static int vidioc_decoder_cmd(struct file *file, void *priv, ...@@ -128,6 +125,57 @@ static int vidioc_decoder_cmd(struct file *file, void *priv,
return 0; return 0;
} }
static int stateless_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
{
return v4l2_m2m_ioctl_stateless_try_decoder_cmd(file, priv, cmd);
}
static int stateless_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
{
struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
int ret;
ret = v4l2_m2m_ioctl_stateless_try_decoder_cmd(file, priv, cmd);
if (ret)
return ret;
mtk_v4l2_vdec_dbg(3, ctx, "decoder cmd=%u", cmd->cmd);
switch (cmd->cmd) {
case V4L2_DEC_CMD_FLUSH:
/*
* If the flag of the output buffer is equals V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF,
* this command will prevent dequeueing the capture buffer containing the last
* decoded frame. Or do nothing
*/
break;
default:
mtk_v4l2_vdec_err(ctx, "invalid stateless decoder cmd=%u", cmd->cmd);
return -EINVAL;
}
return 0;
}
static int vidioc_try_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
{
struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
if (ctx->dev->vdec_pdata->uses_stateless_api)
return stateless_try_decoder_cmd(file, priv, cmd);
return stateful_try_decoder_cmd(file, priv, cmd);
}
static int vidioc_decoder_cmd(struct file *file, void *priv, struct v4l2_decoder_cmd *cmd)
{
struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(priv);
if (ctx->dev->vdec_pdata->uses_stateless_api)
return stateless_decoder_cmd(file, priv, cmd);
return stateful_decoder_cmd(file, priv, cmd);
}
void mtk_vdec_unlock(struct mtk_vcodec_dec_ctx *ctx) void mtk_vdec_unlock(struct mtk_vcodec_dec_ctx *ctx)
{ {
mutex_unlock(&ctx->dev->dec_mutex[ctx->hw_id]); mutex_unlock(&ctx->dev->dec_mutex[ctx->hw_id]);
......
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