[{"id":19608,"web_url":"https://patchwork.libcamera.org/comment/19608/","msgid":"","date":"2021-09-10T15:18:33","subject":"Re: [libcamera-devel] [PATCH v6 2/5] ipa: ipu3: Rename\n\tIspStatsRegion to Accumulator","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Jean-Michel,\n\nThank you for the patch.\n\nOn Thu, Sep 09, 2021 at 03:54:46PM +0200, Jean-Michel Hautbois wrote:\n> The IspStatsRegion structure was introduced as an attempt to prepare for\n> a generic AWB algorithm structure. The structure name by itself is not\n> explicit and it is too optimistic to try and make a generic one for now.\n> \n> Its role is to accumulate the pixels in a given zone. Rename it to\n> accumulator, and remove the uncounted field at the same time. It is\n> always possible to know how many pixels are not relevant for the\n> algorithm by calculating total-counted. The uncounted field was only\n> declared and not used. Amend the documentation accordingly.\n> \n> Signed-off-by: Jean-Michel Hautbois \n> Reviewed-by: Kieran Bingham \n> ---\n> src/ipa/ipu3/algorithms/awb.cpp | 79 +++++++++++++++++++++++++--------\n> src/ipa/ipu3/algorithms/awb.h | 5 +--\n> 2 files changed, 63 insertions(+), 21 deletions(-)\n> \n> diff --git a/src/ipa/ipu3/algorithms/awb.cpp b/src/ipa/ipu3/algorithms/awb.cpp\n> index e05647c9..9b22c783 100644\n> --- a/src/ipa/ipu3/algorithms/awb.cpp\n> +++ b/src/ipa/ipu3/algorithms/awb.cpp\n> @@ -21,26 +21,70 @@ static constexpr uint32_t kMinZonesCounted = 16;\n> static constexpr uint32_t kMinGreenLevelInZone = 32;\n> \n> /**\n> - * \\struct IspStatsRegion\n> - * \\brief RGB statistics for a given region\n> + * \\struct Accumulator\n> + * \\brief RGB statistics for a given zone\n> *\n> - * The IspStatsRegion structure is intended to abstract the ISP specific\n> - * statistics and use an agnostic algorithm to compute AWB.\n> + * - Cells are defined in Pixels\n> + * - Zones are defined in Cells\n> + * - The total zones may overlap the image width and height to meet hardware\n> + * constraints\n\nWhat do you mean by \"overlap the image width\" here ?\n\n> *\n> - * \\var IspStatsRegion::counted\n> - * \\brief Number of pixels used to calculate the sums\n> + * 81 cells\n> + * /───────────── 1280 pixels ───────────\\\n> + * 16 zones\n> + * 16\n> + * ┌────┬────┬────┬────┬────┬─ ──────┬────┐ \\\n> + * │Cell│ │ │ │ │ | │ │ │\n> + *16 │ px │ │ │ │ │ | │ │ │\n> + * ├────┼────┼────┼────┼────┼─ ──────┼────┤ │\n> + * │ │ │ │ │ │ | │ │\n> + * │ │ │ │ │ │ | │ │ 7\n> + * │ ── │ ── │ ── │ ── │ ── │ ── ── ─┤ ── │ 1 2 4\n> + * │ │ │ │ │ │ | │ │ 2 0 5\n> *\n> - * \\var IspStatsRegion::uncounted\n> - * \\brief Remaining number of pixels in the region\n> + * │ │ │ │ │ │ | │ │ z p c\n> + * ├────┼────┼────┼────┼────┼─ ──────┼────┤ o i e\n> + * │ │ │ │ │ │ | │ │ n x l\n> + * │ │ | │ │ e e l\n> + * ├─── ───┼─ ──────┼────┤ s l s\n> + * │ │ | │ │ s\n> + * │ │ | │ │\n> + * ├─── Zone of Cells ───┼─ ──────┼────┤ │\n> + * │ (5 x 4) │ | │ │ │\n> + * │ │ | │ │ │\n> + * ├── ───┼─ ──────┼────┤ │\n> + * │ │ │ | │ │ │\n> + * │ │ │ │ │ │ | │ │ │\n> + * └────┴────┴────┴────┴────┴─ ──────┴────┘ /\n\nNice diagram :-) I wonder how it will render with doxygen, but that's\nfor later.\n\nCould you add one more space on the left side to have a space between\n\"*\" and \"16\" ?\n\n> *\n> - * \\var IspStatsRegion::rSum\n> - * \\brief Sum of the red values in the region\n> + * The algorithm works with a fixed number of zones \\a kAwbStatsSizeX x\n> + * \\a kAwbStatsSizeY. For example, a frame of 1280x720 is divided into 81x45\n> + * cells of [16x16] pixels with a BDS output size of 1296x720 to account for the\n> + * 16x16 pixel alignment restrictions. In the case of \\a kAwbStatsSizeX=16 and\n> + * \\a kAwbStatsSizeY=12 the zones are made of [5x4] cells. The cells are\n> + * left-aligned and calculated by IPAIPU3::calculateBdsGrid().\n> *\n> - * \\var IspStatsRegion::gSum\n> - * \\brief Sum of the green values in the region\n> + * Each statistics cell represents the average value of the pixels in that cell\n> + * split by colour components.\n\nThis should probably be moved somewhere else, it explains how the\nAccumulator structure is used in the IPU3 IPA implementation, not what\nthe Accumulator is. We can deal with that later (but not too late if\npossible, maybe at the time the structure moves out of awb.h as it\nshouldn't refer to kAwbStatsSizeX in that case).\n\n> *\n> - * \\var IspStatsRegion::bSum\n> - * \\brief Sum of the blue values in the region\n> + * The Accumulator structure stores the sum of the average of each cell in a\n> + * zone of the image, as well as the number of cells which were unsaturated and\n> + * therefore included in the average.\n> + *\n> + * Cells which are saturated beyond the threshold defined in\n> + * ipu3_uapi_awb_config_s are not included in the average.\n\nThis will need to be reworked too when we'll move the structure to a\nshared header, to avoid referring to ipu3_uapi_awb_config_s and just\ntalk about thresholds in general (the reference to\nipu3_uapi_awb_config_s should then be kept in awb.cpp though, as it's\nrelevant for the IPU3).\n\n> + *\n> + * \\var Accumulator::counted\n> + * \\brief Number of unsaturated cells used to calculate the sums\n> + *\n> + * \\var Accumulator::rSum\n> + * \\brief Sum of the average red values of each cell in the zone\n\ns/each cell/each unsaturated cell/ ? Same below.\n\nReviewed-by: Laurent Pinchart \n\n> + *\n> + * \\var Accumulator::gSum\n> + * \\brief Sum of the average green values of each cell in the zone\n> + *\n> + * \\var Accumulator::bSum\n> + * \\brief Sum of the average blue values of each cell in the zone\n> */\n> \n> /**\n> @@ -157,7 +201,7 @@ uint32_t Awb::estimateCCT(double red, double green, double blue)\n> \treturn 449 * n * n * n + 3525 * n * n + 6823.3 * n + 5520.33;\n> }\n> \n> -/* Generate an RGB vector with the average values for each region */\n> +/* Generate an RGB vector with the average values for each zone */\n> void Awb::generateZones(std::vector &zones)\n> {\n> \tfor (unsigned int i = 0; i < kAwbStatsSizeX * kAwbStatsSizeY; i++) {\n> @@ -174,7 +218,7 @@ void Awb::generateZones(std::vector &zones)\n> \t}\n> }\n> \n> -/* Translate the IPU3 statistics into the default statistics region array */\n> +/* Translate the IPU3 statistics into the default statistics zone array */\n> void Awb::generateAwbStats(const ipu3_uapi_stats_3a *stats,\n> \t\t\t const ipu3_uapi_grid_config &grid)\n> {\n> @@ -215,7 +259,6 @@ void Awb::clearAwbStats()\n> \t\tawbStats_[i].rSum = 0;\n> \t\tawbStats_[i].gSum = 0;\n> \t\tawbStats_[i].counted = 0;\n> -\t\tawbStats_[i].uncounted = 0;\n> \t}\n> }\n> \n> @@ -304,7 +347,7 @@ void Awb::prepare(IPAContext &context, ipu3_uapi_params *params)\n> \n> \t/*\n> \t * Optical center is column start (respectively row start) of the\n> -\t * region of interest minus its X center (respectively Y center).\n> +\t * cell of interest minus its X center (respectively Y center).\n> \t *\n> \t * For the moment use BDS as a first approximation, but it should\n> \t * be calculated based on Shading (SHD) parameters.\n> diff --git a/src/ipa/ipu3/algorithms/awb.h b/src/ipa/ipu3/algorithms/awb.h\n> index cc848060..ac8ccc84 100644\n> --- a/src/ipa/ipu3/algorithms/awb.h\n> +++ b/src/ipa/ipu3/algorithms/awb.h\n> @@ -33,9 +33,8 @@ struct Ipu3AwbCell {\n> \tunsigned char padding[3];\n> } __attribute__((packed));\n> \n> -struct IspStatsRegion {\n> +struct Accumulator {\n> \tunsigned int counted;\n> -\tunsigned int uncounted;\n> \tunsigned long long rSum;\n> \tunsigned long long gSum;\n> \tunsigned long long bSum;\n> @@ -82,7 +81,7 @@ private:\n> \tuint32_t estimateCCT(double red, double green, double blue);\n> \n> \tstd::vector zones_;\n> -\tIspStatsRegion awbStats_[kAwbStatsSizeX * kAwbStatsSizeY];\n> +\tAccumulator awbStats_[kAwbStatsSizeX * kAwbStatsSizeY];\n> \tAwbStatus asyncResults_;\n> };\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 71FACBDC71\n\tfor ;\n\tFri, 10 Sep 2021 15:18:58 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D4D9F69170;\n\tFri, 10 Sep 2021 17:18:57 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id D531B69169\n\tfor ;\n\tFri, 10 Sep 2021 17:18:55 +0200 (CEST)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 3A080DD;\n\tFri, 10 Sep 2021 17:18:55 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"ErwrHu0N\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631287135;\n\tbh=FS4fMEvdvhgn1jBSKV7LRVG/4Oeqsor5Ktl/1yWynSA=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=ErwrHu0NQr+zvLDpl3BMSAID11hPj7+U0xqgdBSfSevGlnsp3rT+O4VIFtJ2F6DgQ\n\tfRU8VjBZHfvyqHv8hGhPu7iSB4SykHo49A7l2ll5iMyzh6g2rzteCL3On+EvkmBKi0\n\tQCK6vJyxfXRgbbj/XiJ9nqP4elL7CdmT/9eI0FQg=","Date":"Fri, 10 Sep 2021 18:18:33 +0300","From":"Laurent Pinchart ","To":"Jean-Michel Hautbois ","Message-ID":"","References":"<20210909135449.68017-1-jeanmichel.hautbois@ideasonboard.com>\n\t<20210909135449.68017-3-jeanmichel.hautbois@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20210909135449.68017-3-jeanmichel.hautbois@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH v6 2/5] ipa: ipu3: Rename\n\tIspStatsRegion to Accumulator","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@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":19638,"web_url":"https://patchwork.libcamera.org/comment/19638/","msgid":"<672b7233-0dd1-9574-4868-9fa8be2c9776@ideasonboard.com>","date":"2021-09-13T13:11:31","subject":"Re: [libcamera-devel] [PATCH v6 2/5] ipa: ipu3: Rename\n\tIspStatsRegion to Accumulator","submitter":{"id":75,"url":"https://patchwork.libcamera.org/api/people/75/","name":"Jean-Michel Hautbois","email":"jeanmichel.hautbois@ideasonboard.com"},"content":"Hi Laurent,\n\nOn 10/09/2021 17:18, Laurent Pinchart wrote:\n> Hi Jean-Michel,\n> \n> Thank you for the patch.\n> \n> On Thu, Sep 09, 2021 at 03:54:46PM +0200, Jean-Michel Hautbois wrote:\n>> The IspStatsRegion structure was introduced as an attempt to prepare for\n>> a generic AWB algorithm structure. The structure name by itself is not\n>> explicit and it is too optimistic to try and make a generic one for now.\n>>\n>> Its role is to accumulate the pixels in a given zone. Rename it to\n>> accumulator, and remove the uncounted field at the same time. It is\n>> always possible to know how many pixels are not relevant for the\n>> algorithm by calculating total-counted. The uncounted field was only\n>> declared and not used. Amend the documentation accordingly.\n>>\n>> Signed-off-by: Jean-Michel Hautbois \n>> Reviewed-by: Kieran Bingham \n>> ---\n>> src/ipa/ipu3/algorithms/awb.cpp | 79 +++++++++++++++++++++++++--------\n>> src/ipa/ipu3/algorithms/awb.h | 5 +--\n>> 2 files changed, 63 insertions(+), 21 deletions(-)\n>>\n>> diff --git a/src/ipa/ipu3/algorithms/awb.cpp b/src/ipa/ipu3/algorithms/awb.cpp\n>> index e05647c9..9b22c783 100644\n>> --- a/src/ipa/ipu3/algorithms/awb.cpp\n>> +++ b/src/ipa/ipu3/algorithms/awb.cpp\n>> @@ -21,26 +21,70 @@ static constexpr uint32_t kMinZonesCounted = 16;\n>> static constexpr uint32_t kMinGreenLevelInZone = 32;\n>> \n>> /**\n>> - * \\struct IspStatsRegion\n>> - * \\brief RGB statistics for a given region\n>> + * \\struct Accumulator\n>> + * \\brief RGB statistics for a given zone\n>> *\n>> - * The IspStatsRegion structure is intended to abstract the ISP specific\n>> - * statistics and use an agnostic algorithm to compute AWB.\n>> + * - Cells are defined in Pixels\n>> + * - Zones are defined in Cells\n>> + * - The total zones may overlap the image width and height to meet hardware\n>> + * constraints\n> \n> What do you mean by \"overlap the image width\" here ?\n> \n\nIt means that the zones will be adapted to the BDS output size which may\nbe greater than the rendered frame (asking for 1280x720 may lead to a\n1296x720 BDS output and you will have 81 cells of 16 pixels width and\nnot 80)\n\n>> *\n>> - * \\var IspStatsRegion::counted\n>> - * \\brief Number of pixels used to calculate the sums\n>> + * 81 cells\n>> + * /───────────── 1280 pixels ───────────\\\n>> + * 16 zones\n>> + * 16\n>> + * ┌────┬────┬────┬────┬────┬─ ──────┬────┐ \\\n>> + * │Cell│ │ │ │ │ | │ │ │\n>> + *16 │ px │ │ │ │ │ | │ │ │\n>> + * ├────┼────┼────┼────┼────┼─ ──────┼────┤ │\n>> + * │ │ │ │ │ │ | │ │\n>> + * │ │ │ │ │ │ | │ │ 7\n>> + * │ ── │ ── │ ── │ ── │ ── │ ── ── ─┤ ── │ 1 2 4\n>> + * │ │ │ │ │ │ | │ │ 2 0 5\n>> *\n>> - * \\var IspStatsRegion::uncounted\n>> - * \\brief Remaining number of pixels in the region\n>> + * │ │ │ │ │ │ | │ │ z p c\n>> + * ├────┼────┼────┼────┼────┼─ ──────┼────┤ o i e\n>> + * │ │ │ │ │ │ | │ │ n x l\n>> + * │ │ | │ │ e e l\n>> + * ├─── ───┼─ ──────┼────┤ s l s\n>> + * │ │ | │ │ s\n>> + * │ │ | │ │\n>> + * ├─── Zone of Cells ───┼─ ──────┼────┤ │\n>> + * │ (5 x 4) │ | │ │ │\n>> + * │ │ | │ │ │\n>> + * ├── ───┼─ ──────┼────┤ │\n>> + * │ │ │ | │ │ │\n>> + * │ │ │ │ │ │ | │ │ │\n>> + * └────┴────┴────┴────┴────┴─ ──────┴────┘ /\n> \n> Nice diagram :-) I wonder how it will render with doxygen, but that's\n> for later.\n> \n> Could you add one more space on the left side to have a space between\n> \"*\" and \"16\" ?\n> \n>> *\n>> - * \\var IspStatsRegion::rSum\n>> - * \\brief Sum of the red values in the region\n>> + * The algorithm works with a fixed number of zones \\a kAwbStatsSizeX x\n>> + * \\a kAwbStatsSizeY. For example, a frame of 1280x720 is divided into 81x45\n>> + * cells of [16x16] pixels with a BDS output size of 1296x720 to account for the\n>> + * 16x16 pixel alignment restrictions. In the case of \\a kAwbStatsSizeX=16 and\n>> + * \\a kAwbStatsSizeY=12 the zones are made of [5x4] cells. The cells are\n>> + * left-aligned and calculated by IPAIPU3::calculateBdsGrid().\n>> *\n>> - * \\var IspStatsRegion::gSum\n>> - * \\brief Sum of the green values in the region\n>> + * Each statistics cell represents the average value of the pixels in that cell\n>> + * split by colour components.\n> \n> This should probably be moved somewhere else, it explains how the\n> Accumulator structure is used in the IPU3 IPA implementation, not what\n> the Accumulator is. We can deal with that later (but not too late if\n> possible, maybe at the time the structure moves out of awb.h as it\n> shouldn't refer to kAwbStatsSizeX in that case).\n> \n\nI will add a \\todo for that :-)\n\n>> *\n>> - * \\var IspStatsRegion::bSum\n>> - * \\brief Sum of the blue values in the region\n>> + * The Accumulator structure stores the sum of the average of each cell in a\n>> + * zone of the image, as well as the number of cells which were unsaturated and\n>> + * therefore included in the average.\n>> + *\n>> + * Cells which are saturated beyond the threshold defined in\n>> + * ipu3_uapi_awb_config_s are not included in the average.\n> \n> This will need to be reworked too when we'll move the structure to a\n> shared header, to avoid referring to ipu3_uapi_awb_config_s and just\n> talk about thresholds in general (the reference to\n> ipu3_uapi_awb_config_s should then be kept in awb.cpp though, as it's\n> relevant for the IPU3).\n\ndito\n\n> \n>> + *\n>> + * \\var Accumulator::counted\n>> + * \\brief Number of unsaturated cells used to calculate the sums\n>> + *\n>> + * \\var Accumulator::rSum\n>> + * \\brief Sum of the average red values of each cell in the zone\n> \n> s/each cell/each unsaturated cell/ ? Same below.\n> \n> Reviewed-by: Laurent Pinchart \n> \n>> + *\n>> + * \\var Accumulator::gSum\n>> + * \\brief Sum of the average green values of each cell in the zone\n>> + *\n>> + * \\var Accumulator::bSum\n>> + * \\brief Sum of the average blue values of each cell in the zone\n>> */\n>> \n>> /**\n>> @@ -157,7 +201,7 @@ uint32_t Awb::estimateCCT(double red, double green, double blue)\n>> \treturn 449 * n * n * n + 3525 * n * n + 6823.3 * n + 5520.33;\n>> }\n>> \n>> -/* Generate an RGB vector with the average values for each region */\n>> +/* Generate an RGB vector with the average values for each zone */\n>> void Awb::generateZones(std::vector &zones)\n>> {\n>> \tfor (unsigned int i = 0; i < kAwbStatsSizeX * kAwbStatsSizeY; i++) {\n>> @@ -174,7 +218,7 @@ void Awb::generateZones(std::vector &zones)\n>> \t}\n>> }\n>> \n>> -/* Translate the IPU3 statistics into the default statistics region array */\n>> +/* Translate the IPU3 statistics into the default statistics zone array */\n>> void Awb::generateAwbStats(const ipu3_uapi_stats_3a *stats,\n>> \t\t\t const ipu3_uapi_grid_config &grid)\n>> {\n>> @@ -215,7 +259,6 @@ void Awb::clearAwbStats()\n>> \t\tawbStats_[i].rSum = 0;\n>> \t\tawbStats_[i].gSum = 0;\n>> \t\tawbStats_[i].counted = 0;\n>> -\t\tawbStats_[i].uncounted = 0;\n>> \t}\n>> }\n>> \n>> @@ -304,7 +347,7 @@ void Awb::prepare(IPAContext &context, ipu3_uapi_params *params)\n>> \n>> \t/*\n>> \t * Optical center is column start (respectively row start) of the\n>> -\t * region of interest minus its X center (respectively Y center).\n>> +\t * cell of interest minus its X center (respectively Y center).\n>> \t *\n>> \t * For the moment use BDS as a first approximation, but it should\n>> \t * be calculated based on Shading (SHD) parameters.\n>> diff --git a/src/ipa/ipu3/algorithms/awb.h b/src/ipa/ipu3/algorithms/awb.h\n>> index cc848060..ac8ccc84 100644\n>> --- a/src/ipa/ipu3/algorithms/awb.h\n>> +++ b/src/ipa/ipu3/algorithms/awb.h\n>> @@ -33,9 +33,8 @@ struct Ipu3AwbCell {\n>> \tunsigned char padding[3];\n>> } __attribute__((packed));\n>> \n>> -struct IspStatsRegion {\n>> +struct Accumulator {\n>> \tunsigned int counted;\n>> -\tunsigned int uncounted;\n>> \tunsigned long long rSum;\n>> \tunsigned long long gSum;\n>> \tunsigned long long bSum;\n>> @@ -82,7 +81,7 @@ private:\n>> \tuint32_t estimateCCT(double red, double green, double blue);\n>> \n>> \tstd::vector zones_;\n>> -\tIspStatsRegion awbStats_[kAwbStatsSizeX * kAwbStatsSizeY];\n>> +\tAccumulator awbStats_[kAwbStatsSizeX * kAwbStatsSizeY];\n>> \tAwbStatus asyncResults_;\n>> };\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 C494DBDB1D\n\tfor ;\n\tMon, 13 Sep 2021 13:11:36 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 21B316024C;\n\tMon, 13 Sep 2021 15:11:36 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 2CA916024C\n\tfor ;\n\tMon, 13 Sep 2021 15:11:34 +0200 (CEST)","from tatooine.ideasonboard.com (unknown\n\t[IPv6:2a01:e0a:169:7140:edc5:688b:2ede:8b4b])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 6EAA7499;\n\tMon, 13 Sep 2021 15:11:33 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"noCaIX9W\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1631538693;\n\tbh=81KrLMzq9Q29+Zhxvnn1EN8M5Z0hakQthdWVJzSx8EI=;\n\th=Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=noCaIX9WbMjQ3wcmpLl8ud8e5dt/NJFXSYywN959fgdXR07FfLhRNJoNnd7S/6trY\n\tvty719yptfPHO/76LvDXzkdHgDJkjr7vOmyHog4NKf5rFUeY1mCY6ACeYYTQUW9pgB\n\tRH0KgN/RRbCLsaku+kS2WbiRBqlldfHi7p7viUk4=","To":"Laurent Pinchart ","References":"<20210909135449.68017-1-jeanmichel.hautbois@ideasonboard.com>\n\t<20210909135449.68017-3-jeanmichel.hautbois@ideasonboard.com>\n\t","From":"Jean-Michel Hautbois ","Message-ID":"<672b7233-0dd1-9574-4868-9fa8be2c9776@ideasonboard.com>","Date":"Mon, 13 Sep 2021 15:11:31 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101\n\tThunderbird/78.13.0","MIME-Version":"1.0","In-Reply-To":"","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-US","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH v6 2/5] ipa: ipu3: Rename\n\tIspStatsRegion to Accumulator","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@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]