From: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
To: Boris Brezillon <boris.brezillon@collabora.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hans.verkuil@cisco.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Sakari Ailus <sakari.ailus@iki.fi>,
linux-media@vger.kernel.org
Cc: Tomasz Figa <tfiga@chromium.org>,
Nicolas Dufresne <nicolas@ndufresne.ca>,
kernel@collabora.com, Ezequiel Garcia <ezequiel@collabora.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@siol.net>,
Alexandre Courbot <acourbot@chromium.org>,
Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCH RFC 4/6] media: cedrus: Prepare things to support !compound controls
Date: Wed, 05 Jun 2019 22:57:21 +0200 [thread overview]
Message-ID: <224115574fd3ebb3309c726b63e7a61ff66fd888.camel@bootlin.com> (raw)
In-Reply-To: <20190603110946.4952-5-boris.brezillon@collabora.com>
Hi,
Le lundi 03 juin 2019 à 13:09 +0200, Boris Brezillon a écrit :
> We are about to add a menu control, so let's make the code more generic
> to support other control types.
Did you have a chance to test this or should I better give it a spin
before adding my Reviewed-By?
Cheers,
Paul
> Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
> ---
> drivers/staging/media/sunxi/cedrus/cedrus.c | 47 ++++++++++++---------
> drivers/staging/media/sunxi/cedrus/cedrus.h | 3 +-
> 2 files changed, 29 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/staging/media/sunxi/cedrus/cedrus.c b/drivers/staging/media/sunxi/cedrus/cedrus.c
> index 370937edfc14..378032fe71f9 100644
> --- a/drivers/staging/media/sunxi/cedrus/cedrus.c
> +++ b/drivers/staging/media/sunxi/cedrus/cedrus.c
> @@ -29,44 +29,44 @@
>
> static const struct cedrus_control cedrus_controls[] = {
> {
> - .id = V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS,
> - .elem_size = sizeof(struct v4l2_ctrl_mpeg2_slice_params),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_mpeg2_slice_params),
> .codec = CEDRUS_CODEC_MPEG2,
> .required = true,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_MPEG2_QUANTIZATION,
> - .elem_size = sizeof(struct v4l2_ctrl_mpeg2_quantization),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_MPEG2_QUANTIZATION,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_mpeg2_quantization),
> .codec = CEDRUS_CODEC_MPEG2,
> .required = false,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_H264_DECODE_PARAMS,
> - .elem_size = sizeof(struct v4l2_ctrl_h264_decode_params),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_H264_DECODE_PARAMS,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_h264_decode_params),
> .codec = CEDRUS_CODEC_H264,
> .required = true,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_H264_SLICE_PARAMS,
> - .elem_size = sizeof(struct v4l2_ctrl_h264_slice_params),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_H264_SLICE_PARAMS,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_h264_slice_params),
> .codec = CEDRUS_CODEC_H264,
> .required = true,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_H264_SPS,
> - .elem_size = sizeof(struct v4l2_ctrl_h264_sps),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_H264_SPS,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_h264_sps),
> .codec = CEDRUS_CODEC_H264,
> .required = true,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_H264_PPS,
> - .elem_size = sizeof(struct v4l2_ctrl_h264_pps),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_H264_PPS,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_h264_pps),
> .codec = CEDRUS_CODEC_H264,
> .required = true,
> },
> {
> - .id = V4L2_CID_MPEG_VIDEO_H264_SCALING_MATRIX,
> - .elem_size = sizeof(struct v4l2_ctrl_h264_scaling_matrix),
> + .cfg.id = V4L2_CID_MPEG_VIDEO_H264_SCALING_MATRIX,
> + .cfg.elem_size = sizeof(struct v4l2_ctrl_h264_scaling_matrix),
> .codec = CEDRUS_CODEC_H264,
> .required = true,
> },
> @@ -106,12 +106,21 @@ static int cedrus_init_ctrls(struct cedrus_dev *dev, struct cedrus_ctx *ctx)
> return -ENOMEM;
>
> for (i = 0; i < CEDRUS_CONTROLS_COUNT; i++) {
> - struct v4l2_ctrl_config cfg = {};
> + const struct v4l2_ctrl_config *cfg = &cedrus_controls[i].cfg;
>
> - cfg.elem_size = cedrus_controls[i].elem_size;
> - cfg.id = cedrus_controls[i].id;
> + if (cfg->elem_size)
> + ctrl = v4l2_ctrl_new_custom(hdl, cfg, NULL);
> + else if (cfg->type == V4L2_CTRL_TYPE_MENU ||
> + cfg->type == V4L2_CTRL_TYPE_INTEGER_MENU)
> + ctrl = v4l2_ctrl_new_std_menu(hdl, NULL,
> + cfg->id, cfg->max,
> + cfg->menu_skip_mask,
> + cfg->def);
> + else
> + ctrl = v4l2_ctrl_new_std(hdl, NULL, cfg->id, cfg->min,
> + cfg->max, cfg->step,
> + cfg->def);
>
> - ctrl = v4l2_ctrl_new_custom(hdl, &cfg, NULL);
> if (hdl->error) {
> v4l2_err(&dev->v4l2_dev,
> "Failed to create new custom control\n");
> @@ -178,7 +187,7 @@ static int cedrus_request_validate(struct media_request *req)
> continue;
>
> ctrl_test = v4l2_ctrl_request_hdl_ctrl_find(hdl,
> - cedrus_controls[i].id);
> + cedrus_controls[i].cfg.id);
> if (!ctrl_test) {
> v4l2_info(&ctx->dev->v4l2_dev,
> "Missing required codec control\n");
> diff --git a/drivers/staging/media/sunxi/cedrus/cedrus.h b/drivers/staging/media/sunxi/cedrus/cedrus.h
> index 3f476d0fd981..69c037724d93 100644
> --- a/drivers/staging/media/sunxi/cedrus/cedrus.h
> +++ b/drivers/staging/media/sunxi/cedrus/cedrus.h
> @@ -49,8 +49,7 @@ enum cedrus_h264_pic_type {
> };
>
> struct cedrus_control {
> - u32 id;
> - u32 elem_size;
> + struct v4l2_ctrl_config cfg;
> enum cedrus_codec codec;
> unsigned char required:1;
> };
next prev parent reply other threads:[~2019-06-05 20:57 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-03 11:09 [PATCH RFC 0/6] media: uapi: h264: First batch of adjusments Boris Brezillon
2019-06-03 11:09 ` [PATCH RFC 1/6] media: uapi: h264: Clarify our expectations regarding NAL header format Boris Brezillon
2019-06-03 11:09 ` [PATCH RFC 2/6] media: uapi: h264: Add the concept of decoding mode Boris Brezillon
2019-06-03 12:30 ` Thierry Reding
2019-06-03 12:51 ` Boris Brezillon
2019-06-03 14:05 ` Thierry Reding
2019-06-03 15:37 ` Boris Brezillon
2019-06-04 8:16 ` Thierry Reding
2019-06-05 20:48 ` Paul Kocialkowski
2019-06-06 6:55 ` Boris Brezillon
2019-06-05 20:55 ` Paul Kocialkowski
2019-06-03 11:09 ` [PATCH RFC 3/6] media: uapi: h264: Get rid of the p0/b0/b1 ref-lists Boris Brezillon
2019-06-03 11:09 ` [PATCH RFC 4/6] media: cedrus: Prepare things to support !compound controls Boris Brezillon
2019-06-05 20:57 ` Paul Kocialkowski [this message]
2019-06-06 6:58 ` Boris Brezillon
2019-06-03 11:09 ` [PATCH RFC 5/6] media: cedrus: Make the slice_params array size limitation more explicit Boris Brezillon
2019-06-03 21:48 ` Jernej Škrabec
2019-06-03 23:55 ` Nicolas Dufresne
2019-06-04 8:12 ` Thierry Reding
2019-06-04 8:28 ` Boris Brezillon
2019-06-04 14:31 ` Nicolas Dufresne
2019-06-05 21:01 ` Paul Kocialkowski
2019-06-06 6:59 ` Boris Brezillon
2019-06-03 11:09 ` [PATCH RFC 6/6] media: cedrus: Add the H264_DECODING_MODE control Boris Brezillon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=224115574fd3ebb3309c726b63e7a61ff66fd888.camel@bootlin.com \
--to=paul.kocialkowski@bootlin.com \
--cc=acourbot@chromium.org \
--cc=boris.brezillon@collabora.com \
--cc=ezequiel@collabora.com \
--cc=hans.verkuil@cisco.com \
--cc=jernej.skrabec@siol.net \
--cc=jonas@kwiboo.se \
--cc=kernel@collabora.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=nicolas@ndufresne.ca \
--cc=sakari.ailus@iki.fi \
--cc=tfiga@chromium.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).