2020-03-03 13:04:27

by Xia Jiang

Subject: [PATCH v7 03/11] media: platform: Improve s_selection flow for bug fixing

Get correct compose value in mtk_jpeg_s_selection function.

Signed-off-by: Xia Jiang <[email protected]>
---
drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c b/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
index da0dae4b0fc9..fb2c8d026580 100644
--- a/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
+++ b/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
@@ -492,8 +492,8 @@ static int mtk_jpeg_s_selection(struct file *file, void *priv,
case V4L2_SEL_TGT_COMPOSE:
s->r.left = 0;
s->r.top = 0;
- s->r.width = ctx->out_q.w;
- s->r.height = ctx->out_q.h;
+ ctx->out_q.w = s->r.width;
+ ctx->out_q.h = s->r.height;
break;
default:
return -EINVAL;
--
2.18.0

2020-03-11 11:06:03

by Tomasz Figa

Subject: Re: [PATCH v7 03/11] media: platform: Improve s_selection flow for bug fixing

Hi Xia,

On Tue, Mar 3, 2020 at 9:35 PM Xia Jiang <[email protected]> wrote:
>
> Get correct compose value in mtk_jpeg_s_selection function.

It's a good practice to describe why the current code is wrong and new is good.

>
> Signed-off-by: Xia Jiang <[email protected]>
> ---
> drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>

Thanks for the patch. Please see my comment inline.

> diff --git a/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c b/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
> index da0dae4b0fc9..fb2c8d026580 100644
> --- a/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
> +++ b/drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
> @@ -492,8 +492,8 @@ static int mtk_jpeg_s_selection(struct file *file, void *priv,
> case V4L2_SEL_TGT_COMPOSE:
> s->r.left = 0;
> s->r.top = 0;
> - s->r.width = ctx->out_q.w;
> - s->r.height = ctx->out_q.h;
> + ctx->out_q.w = s->r.width;
> + ctx->out_q.h = s->r.height;
> break;
> default:
> return -EINVAL;

The decoder compose target means the visible rectangle of the decoded
image, which comes from the stream metadata. It's not something that
can be set by the userspace.

Best regards,
Tomasz