From: "mirela.rabulea@oss.nxp.com" <mirela.rabulea@oss.nxp.com>
To: Ming Qian <ming.qian@nxp.com>,
mchehab@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de
Cc: hverkuil-cisco@xs4all.nl, kernel@pengutronix.de,
festevam@gmail.com, linux-imx@nxp.com,
linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v4] media: imx-jpeg: Encoder support to set jpeg quality
Date: Fri, 22 Apr 2022 12:56:18 +0300 [thread overview]
Message-ID: <77a60ba0-91f0-05ae-d4d5-2675783e70f7@oss.nxp.com> (raw)
In-Reply-To: <20220422015320.14600-1-ming.qian@nxp.com>
On 22.04.2022 04:53, Ming Qian wrote:
> Implement V4L2_CID_JPEG_COMPRESSION_QUALITY
> to set jpeg quality
>
> Signed-off-by: Ming Qian <ming.qian@nxp.com>
Reviewed-by: Mirela Rabulea <mirela.rabulea@nxp.com>
> ---
> v4
> - put the changelog in wrong place in v3
> v3
> - put this v3 information below the --- line
> v2
> - free ctrl handler if error is set and return error
>
> .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c | 11 +++-
> .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h | 1 +
> .../media/platform/nxp/imx-jpeg/mxc-jpeg.c | 57 +++++++++++++++++++
> .../media/platform/nxp/imx-jpeg/mxc-jpeg.h | 2 +
> 4 files changed, 68 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c
> index 29c604b1b179..c482228262a3 100644
> --- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c
> +++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c
> @@ -100,9 +100,6 @@ void mxc_jpeg_enc_mode_conf(struct device *dev, void __iomem *reg)
>
> /* all markers and segments */
> writel(0x3ff, reg + CAST_CFG_MODE);
> -
> - /* quality factor */
> - writel(0x4b, reg + CAST_QUALITY);
> }
>
> void mxc_jpeg_enc_mode_go(struct device *dev, void __iomem *reg)
> @@ -114,6 +111,14 @@ void mxc_jpeg_enc_mode_go(struct device *dev, void __iomem *reg)
> writel(0x140, reg + CAST_MODE);
> }
>
> +void mxc_jpeg_enc_set_quality(struct device *dev, void __iomem *reg, u8 quality)
> +{
> + dev_dbg(dev, "CAST Encoder Quality %d...\n", quality);
> +
> + /* quality factor */
> + writel(quality, reg + CAST_QUALITY);
> +}
> +
> void mxc_jpeg_dec_mode_go(struct device *dev, void __iomem *reg)
> {
> dev_dbg(dev, "CAST Decoder GO...\n");
> diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h
> index ae70d3a0dc24..356e40140987 100644
> --- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h
> +++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h
> @@ -119,6 +119,7 @@ int mxc_jpeg_enable(void __iomem *reg);
> void wait_frmdone(struct device *dev, void __iomem *reg);
> void mxc_jpeg_enc_mode_conf(struct device *dev, void __iomem *reg);
> void mxc_jpeg_enc_mode_go(struct device *dev, void __iomem *reg);
> +void mxc_jpeg_enc_set_quality(struct device *dev, void __iomem *reg, u8 quality);
> void mxc_jpeg_dec_mode_go(struct device *dev, void __iomem *reg);
> int mxc_jpeg_get_slot(void __iomem *reg);
> u32 mxc_jpeg_get_offset(void __iomem *reg, int slot);
> diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> index 0c3a1efbeae7..1bd245ba00e2 100644
> --- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> +++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> @@ -624,6 +624,7 @@ static irqreturn_t mxc_jpeg_dec_irq(int irq, void *priv)
> ctx->enc_state == MXC_JPEG_ENC_CONF) {
> ctx->enc_state = MXC_JPEG_ENCODING;
> dev_dbg(dev, "Encoder config finished. Start encoding...\n");
> + mxc_jpeg_enc_set_quality(dev, reg, ctx->jpeg_quality);
> mxc_jpeg_enc_mode_go(dev, reg);
> goto job_unlock;
> }
> @@ -1563,6 +1564,51 @@ static void mxc_jpeg_set_default_params(struct mxc_jpeg_ctx *ctx)
> }
> }
>
> +static int mxc_jpeg_s_ctrl(struct v4l2_ctrl *ctrl)
> +{
> + struct mxc_jpeg_ctx *ctx =
> + container_of(ctrl->handler, struct mxc_jpeg_ctx, ctrl_handler);
> +
> + switch (ctrl->id) {
> + case V4L2_CID_JPEG_COMPRESSION_QUALITY:
> + ctx->jpeg_quality = ctrl->val;
> + break;
> + default:
> + dev_err(ctx->mxc_jpeg->dev, "Invalid control, id = %d, val = %d\n",
> + ctrl->id, ctrl->val);
> + return -EINVAL;
> + }
> +
> + return 0;
> +}
> +
> +static const struct v4l2_ctrl_ops mxc_jpeg_ctrl_ops = {
> + .s_ctrl = mxc_jpeg_s_ctrl,
> +};
> +
> +static void mxc_jpeg_encode_ctrls(struct mxc_jpeg_ctx *ctx)
> +{
> + v4l2_ctrl_new_std(&ctx->ctrl_handler, &mxc_jpeg_ctrl_ops,
> + V4L2_CID_JPEG_COMPRESSION_QUALITY, 1, 100, 1, 75);
> +}
> +
> +static int mxc_jpeg_ctrls_setup(struct mxc_jpeg_ctx *ctx)
> +{
> + v4l2_ctrl_handler_init(&ctx->ctrl_handler, 2);
> +
> + if (ctx->mxc_jpeg->mode == MXC_JPEG_ENCODE)
> + mxc_jpeg_encode_ctrls(ctx);
> +
> + if (ctx->ctrl_handler.error) {
> + int err = ctx->ctrl_handler.error;
> +
> + v4l2_ctrl_handler_free(&ctx->ctrl_handler);
> + return err;
> + }
> +
> + return v4l2_ctrl_handler_setup(&ctx->ctrl_handler);
> +}
> +
> static int mxc_jpeg_open(struct file *file)
> {
> struct mxc_jpeg_dev *mxc_jpeg = video_drvdata(file);
> @@ -1594,6 +1640,12 @@ static int mxc_jpeg_open(struct file *file)
> goto error;
> }
>
> + ret = mxc_jpeg_ctrls_setup(ctx);
> + if (ret) {
> + dev_err(ctx->mxc_jpeg->dev, "failed to setup mxc jpeg controls\n");
> + goto err_ctrls_setup;
> + }
> + ctx->fh.ctrl_handler = &ctx->ctrl_handler;
> mxc_jpeg_set_default_params(ctx);
> ctx->slot = MXC_MAX_SLOTS; /* slot not allocated yet */
>
> @@ -1605,6 +1657,8 @@ static int mxc_jpeg_open(struct file *file)
>
> return 0;
>
> +err_ctrls_setup:
> + v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
> error:
> v4l2_fh_del(&ctx->fh);
> v4l2_fh_exit(&ctx->fh);
> @@ -1962,6 +2016,8 @@ static int mxc_jpeg_subscribe_event(struct v4l2_fh *fh,
> return v4l2_event_subscribe(fh, sub, 0, NULL);
> case V4L2_EVENT_SOURCE_CHANGE:
> return v4l2_src_change_event_subscribe(fh, sub);
> + case V4L2_EVENT_CTRL:
> + return v4l2_ctrl_subscribe_event(fh, sub);
> default:
> return -EINVAL;
> }
> @@ -2035,6 +2091,7 @@ static int mxc_jpeg_release(struct file *file)
> else
> dev_dbg(dev, "Release JPEG encoder instance on slot %d.",
> ctx->slot);
> + v4l2_ctrl_handler_free(&ctx->ctrl_handler);
> v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
> v4l2_fh_del(&ctx->fh);
> v4l2_fh_exit(&ctx->fh);
> diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
> index 9ae56e6e0fbe..9c9da32b2125 100644
> --- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
> +++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
> @@ -96,6 +96,8 @@ struct mxc_jpeg_ctx {
> unsigned int slot;
> unsigned int source_change;
> bool header_parsed;
> + struct v4l2_ctrl_handler ctrl_handler;
> + u8 jpeg_quality;
> };
>
> struct mxc_jpeg_slot_data {
next prev parent reply other threads:[~2022-04-22 9:56 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-22 1:53 [PATCH v4] media: imx-jpeg: Encoder support to set jpeg quality Ming Qian
2022-04-22 9:56 ` mirela.rabulea [this message]
2022-05-18 11:22 ` Hans Verkuil
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=77a60ba0-91f0-05ae-d4d5-2675783e70f7@oss.nxp.com \
--to=mirela.rabulea@oss.nxp.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=hverkuil-cisco@xs4all.nl \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=ming.qian@nxp.com \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/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).