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=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 4B055C433DB for ; Wed, 6 Jan 2021 20:21:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 187B522B40 for ; Wed, 6 Jan 2021 20:21:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728084AbhAFUUq (ORCPT ); Wed, 6 Jan 2021 15:20:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55588 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728062AbhAFUUo (ORCPT ); Wed, 6 Jan 2021 15:20:44 -0500 Received: from mail-wm1-x330.google.com (mail-wm1-x330.google.com [IPv6:2a00:1450:4864:20::330]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 482E0C06134C for ; Wed, 6 Jan 2021 12:20:03 -0800 (PST) Received: by mail-wm1-x330.google.com with SMTP id q75so3698466wme.2 for ; Wed, 06 Jan 2021 12:20:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=w3vu7Q77s3pJWhDM2ZQklqoYrnX9v2imWoObUc3E/RA=; b=ctIAJZXn1n8n6ErtdFx+i0m//hYktYCyTIp+fkL2LcY4Jfm0wKjplw4EPnBOWExPAF tm1apLDsKQTMmoQJulDnTwruR9Hsv/yMpbPSwvB33Dn0xO8hHEGo06j0Mq+sinPqwyZs kWZVEpQfTT0gtFMj0arQ6YCHOg8raZObUVbJLy3IpkYwflXc322j700Bc561ZRWT68Qr WImc4bVeS/KdcVxYAC3M6Lh5048xfOZ+mJi3bkLkEPN0k184n8amf7w7eb1X+DXPr41W BYDH63AMZUvCTtDjclrX3JOjBQH+bUaDAXZVik31j8KCdfx49K2tkomwviBCLVObiuL0 DAOg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=w3vu7Q77s3pJWhDM2ZQklqoYrnX9v2imWoObUc3E/RA=; b=JsVm6xw4sYS2n09XX3ZLpx/cb+rdM2Qcn0l5Nm3YVxoI4aeO47Q/Kp4318nf1Cdfrp UdGB3uRkbEQyvKVjD39wwnbIE1P7FtRLkWxanZFsdVtRskdrsb0bj957J1mxSh6No258 pAQS+RAPddVVOiVF7/EvyExjVE49puYt1uNnshpNm8M8UU2DA+9M/9LIhLIsDvYK2nuY 5e0YoR5mv9zxEjY/A4l11HeeN6rw0PYAsmOv7kDAbWEEMfjAfmb7va4WblD/nvgjE38N cys3Y5XTJLrvzG+VyCYe67hlBCb5bVrM7OdZn4ocSYZqkYedvR+qN7hGi+MjLX+x+Ezg Ip2A== X-Gm-Message-State: AOAM533hFQto4Amfy4emcLP87y/6XGYVCDLsxOIcW7bIq7QOyBbPiMBH YPSzQFqPioo9t8+3bsz/DNYXvA== X-Google-Smtp-Source: ABdhPJyBV1GURnIjVGPvwWRfM4Ufh3xwem8TBRk7I5b9Hoc2Gd8ZantU2ae2Sew+89YLsf+t89oB0A== X-Received: by 2002:a7b:c1d7:: with SMTP id a23mr5101496wmj.62.1609964401954; Wed, 06 Jan 2021 12:20:01 -0800 (PST) Received: from Red ([2a01:cb1d:3d5:a100:264b:feff:fe03:2806]) by smtp.googlemail.com with ESMTPSA id n16sm4435939wrj.26.2021.01.06.12.20.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Jan 2021 12:20:01 -0800 (PST) Date: Wed, 6 Jan 2021 21:19:58 +0100 From: LABBE Corentin To: Dan Carpenter Cc: Zheng Yongjun , mchehab@kernel.org, mjpeg-users@lists.sourceforge.net, linux-media@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org Subject: Re: [PATCH -next] media: zoran: use resource_size Message-ID: References: <20210106131702.32507-1-zhengyongjun3@huawei.com> <20210106145100.GJ2809@kadam> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20210106145100.GJ2809@kadam> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le Wed, Jan 06, 2021 at 05:51:00PM +0300, Dan Carpenter a écrit : > On Wed, Jan 06, 2021 at 09:17:02PM +0800, Zheng Yongjun wrote: > > Use resource_size rather than a verbose computation on > > the end and start fields. > > > > Signed-off-by: Zheng Yongjun > > --- > > drivers/staging/media/zoran/zoran_driver.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/staging/media/zoran/zoran_driver.c b/drivers/staging/media/zoran/zoran_driver.c > > index 808196ea5b81..d60b4c73ea80 100644 > > --- a/drivers/staging/media/zoran/zoran_driver.c > > +++ b/drivers/staging/media/zoran/zoran_driver.c > > @@ -1020,7 +1020,7 @@ int zoran_queue_init(struct zoran *zr, struct vb2_queue *vq) > > vq->buf_struct_size = sizeof(struct zr_buffer); > > vq->ops = &zr_video_qops; > > vq->mem_ops = &vb2_dma_contig_memops; > > - vq->gfp_flags = GFP_DMA32, > > + vq->gfp_flags = GFP_DMA32; > > The commit doesn't match the patch. Also this driver is in > staging because it's going to be deleted soon so there probably isn't > much point doing cleanups. > No, the driver just came back in staging since I fixed the videobuf2 conversion. One of the reason it is kept in staging is that media maintainer want to test it with its own zoran card but covid19 delayed the physical recovery of it. So the patch need to be resent, please. Regards