[{"id":11706,"web_url":"https://patchwork.libcamera.org/comment/11706/","msgid":"<20200729114412.GE322953@oden.dyn.berto.se>","date":"2020-07-29T11:44:12","subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","submitter":{"id":5,"url":"https://patchwork.libcamera.org/api/people/5/","name":"Niklas Söderlund","email":"niklas.soderlund@ragnatech.se"},"content":"Hi Kieran,\n\nThanks for your work.\n\nOn 2020-07-29 12:25:24 +0100, Kieran Bingham wrote:\n> Determine the number of planes used by a format by counting the number\n> of PxielFormatPlaneInfo entries with a valid entry.\n> \n> Signed-off-by: Kieran Bingham \n\nReviewed-by: Niklas Söderlund \n\n> ---\n> include/libcamera/internal/formats.h | 2 ++\n> src/libcamera/formats.cpp | 22 ++++++++++++++++++++++\n> 2 files changed, 24 insertions(+)\n> \n> diff --git a/include/libcamera/internal/formats.h b/include/libcamera/internal/formats.h\n> index 0bb151044294..beeaab1ae2f5 100644\n> --- a/include/libcamera/internal/formats.h\n> +++ b/include/libcamera/internal/formats.h\n> @@ -45,6 +45,8 @@ public:\n> \tunsigned int frameSize(const Size &size,\n> \t\t\t const std::array &strides) const;\n> \n> +\tunsigned int planeCount() const;\n> +\n> \t/* \\todo Add support for non-contiguous memory planes */\n> \tconst char *name;\n> \tPixelFormat format;\n> diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp\n> index cd63c15cb926..70d31fb36c37 100644\n> --- a/src/libcamera/formats.cpp\n> +++ b/src/libcamera/formats.cpp\n> @@ -805,4 +805,26 @@ PixelFormatInfo::frameSize(const Size &size,\n> \treturn sum;\n> }\n> \n> +/**\n> + * \\brief Identify the number of planes represented by the format\n> + *\n> + * This function counts the number of planes which have a valid configuration\n> + * and uses that to determine the number of planes used by the format.\n> + *\n> + * \\return The number of planes used by the format\n> + */\n> +unsigned int PixelFormatInfo::planeCount() const\n> +{\n> +\tunsigned int count = 0;\n> +\n> +\tfor (const PixelFormatPlaneInfo &p : planes) {\n> +\t\tif (p.bytesPerGroup == 0)\n> +\t\t\tbreak;\n> +\n> +\t\tcount++;\n> +\t}\n> +\n> +\treturn count;\n> +}\n> +\n> } /* namespace libcamera */\n> -- \n> 2.25.1\n> \n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 4CF61BD878\n\tfor ;\n\tWed, 29 Jul 2020 11:44:16 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id DD1D4613C6;\n\tWed, 29 Jul 2020 13:44:15 +0200 (CEST)","from mail-lj1-x243.google.com (mail-lj1-x243.google.com\n\t[IPv6:2a00:1450:4864:20::243])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8F0A76039D\n\tfor ;\n\tWed, 29 Jul 2020 13:44:14 +0200 (CEST)","by mail-lj1-x243.google.com with SMTP id t6so11739364ljk.9\n\tfor ;\n\tWed, 29 Jul 2020 04:44:14 -0700 (PDT)","from localhost (h-209-203.A463.priv.bahnhof.se. [155.4.209.203])\n\tby smtp.gmail.com with ESMTPSA id\n\tw29sm404056lfq.65.2020.07.29.04.44.13\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tWed, 29 Jul 2020 04:44:13 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=ragnatech-se.20150623.gappssmtp.com\n\theader.i=@ragnatech-se.20150623.gappssmtp.com\n\theader.b=\"oqjO6Q+H\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ragnatech-se.20150623.gappssmtp.com; s=20150623;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:content-transfer-encoding:in-reply-to;\n\tbh=ntsHpxGGF8eC1Z4oyaxJ3OUJhG8nnzZoRzKq+aLb6Tc=;\n\tb=oqjO6Q+HVdx/+MXBz5p4oHSQTd/zcUSQmuj50tqygCpCrZNvOWDmE61p4COR36eV7S\n\tRxWI/QsEAttDO1Jx8ct8n0uE8RrRaJ0Sukx8416iPmfgYpmGeihL14yXjxks5T+nLC/V\n\tJQ+KtukMo6lHUOINbxxvVXjFnsnlVKNvIdQO77PWQPrNt+wSGZco7t9wksbFWU1Mt8L5\n\tiUIP+5lWovlAWaFswPOB8UsoD/8gCn3w8z/UFe1+n+RI6fFBh+yUC6HrwmR4cV4FaFhm\n\t4RUZWzrOVmfNvyZS/olSCrMNJr/kWIzLLGzPmgRK6wEC0rNFdKwYrSzNUiaIuxzI1Hjl\n\t2aAg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:content-transfer-encoding\n\t:in-reply-to;\n\tbh=ntsHpxGGF8eC1Z4oyaxJ3OUJhG8nnzZoRzKq+aLb6Tc=;\n\tb=WUbGjfkUS3riSVkzs/kbl0u227mKygys9uJ7Nsn0IxBi2Xys6mfiHvwZAtoGfjKXMT\n\tns3wUr8X/WsldfrO8ZVbsWU4lKLtf3jvR9AAAqso585DFD2hsWAfWSz82JFC4tRDnyuc\n\tYgn6WtOjS4+yHhfi4phH+OPy8kr78rYkXZwf9QwW77iXCgGP2fjbkA/E7kBkxy5VXBmB\n\tOe8n7rIHwHu1gejkp4aHSVfnd0Mr46wmV6g40dgJAZPpJdF9W8bI/T8THF3W2VoI6Kur\n\txms3ywXkSSmbEmi/jE3omSyv1BLucD4F2/csplh3HKSc4VFZTgTfkL+MZt8n9e5FVX+t\n\tZM+Q==","X-Gm-Message-State":"AOAM53021F9k8C5M1MdsEMYOUB9r/JJXssGCuchd2c6VN3t1Kl3iHmZp\n\t5JCtZnQDEIdRzWFmu1Y9xSz7vPDbEC4=","X-Google-Smtp-Source":"ABdhPJyA8vGQJPavXxnNArHxuUDkPsddJLmjDIazcRF0K0Me+Z+ddJvXqlt6xyYdAGiOAS+sDfQGWQ==","X-Received":"by 2002:a05:651c:1105:: with SMTP id\n\td5mr15723426ljo.62.1596023053942; \n\tWed, 29 Jul 2020 04:44:13 -0700 (PDT)","Date":"Wed, 29 Jul 2020 13:44:12 +0200","From":"Niklas =?iso-8859-1?q?S=F6derlund?= ","To":"Kieran Bingham ","Message-ID":"<20200729114412.GE322953@oden.dyn.berto.se>","References":"<20200729112524.498276-1-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20200729112524.498276-1-kieran.bingham@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"iso-8859-1\"","Content-Transfer-Encoding":"quoted-printable","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":11709,"web_url":"https://patchwork.libcamera.org/comment/11709/","msgid":"<20200729133628.GA6183@pendragon.ideasonboard.com>","date":"2020-07-29T13:36:28","subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nThank you for the patch.\n\nOn Wed, Jul 29, 2020 at 12:25:24PM +0100, Kieran Bingham wrote:\n> Determine the number of planes used by a format by counting the number\n> of PxielFormatPlaneInfo entries with a valid entry.\n> \n> Signed-off-by: Kieran Bingham \n> ---\n> include/libcamera/internal/formats.h | 2 ++\n> src/libcamera/formats.cpp | 22 ++++++++++++++++++++++\n> 2 files changed, 24 insertions(+)\n> \n> diff --git a/include/libcamera/internal/formats.h b/include/libcamera/internal/formats.h\n> index 0bb151044294..beeaab1ae2f5 100644\n> --- a/include/libcamera/internal/formats.h\n> +++ b/include/libcamera/internal/formats.h\n> @@ -45,6 +45,8 @@ public:\n> \tunsigned int frameSize(const Size &size,\n> \t\t\t const std::array &strides) const;\n> \n> +\tunsigned int planeCount() const;\n> +\n> \t/* \\todo Add support for non-contiguous memory planes */\n> \tconst char *name;\n> \tPixelFormat format;\n> diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp\n> index cd63c15cb926..70d31fb36c37 100644\n> --- a/src/libcamera/formats.cpp\n> +++ b/src/libcamera/formats.cpp\n> @@ -805,4 +805,26 @@ PixelFormatInfo::frameSize(const Size &size,\n> \treturn sum;\n> }\n> \n> +/**\n> + * \\brief Identify the number of planes represented by the format\n> + *\n> + * This function counts the number of planes which have a valid configuration\n> + * and uses that to determine the number of planes used by the format.\n\nI think this is just an internal implementation detail.\n\n * \\brief Retrieve the number of planes used by the format\n * \\return The number of planes used by the format\n\nshould be enough.\n\nAnd if we hide the fact that we count them, maybe\ns/planeCount/numPlanes/ ? I think that's a bit more consistent with how\nwe name fields (haven't checked though).\n\n> + *\n> + * \\return The number of planes used by the format\n> + */\n> +unsigned int PixelFormatInfo::planeCount() const\n> +{\n> +\tunsigned int count = 0;\n> +\n> +\tfor (const PixelFormatPlaneInfo &p : planes) {\n> +\t\tif (p.bytesPerGroup == 0)\n> +\t\t\tbreak;\n> +\n> +\t\tcount++;\n> +\t}\n> +\n> +\treturn count;\n\n\treturn std::count_if(planes.begin(), planes.end(),\n\t\t\t [](const PixelFormatPlaneInfo &p) {\n\t\t\t\t return p.bytesPerGroup != 0;\n\t\t\t });\n\nwould also work, entirely up to you.\n\nReviewed-by: Laurent Pinchart \n\n> +}\n> +\n> } /* namespace libcamera */","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 32C44BD86F\n\tfor ;\n\tWed, 29 Jul 2020 13:36:39 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 96A0A613C6;\n\tWed, 29 Jul 2020 15:36:38 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9850C6039D\n\tfor ;\n\tWed, 29 Jul 2020 15:36:37 +0200 (CEST)","from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi\n\t[81.175.216.236])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 0BF3631F;\n\tWed, 29 Jul 2020 15:36:36 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"kjBDwIAI\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1596029797;\n\tbh=Grw7RHhvnheP/52ztzf1ZLw8ooIHCdIWzE5Kk/QiVgk=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=kjBDwIAIl4fX85meeOrYn52DN1EdZ8FkJPaZfI4v/jx24/CdQVPAljNSdX3BuPhAA\n\tXvTu/F53jmHwijAv//r/Bq+bVTbWDynq/PE5rENe5P4n7HmHyt86fon7clNSjKbV1z\n\tVesa51W27kHR5Qkjv8avOYNClTXZdOB0iF8r9c9Y=","Date":"Wed, 29 Jul 2020 16:36:28 +0300","From":"Laurent Pinchart ","To":"Kieran Bingham ","Message-ID":"<20200729133628.GA6183@pendragon.ideasonboard.com>","References":"<20200729112524.498276-1-kieran.bingham@ideasonboard.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20200729112524.498276-1-kieran.bingham@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":11710,"web_url":"https://patchwork.libcamera.org/comment/11710/","msgid":"","date":"2020-07-29T13:50:54","subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Laurent,\n\nOn 29/07/2020 14:36, Laurent Pinchart wrote:\n> Hi Kieran,\n> \n> Thank you for the patch.\n> \n> On Wed, Jul 29, 2020 at 12:25:24PM +0100, Kieran Bingham wrote:\n>> Determine the number of planes used by a format by counting the number\n>> of PxielFormatPlaneInfo entries with a valid entry.\n>>\n>> Signed-off-by: Kieran Bingham \n>> ---\n>> include/libcamera/internal/formats.h | 2 ++\n>> src/libcamera/formats.cpp | 22 ++++++++++++++++++++++\n>> 2 files changed, 24 insertions(+)\n>>\n>> diff --git a/include/libcamera/internal/formats.h b/include/libcamera/internal/formats.h\n>> index 0bb151044294..beeaab1ae2f5 100644\n>> --- a/include/libcamera/internal/formats.h\n>> +++ b/include/libcamera/internal/formats.h\n>> @@ -45,6 +45,8 @@ public:\n>> \tunsigned int frameSize(const Size &size,\n>> \t\t\t const std::array &strides) const;\n>> \n>> +\tunsigned int planeCount() const;\n>> +\n>> \t/* \\todo Add support for non-contiguous memory planes */\n>> \tconst char *name;\n>> \tPixelFormat format;\n>> diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp\n>> index cd63c15cb926..70d31fb36c37 100644\n>> --- a/src/libcamera/formats.cpp\n>> +++ b/src/libcamera/formats.cpp\n>> @@ -805,4 +805,26 @@ PixelFormatInfo::frameSize(const Size &size,\n>> \treturn sum;\n>> }\n>> \n>> +/**\n>> + * \\brief Identify the number of planes represented by the format\n>> + *\n>> + * This function counts the number of planes which have a valid configuration\n>> + * and uses that to determine the number of planes used by the format.\n> \n> I think this is just an internal implementation detail.\n> \n> * \\brief Retrieve the number of planes used by the format\n> * \\return The number of planes used by the format\n> \n> should be enough.\n\nSure, shorter is sweeter ...\n\n> \n> And if we hide the fact that we count them, maybe\n> s/planeCount/numPlanes/ ? I think that's a bit more consistent with how\n> we name fields (haven't checked though).\n\nnumPlanes is indeed better, and doesn't define how it's implemented.\n\n\n>> + *\n>> + * \\return The number of planes used by the format\n>> + */\n>> +unsigned int PixelFormatInfo::planeCount() const\n>> +{\n>> +\tunsigned int count = 0;\n>> +\n>> +\tfor (const PixelFormatPlaneInfo &p : planes) {\n>> +\t\tif (p.bytesPerGroup == 0)\n>> +\t\t\tbreak;\n>> +\n>> +\t\tcount++;\n>> +\t}\n>> +\n>> +\treturn count;\n> \n> \treturn std::count_if(planes.begin(), planes.end(),\n> \t\t\t [](const PixelFormatPlaneInfo &p) {\n> \t\t\t\t return p.bytesPerGroup != 0;\n> \t\t\t });\n> \n> would also work, entirely up to you.\n> \n\nI find the lambda's are often less readable. Yes, more C++'ified. but\n... not very friendly at all on the eye.\n\nI assume there would be no performance gain or improvement, so I'd\nprefer to keep the visible count ...\n\nMaybe I'll change my mind in the future ...\n\n> Reviewed-by: Laurent Pinchart \n\nThanks,\n\n\n\n>> +}\n>> +\n>> } /* namespace libcamera */\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id C4B0DBD878\n\tfor ;\n\tWed, 29 Jul 2020 13:51:02 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 53E086114F;\n\tWed, 29 Jul 2020 15:51:02 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 79BC86039D\n\tfor ;\n\tWed, 29 Jul 2020 15:51:00 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 9C7AB31F;\n\tWed, 29 Jul 2020 15:50:57 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"QlIREslW\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1596030657;\n\tbh=fxGGvjBRmwFC68hfYDQSEBwyo46ZRYY2eEkJNM4D7Wg=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=QlIREslW8q0Auh4eZelgRjNzUC3VmHczc7lGi/kFUQ51KqVJBSdxTKC85TJ27xlKA\n\tPe63RvQYM5QpHY/Nucixba19Odfyu0c2b8PtuqT4+mh1KDVe/fMFSL64sPXHs0o0xd\n\tyabA1T95Adfk20wY4oQKESLJNm2gG2/Kx3mm42m4=","To":"Laurent Pinchart ","References":"<20200729112524.498276-1-kieran.bingham@ideasonboard.com>\n\t<20200729133628.GA6183@pendragon.ideasonboard.com>","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"","Date":"Wed, 29 Jul 2020 14:50:54 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.10.0","MIME-Version":"1.0","In-Reply-To":"<20200729133628.GA6183@pendragon.ideasonboard.com>","Content-Language":"en-GB","Subject":"Re: [libcamera-devel] [PATCH] libcamera: formats: add planeCount\n\thelper","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Reply-To":"kieran.bingham@ideasonboard.com","Cc":"libcamera devel ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]