Received: by 2002:a05:6358:11c7:b0:104:8066:f915 with SMTP id i7csp1491415rwl; Fri, 31 Mar 2023 11:59:29 -0700 (PDT) X-Google-Smtp-Source: AKy350aV7JyAiMt9++qBAO8nPx2e3TY3Bw8K/w/+63nJxVV7VRR0D2ss6mEGwxd+Ndm3Pl5cmzBE X-Received: by 2002:a05:6a20:440c:b0:d7:19b3:a9bd with SMTP id ce12-20020a056a20440c00b000d719b3a9bdmr8470473pzb.4.1680289169004; Fri, 31 Mar 2023 11:59:29 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680289168; cv=none; d=google.com; s=arc-20160816; b=h7+ryV/HQcWGSwhD5NjBmMH0shEDDPV35MYYzJnmpn6SrIJC6djh1fHw/L6YSNIRev WwRifMPBMO6QJB1LgQ8RiHXXdqrmRsTVfP3+U7pu/lPOpRWVATF/ueKoeBoJmb8Q7if3 ZQ/3/KHFTfR1/hrPp8unf88F0DVzuFLVxdTfuX/dBRTaK6/tTdzoQn1izjgE6IImJ5bl gNGNpPIYWntN/dfe26VQQpZqs1KfJkJ/241I0Tnmn90YQ/Kcq70QXOCYCY/3/czDXoZD EEosl5UqJpdoj1xyLNbJ0wcfEO7vk4+ggweFM8SsZMkhILYJcfuiEqzQA3q6wYgOpqky 0xtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id:dkim-signature; bh=uAm5w/BgV+ualcvdqUfpIBSmMWRJ7Ixo/8KUSDUf6wY=; b=JuVSQ4f3SiiZCQ27jfU8jQznojwFvuI05HGgzOaWn+rNoSVyJ7newCRl33F+rE7x0b HZjB14NsyzP4EcjXGbVh2kZGotheoYKAXXG7CWySTvjsINky9/YzS63NYIW4NPm9hbmr PbOLCyCyMPHDMeufESPrA/Hy5cOxAp5+y+YB3ehTzbuvnx5ElMhVV2nz+XGzqkHdIFKJ fOE9gbF+rZcdIOy1wIaMqvayLDKHCWF1xyueN0/zr6NYTSDH4Hh4vbUw2g8Ai2k2/jgH tfBec5VWNFLKJPqV5XEBKz6sPKtzW6lSlrQWp4wmfNq46w7mGCT11CVB1alAZKn44R4p qLLg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ndufresne-ca.20210112.gappssmtp.com header.s=20210112 header.b=Khz9WpXF; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 17-20020a630c51000000b0050bcfe66805si3046854pgm.240.2023.03.31.11.59.16; Fri, 31 Mar 2023 11:59:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@ndufresne-ca.20210112.gappssmtp.com header.s=20210112 header.b=Khz9WpXF; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233170AbjCaSyY (ORCPT + 99 others); Fri, 31 Mar 2023 14:54:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57350 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229983AbjCaSyX (ORCPT ); Fri, 31 Mar 2023 14:54:23 -0400 Received: from mail-qt1-x829.google.com (mail-qt1-x829.google.com [IPv6:2607:f8b0:4864:20::829]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8240B22202 for ; Fri, 31 Mar 2023 11:54:22 -0700 (PDT) Received: by mail-qt1-x829.google.com with SMTP id cr18so18773041qtb.0 for ; Fri, 31 Mar 2023 11:54:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ndufresne-ca.20210112.gappssmtp.com; s=20210112; t=1680288861; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=uAm5w/BgV+ualcvdqUfpIBSmMWRJ7Ixo/8KUSDUf6wY=; b=Khz9WpXFZoEp7QKh2GVHbGIFvMum3FoxXwERbrXyJLQdIeQl/rFcxEllenep1Zq8wM 9CkHSaG64XbMyzbOaffevp1JK1clvO7fpUcQVAltBAPJLYORdjiXklRRvxT3/0HAt0C/ juuUW0Ce9a2kgk+SFjCthUMnlJm4sqyk/uZq1/+HOIrSTM6OwgyMw8Pn4WSsnV/pkYS4 fzF8fASURhgdpIQvT8wKwBxvoNkt10co61XwVTX2NwMy6Bb1dS+VdFkJsTsh8CPFiFnG J4JNXue1mN8XWPNNGtREwNCFN6ihnuippPVT3B8Z7hiHvh2imFwDArJw4kgl4ZY8McV1 AJsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680288861; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=uAm5w/BgV+ualcvdqUfpIBSmMWRJ7Ixo/8KUSDUf6wY=; b=G8ymCGcnv6mybdDSMrPkabe0MJmeTXx/aa3/teX2SbqnGJk9FD4En1slw2uTUU7K4R hUQqsRkWRN0XSAFkzAu+or9w2djiclM2uI02yQNXns7PiItjD7XEE4A2uWvTi33tG1yd MgO1dehkx+Znisx0bTIuwW1YG583tCT9VkeEBhhspbdvxpJFZGYcvTmz7z+zuD6cIoKO iBGqJ+W8qAVqRbh/MfY0JNv5nYMkdL1Pn7L7VhYp/VnWcIGnXDflcTPJIXmoPwsxsu2r 9NmG0Lvc5kBLVfS9uZR4yKSRcmmsJxTqqpLGNSM6yH/MyWMmMBUSTOrTtjHk6copa8lj tmRQ== X-Gm-Message-State: AO0yUKXb6CvowIZIiCQOJIfZWiRsY+SDjr5QUoii5o2ibvITsn/p0Rda k4c5dMVo6Glppc6t8Iujjdo+hg== X-Received: by 2002:a05:622a:184:b0:3bf:e13e:438f with SMTP id s4-20020a05622a018400b003bfe13e438fmr49210416qtw.36.1680288861692; Fri, 31 Mar 2023 11:54:21 -0700 (PDT) Received: from nicolas-tpx395.localdomain ([2606:6d00:15:a07e::580]) by smtp.gmail.com with ESMTPSA id c7-20020ac84e07000000b003e3921077d9sm799278qtw.38.2023.03.31.11.54.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 31 Mar 2023 11:54:21 -0700 (PDT) Message-ID: <40774dc0ef32246af76a202caca3632abc1ae25f.camel@ndufresne.ca> Subject: Re: [PATCH 2/9] media: v4l2: Add NV12_16L16 pixel format to v4l2 format info From: Nicolas Dufresne To: Laurent Pinchart , Paul Kocialkowski Cc: linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev, Mauro Carvalho Chehab , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Adam Pigg , Thomas Petazzoni Date: Fri, 31 Mar 2023 14:54:20 -0400 In-Reply-To: <20230325210120.GB22214@pendragon.ideasonboard.com> References: <20230324151228.2778112-1-paul.kocialkowski@bootlin.com> <20230324151228.2778112-3-paul.kocialkowski@bootlin.com> <20230325210120.GB22214@pendragon.ideasonboard.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.46.4 (3.46.4-1.fc37) MIME-Version: 1.0 X-Spam-Status: No, score=0.0 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le samedi 25 mars 2023 =C3=A0 23:01 +0200, Laurent Pinchart a =C3=A9crit=C2= =A0: > Hi Paul, >=20 > Thank you for the patch. >=20 > On Fri, Mar 24, 2023 at 04:12:21PM +0100, Paul Kocialkowski wrote: > > Represent the NV12_16L16 pixel format in the v4l2 format info table. > > This is a 16x16 tiled version of NV12. > >=20 > > Signed-off-by: Paul Kocialkowski > > --- > > drivers/media/v4l2-core/v4l2-common.c | 2 ++ > > 1 file changed, 2 insertions(+) > >=20 > > diff --git a/drivers/media/v4l2-core/v4l2-common.c b/drivers/media/v4l2= -core/v4l2-common.c > > index 3d044b31caad..5101989716aa 100644 > > --- a/drivers/media/v4l2-core/v4l2-common.c > > +++ b/drivers/media/v4l2-core/v4l2-common.c > > @@ -280,6 +280,8 @@ const struct v4l2_format_info *v4l2_format_info(u32= format) > > /* Tiled YUV formats */ > > { .format =3D V4L2_PIX_FMT_NV12_4L4, .pixel_enc =3D V4L2_PIXEL_ENC_Y= UV, .mem_planes =3D 1, .comp_planes =3D 2, .bpp =3D { 1, 2, 0, 0 }, .hdiv = =3D 2, .vdiv =3D 2 }, > > { .format =3D V4L2_PIX_FMT_P010_4L4, .pixel_enc =3D V4L2_PIXEL_ENC_Y= UV, .mem_planes =3D 1, .comp_planes =3D 2, .bpp =3D { 2, 4, 0, 0 }, .hdiv = =3D 2, .vdiv =3D 2 }, > > + { .format =3D V4L2_PIX_FMT_NV12_16L16, .pixel_enc =3D V4L2_PIXEL_ENC= _YUV, .mem_planes =3D 1, .comp_planes =3D 2, .bpp =3D { 1, 2, 0, 0 }, .hdiv= =3D 2, .vdiv =3D 2, > > + .block_w =3D { 16, 16, 0, 0 }, .block_h =3D { 16, 16, 0, 0 } }, >=20 > Not necessarily related to this patch, but I'm a bit puzzled by why > V4L2_PIX_FMT_NV12_4L4 doesn't list block sizes. It looks like Ezequiel introduced that initially, but didn't introduce any = tiled format, as a side effect, we missed it and no one ever used it. In practice, its not dramatic, since most of the time, the alignment needed= is bigger then the block (specially with only 4x4 tiles), but we should certai= nly fix that, thanks for spotting. >=20 > > =20 > > /* YUV planar formats, non contiguous variant */ > > { .format =3D V4L2_PIX_FMT_YUV420M, .pixel_enc =3D V4L2_PIXEL_ENC_YU= V, .mem_planes =3D 3, .comp_planes =3D 3, .bpp =3D { 1, 1, 1, 0 }, .hdiv = =3D 2, .vdiv =3D 2 }, >=20