From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3BD93C10F27 for ; Wed, 11 Mar 2020 11:03:04 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0B45821655 for ; Wed, 11 Mar 2020 11:03:04 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="OzXiZdZx" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726713AbgCKLDD (ORCPT ); Wed, 11 Mar 2020 07:03:03 -0400 Received: from mail-ed1-f66.google.com ([209.85.208.66]:33233 "EHLO mail-ed1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726000AbgCKLDD (ORCPT ); Wed, 11 Mar 2020 07:03:03 -0400 Received: by mail-ed1-f66.google.com with SMTP id z65so2374363ede.0 for ; Wed, 11 Mar 2020 04:03:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=jb6YNfpy0i4p0prPq7fXurxyCpu3UioYP/wK0SXZz40=; b=OzXiZdZxcla8lff01e5y1xu8R1or7O5akJh7vkF0kGDv+FIl4k+KI5dt7+u7bNa22f pEgi1+VeIA0u2roWjwGSIIVN7DccoliwZNCmhlmVo4+QDcI5e0XEPbQld3FfpR1P8z8I MP4yhJz3sMk7jvtXq8o0TB7XLMZod2m/M8uIQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=jb6YNfpy0i4p0prPq7fXurxyCpu3UioYP/wK0SXZz40=; b=Mg/sveJlrHJhdntk6Uii11TZqS8ycUaYnirJpO0+qhxDS1ui66f/pSddsY4Ol707o2 TwnaJPecPV91ZT8vbuM0pRVeh2woHsuGHt+LW8isqrEYjcnr/ROaC1kWnnlhyqeizHbG i2LHsO4YR/hasjr0yRJuWYWL0GI3QsVLue3sJ11RImeAOnnr8WdAwMC0wcW0u2+mNOy0 Po9+e9o8qDPKaXQoOz93HoJBBbU5MluqeSaP/8cX82jSN5YGd8JCex8Wn27I/ZJ4V9DQ jpyCmEIWYW8gPuK1qPhUfxazxP+zXVxEmj9AUmGs7a4ivFyd84pI5+VsZ+ln8F5HR/WE 3s7w== X-Gm-Message-State: ANhLgQ1X6cN1b1qG45JBrMMwLiyQdbgx/Zl7G6kFEyjhXtx6BSMEUj+T b22BPA91SPa4CZtTgfCIdeeLgsobdlmnwg== X-Google-Smtp-Source: ADFU+vuUb4OtFe/ReI+k9Zq27ZMvryrWj5IG/n7tFpBU9c8inbGINxlqvXitPWjUH/DA2Ei4l1EVQQ== X-Received: by 2002:aa7:d702:: with SMTP id t2mr2306638edq.342.1583924581676; Wed, 11 Mar 2020 04:03:01 -0700 (PDT) Received: from mail-wr1-f47.google.com (mail-wr1-f47.google.com. [209.85.221.47]) by smtp.gmail.com with ESMTPSA id t24sm4546937edy.94.2020.03.11.04.03.00 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 11 Mar 2020 04:03:01 -0700 (PDT) Received: by mail-wr1-f47.google.com with SMTP id v11so2000725wrm.9 for ; Wed, 11 Mar 2020 04:03:00 -0700 (PDT) X-Received: by 2002:adf:f545:: with SMTP id j5mr3918965wrp.295.1583924580138; Wed, 11 Mar 2020 04:03:00 -0700 (PDT) MIME-Version: 1.0 References: <20200303123446.20095-1-xia.jiang@mediatek.com> <20200303123446.20095-4-xia.jiang@mediatek.com> In-Reply-To: <20200303123446.20095-4-xia.jiang@mediatek.com> From: Tomasz Figa Date: Wed, 11 Mar 2020 20:02:48 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v7 03/11] media: platform: Improve s_selection flow for bug fixing To: Xia Jiang Cc: Hans Verkuil , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Rick Chang , Linux Media Mailing List , linux-devicetree , Linux Kernel Mailing List , "list@263.net:IOMMU DRIVERS , Joerg Roedel ," , "moderated list:ARM/Mediatek SoC support" , Marek Szyprowski , srv_heupstream Content-Type: text/plain; charset="UTF-8" Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Xia, On Tue, Mar 3, 2020 at 9:35 PM Xia Jiang 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 > --- > 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