[{"id":38529,"web_url":"https://patchwork.libcamera.org/comment/38529/","msgid":"","date":"2026-04-08T08:36:32","subject":"Re: [PATCH 08/13] ipa: libipa: Awb: Provide common context storage","submitter":{"id":143,"url":"https://patchwork.libcamera.org/api/people/143/","name":"Jacopo Mondi","email":"jacopo.mondi@ideasonboard.com"},"content":"Hi Kieran\n\nOn Tue, Apr 07, 2026 at 11:01:11PM +0100, Kieran Bingham wrote:\n> Expose a common structure for storing Awb Session configuration, Active\n> state and Frame Context data.\n>\n> Signed-off-by: Kieran Bingham \n> ---\n> src/ipa/libipa/awb.cpp | 47 ++++++++++++++++++++++++++++++++++++++++++++\n> src/ipa/libipa/awb.h | 26 ++++++++++++++++++++++++\n> src/ipa/rkisp1/ipa_context.h | 28 ++++++++------------------\n> 3 files changed, 81 insertions(+), 20 deletions(-)\n>\n> diff --git a/src/ipa/libipa/awb.cpp b/src/ipa/libipa/awb.cpp\n> index 214bac8b56a6ca7b0ac9954f0aa742cd613e6141..f215bea0b59483eadf95572f073928a4eb1275f4 100644\n> --- a/src/ipa/libipa/awb.cpp\n> +++ b/src/ipa/libipa/awb.cpp\n> @@ -22,6 +22,53 @@ LOG_DEFINE_CATEGORY(Awb)\n>\n> namespace ipa {\n>\n> +/**\n> + * \\struct awb::Session\n> + * \\brief Session-wide AWB configuration\n> + *\n> + * \\var awb::Session::enabled\n> + * \\brief True when AWB processing is enabled for the session\n\nThis is easy to confuse with \"auto awb\" enabled.\n\nI would say\n \\brief True when AWB support is enabled\n\nI wonder if this a \"per-session\" parameter or a global one (it depends\non the config file content, doesn't it?). But this is already like\nthis, we can change it on top if needed\n\n> + */\n> +\n> +/**\n> + * \\struct awb::ActiveState\n> + * \\brief Active AWB state shared across frames\n\nI would drop \"Active\" and use something like \"most recent\" ?\n\n> + *\n> + * \\var awb::ActiveState::manual\n> + * \\brief The most recent manually requested AWB state\n> + *\n> + * \\var awb::ActiveState::automatic\n> + * \\brief The most recent automatically calculated AWB state\n\nAnd drop most recent from these two\n\n> + *\n> + * \\var awb::ActiveState::autoEnabled\n> + * \\brief True when automatic AWB is currently selected\n> + */\n> +\n> +/**\n> + * \\struct awb::ActiveState::AwbState\n> + * \\brief AWB gains and colour temperature for one operating mode\n\nand drop \"for one operating mode\"\n\n> + *\n> + * \\var awb::ActiveState::AwbState::gains\n> + * \\brief The white balance gains for this AWB state\n> + *\n> + * \\var awb::ActiveState::AwbState::temperatureK\n> + * \\brief The colour temperature for this AWB state, in Kelvin\n\nand \"for this AWB state\"\n\n> + */\n> +\n> +/**\n> + * \\struct awb::FrameContext\n> + * \\brief Per-frame AWB state applied to a captured frame\n\nEither \"Per-frame\" or \"applied to a captured frame\". Don't they mean\nthe same thing ?\n\n> + *\n> + * \\var awb::FrameContext::gains\n> + * \\brief The white balance gains applied to the frame\n> + *\n> + * \\var awb::FrameContext::autoEnabled\n> + * \\brief True when the frame uses automatic AWB\n\nTrue when the gains have been computed by the AWB algorithm ?\n\n> + *\n> + * \\var awb::FrameContext::temperatureK\n> + * \\brief The colour temperature used for the frame, in Kelvin\n\nAnd drop \"to the frame\"/\"for the frame\" ?\n\n> + */\n> +\n> /**\n> * \\class AwbResult\n> * \\brief The result of an AWB calculation\n> diff --git a/src/ipa/libipa/awb.h b/src/ipa/libipa/awb.h\n> index f4a86038635f984ac03b1e466c0fcd4e6d5e22bd..764be849270bcd42ecdf67aea3d13afa170c7499 100644\n> --- a/src/ipa/libipa/awb.h\n> +++ b/src/ipa/libipa/awb.h\n> @@ -20,6 +20,32 @@ namespace libcamera {\n>\n> namespace ipa {\n>\n> +namespace awb {\n> +\n> +struct Session {\n> +\tbool enabled;\n> +};\n> +\n> +struct ActiveState {\n> +\tstruct AwbState {\n> +\t\tRGB gains;\n> +\t\tunsigned int temperatureK;\n> +\t};\n> +\n> +\tAwbState manual;\n> +\tAwbState automatic;\n> +\n> +\tbool autoEnabled;\n> +};\n> +\n> +struct FrameContext {\n> +\tRGB gains;\n> +\tbool autoEnabled;\n> +\tunsigned int temperatureK;\n> +};\n> +\n> +} /* namespace awb */\n> +\n> struct AwbResult {\n> \tRGB gains;\n> \tdouble colourTemperature;\n> diff --git a/src/ipa/rkisp1/ipa_context.h b/src/ipa/rkisp1/ipa_context.h\n> index 63a6f2b7d884bfff2553175fa32e45ae747fb4c7..85a8ea3acc1fd75ff6b49576800ab7615cebce2c 100644\n> --- a/src/ipa/rkisp1/ipa_context.h\n> +++ b/src/ipa/rkisp1/ipa_context.h\n> @@ -25,6 +25,7 @@\n> #include \"libcamera/internal/vector.h\"\n>\n> #include \"libipa/agc_mean_luminance.h\"\n> +#include \"libipa/awb.h\"\n> #include \"libipa/camera_sensor_helper.h\"\n> #include \"libipa/fc_queue.h\"\n> #include \"libipa/fixedpoint.h\"\n> @@ -49,15 +50,16 @@ struct IPAHwSettings {\n> \tbool compand;\n> };\n>\n> +struct RKISP1AwbSession : public ipa::awb::Session {\n> +\tstruct rkisp1_cif_isp_window measureWindow;\n> +};\n> +\n> struct IPASessionConfiguration {\n> \tstruct {\n> \t\tstruct rkisp1_cif_isp_window measureWindow;\n> \t} agc;\n>\n> -\tstruct {\n> -\t\tstruct rkisp1_cif_isp_window measureWindow;\n> -\t\tbool enabled;\n> -\t} awb;\n> +\tstruct RKISP1AwbSession awb;\n>\n> \tstruct {\n> \t\tbool supported;\n> @@ -103,17 +105,7 @@ struct IPAActiveState {\n> \t\tutils::Duration maxFrameDuration;\n> \t} agc;\n>\n> -\tstruct {\n> -\t\tstruct AwbState {\n> -\t\t\tRGB gains;\n> -\t\t\tunsigned int temperatureK;\n> -\t\t};\n> -\n> -\t\tAwbState manual;\n> -\t\tAwbState automatic;\n> -\n> -\t\tbool autoEnabled;\n> -\t} awb;\n> +\tipa::awb::ActiveState awb;\n>\n> \tstruct {\n> \t\tMatrix manual;\n> @@ -175,11 +167,7 @@ struct IPAFrameContext : public FrameContext {\n> \t\tbool autoGainModeChange;\n> \t} agc;\n>\n> -\tstruct {\n> -\t\tRGB gains;\n> -\t\tbool autoEnabled;\n> -\t\tunsigned int temperatureK;\n> -\t} awb;\n> +\tipa::awb::FrameContext awb;\n\nAll minors or suggestions\n\nReviewed-by: Jacopo Mondi \n\nThanks\n j\n\n>\n> \tstruct {\n> \t\tBrightnessQ brightness;\n>\n> --\n> 2.53.0\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 C4BB0BEFBE\n\tfor ;\n\tWed, 8 Apr 2026 08:36:38 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 90F8962DC1;\n\tWed, 8 Apr 2026 10:36:37 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id D3C5562DA6\n\tfor ;\n\tWed, 8 Apr 2026 10:36:35 +0200 (CEST)","from ideasonboard.com (mob-109-113-47-41.net.vodafone.it\n\t[109.113.47.41])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 9CF121544;\n\tWed, 8 Apr 2026 10:35:07 +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=\"snKGG5xE\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1775637307;\n\tbh=la2BRSGfuSgrOKA3NeEhyqrLirBeray0HMeTURUW7+w=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=snKGG5xE0dliKRNPLJmrklwEdsk7LLQkmIe02bEAeBUqumEb36TAdpDLpSAr4Vouu\n\tvnHb57pVyoA1817AZhtLw8+vxIH2Kja8wP8B50q5M6jHQZlOKmukW5sXcurl8Vhh2P\n\tfhL+RDJPSroclNIlkuvHs1sBkmy97KHob0PDswZQ=","Date":"Wed, 8 Apr 2026 10:36:32 +0200","From":"Jacopo Mondi ","To":"Kieran Bingham ","Cc":"libcamera-devel@lists.libcamera.org","Subject":"Re: [PATCH 08/13] ipa: libipa: Awb: Provide common context storage","Message-ID":"","References":"<20260407-kbingham-awb-split-v1-0-a39af3f4dc20@ideasonboard.com>\n\t<20260407-kbingham-awb-split-v1-8-a39af3f4dc20@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<20260407-kbingham-awb-split-v1-8-a39af3f4dc20@ideasonboard.com>","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":38786,"web_url":"https://patchwork.libcamera.org/comment/38786/","msgid":"<1c222221-e6cf-4702-bc72-e32df3e31b3f@oss.qualcomm.com>","date":"2026-05-07T14:35:41","subject":"Re: [PATCH 08/13] ipa: libipa: Awb: Provide common context storage","submitter":{"id":242,"url":"https://patchwork.libcamera.org/api/people/242/","name":"Hans de Goede","email":"johannes.goede@oss.qualcomm.com"},"content":"Hi,\n\nOn 8-Apr-26 00:01, Kieran Bingham wrote:\n> Expose a common structure for storing Awb Session configuration, Active\n> state and Frame Context data.\n> \n> Signed-off-by: Kieran Bingham \n\nThanks, patch looks good to me:\n\nReviewed-by: Hans de Goede \n\nRegards,\n\nHans\n\n\n> ---\n> src/ipa/libipa/awb.cpp | 47 ++++++++++++++++++++++++++++++++++++++++++++\n> src/ipa/libipa/awb.h | 26 ++++++++++++++++++++++++\n> src/ipa/rkisp1/ipa_context.h | 28 ++++++++------------------\n> 3 files changed, 81 insertions(+), 20 deletions(-)\n> \n> diff --git a/src/ipa/libipa/awb.cpp b/src/ipa/libipa/awb.cpp\n> index 214bac8b56a6ca7b0ac9954f0aa742cd613e6141..f215bea0b59483eadf95572f073928a4eb1275f4 100644\n> --- a/src/ipa/libipa/awb.cpp\n> +++ b/src/ipa/libipa/awb.cpp\n> @@ -22,6 +22,53 @@ LOG_DEFINE_CATEGORY(Awb)\n> \n> namespace ipa {\n> \n> +/**\n> + * \\struct awb::Session\n> + * \\brief Session-wide AWB configuration\n> + *\n> + * \\var awb::Session::enabled\n> + * \\brief True when AWB processing is enabled for the session\n> + */\n> +\n> +/**\n> + * \\struct awb::ActiveState\n> + * \\brief Active AWB state shared across frames\n> + *\n> + * \\var awb::ActiveState::manual\n> + * \\brief The most recent manually requested AWB state\n> + *\n> + * \\var awb::ActiveState::automatic\n> + * \\brief The most recent automatically calculated AWB state\n> + *\n> + * \\var awb::ActiveState::autoEnabled\n> + * \\brief True when automatic AWB is currently selected\n> + */\n> +\n> +/**\n> + * \\struct awb::ActiveState::AwbState\n> + * \\brief AWB gains and colour temperature for one operating mode\n> + *\n> + * \\var awb::ActiveState::AwbState::gains\n> + * \\brief The white balance gains for this AWB state\n> + *\n> + * \\var awb::ActiveState::AwbState::temperatureK\n> + * \\brief The colour temperature for this AWB state, in Kelvin\n> + */\n> +\n> +/**\n> + * \\struct awb::FrameContext\n> + * \\brief Per-frame AWB state applied to a captured frame\n> + *\n> + * \\var awb::FrameContext::gains\n> + * \\brief The white balance gains applied to the frame\n> + *\n> + * \\var awb::FrameContext::autoEnabled\n> + * \\brief True when the frame uses automatic AWB\n> + *\n> + * \\var awb::FrameContext::temperatureK\n> + * \\brief The colour temperature used for the frame, in Kelvin\n> + */\n> +\n> /**\n> * \\class AwbResult\n> * \\brief The result of an AWB calculation\n> diff --git a/src/ipa/libipa/awb.h b/src/ipa/libipa/awb.h\n> index f4a86038635f984ac03b1e466c0fcd4e6d5e22bd..764be849270bcd42ecdf67aea3d13afa170c7499 100644\n> --- a/src/ipa/libipa/awb.h\n> +++ b/src/ipa/libipa/awb.h\n> @@ -20,6 +20,32 @@ namespace libcamera {\n> \n> namespace ipa {\n> \n> +namespace awb {\n> +\n> +struct Session {\n> +\tbool enabled;\n> +};\n> +\n> +struct ActiveState {\n> +\tstruct AwbState {\n> +\t\tRGB gains;\n> +\t\tunsigned int temperatureK;\n> +\t};\n> +\n> +\tAwbState manual;\n> +\tAwbState automatic;\n> +\n> +\tbool autoEnabled;\n> +};\n> +\n> +struct FrameContext {\n> +\tRGB gains;\n> +\tbool autoEnabled;\n> +\tunsigned int temperatureK;\n> +};\n> +\n> +} /* namespace awb */\n> +\n> struct AwbResult {\n> \tRGB gains;\n> \tdouble colourTemperature;\n> diff --git a/src/ipa/rkisp1/ipa_context.h b/src/ipa/rkisp1/ipa_context.h\n> index 63a6f2b7d884bfff2553175fa32e45ae747fb4c7..85a8ea3acc1fd75ff6b49576800ab7615cebce2c 100644\n> --- a/src/ipa/rkisp1/ipa_context.h\n> +++ b/src/ipa/rkisp1/ipa_context.h\n> @@ -25,6 +25,7 @@\n> #include \"libcamera/internal/vector.h\"\n> \n> #include \"libipa/agc_mean_luminance.h\"\n> +#include \"libipa/awb.h\"\n> #include \"libipa/camera_sensor_helper.h\"\n> #include \"libipa/fc_queue.h\"\n> #include \"libipa/fixedpoint.h\"\n> @@ -49,15 +50,16 @@ struct IPAHwSettings {\n> \tbool compand;\n> };\n> \n> +struct RKISP1AwbSession : public ipa::awb::Session {\n> +\tstruct rkisp1_cif_isp_window measureWindow;\n> +};\n> +\n> struct IPASessionConfiguration {\n> \tstruct {\n> \t\tstruct rkisp1_cif_isp_window measureWindow;\n> \t} agc;\n> \n> -\tstruct {\n> -\t\tstruct rkisp1_cif_isp_window measureWindow;\n> -\t\tbool enabled;\n> -\t} awb;\n> +\tstruct RKISP1AwbSession awb;\n> \n> \tstruct {\n> \t\tbool supported;\n> @@ -103,17 +105,7 @@ struct IPAActiveState {\n> \t\tutils::Duration maxFrameDuration;\n> \t} agc;\n> \n> -\tstruct {\n> -\t\tstruct AwbState {\n> -\t\t\tRGB gains;\n> -\t\t\tunsigned int temperatureK;\n> -\t\t};\n> -\n> -\t\tAwbState manual;\n> -\t\tAwbState automatic;\n> -\n> -\t\tbool autoEnabled;\n> -\t} awb;\n> +\tipa::awb::ActiveState awb;\n> \n> \tstruct {\n> \t\tMatrix manual;\n> @@ -175,11 +167,7 @@ struct IPAFrameContext : public FrameContext {\n> \t\tbool autoGainModeChange;\n> \t} agc;\n> \n> -\tstruct {\n> -\t\tRGB gains;\n> -\t\tbool autoEnabled;\n> -\t\tunsigned int temperatureK;\n> -\t} awb;\n> +\tipa::awb::FrameContext awb;\n> \n> \tstruct {\n> \t\tBrightnessQ brightness;\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 C2279BDCB5\n\tfor ;\n\tThu, 7 May 2026 14:35:48 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B72966302D;\n\tThu, 7 May 2026 16:35:47 +0200 (CEST)","from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com\n\t[205.220.180.131])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id EDD3962FEC\n\tfor ;\n\tThu, 7 May 2026 16:35:45 +0200 (CEST)","from pps.filterd (m0279869.ppops.net [127.0.0.1])\n\tby mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id\n\t647BDigR3157534 for ;\n\tThu, 7 May 2026 14:35:44 GMT","from mail-vs1-f72.google.com (mail-vs1-f72.google.com\n\t[209.85.217.72])\n\tby mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4e0mhf21jp-1\n\t(version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT)\n\tfor ;\n\tThu, 07 May 2026 14:35:44 +0000 (GMT)","by mail-vs1-f72.google.com with SMTP id\n\tada2fe7eead31-62e66a6ee35so649985137.3\n\tfor ;\n\tThu, 07 May 2026 07:35:44 -0700 (PDT)","from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec?\n\t(2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl.\n\t[2001:1c00:c32:7800:5bfa:a036:83f0:f9ec])\n\tby smtp.gmail.com with ESMTPSA id\n\ta640c23a62f3a-bc81d5e95bbsm89580666b.19.2026.05.07.07.35.42\n\t(version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n\tThu, 07 May 2026 07:35:42 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=qualcomm.com header.i=@qualcomm.com\n\theader.b=\"YaezA4FH\"; dkim=pass (2048-bit key;\n\tunprotected) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com\n\theader.b=\"FAPThLuz\"; dkim-atps=neutral","DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h=\n\tcontent-transfer-encoding:content-type:date:from:in-reply-to\n\t:message-id:mime-version:references:subject:to; s=qcppdkim1; bh=\n\tST1CEAtWzDRasW80f/uET7xk1TrYBLsoaoq1/nosjAM=; b=YaezA4FHaIP0XrBN\n\tlV3oALAIQzHzoc68Zxg5wp0IQnd2xZ530JsCXHgjmJtT+B9ZGUnNoytMGQfCR4OM\n\tb/zFiZKWgkUXUUqgcKtaeJ+xzP235UhWBBfsLcvtuI3z6o/ko/RHKyv3qRctl92x\n\t2jOEPYPnP0uYzUJyuZNGo8nVNuOTp5yYduNlfzRD5Z7jzUEaJcUPVDcRq92ZRWfq\n\toAmX+MueDEOhUqk3AZpRzqJ9iNGufrhXjF5OxX5G50KbkbBBqYDcmDIwfZvgpp4/\n\tvi5t+S75Y/kUZnlbpZCeT3puEmaP/gGUuADvSKCoJyP+PGqAmtca0G8tRPmUBZsp\n\tQEE9zQ==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=oss.qualcomm.com; s=google; t=1778164544; x=1778769344;\n\tdarn=lists.libcamera.org; \n\th=content-transfer-encoding:in-reply-to:content-language:references\n\t:to:subject:from:user-agent:mime-version:date:message-id:from:to:cc\n\t:subject:date:message-id:reply-to;\n\tbh=ST1CEAtWzDRasW80f/uET7xk1TrYBLsoaoq1/nosjAM=;\n\tb=FAPThLuzto8csJ9vf1/13C9GQEnHXDalrI9aQUk5RqJSPkv4zIu8YgS2D6D1FiZ9EJ\n\tK1VckQJft2xLjG6DpZbzqCFj0r2FznXOkX1CwrDGQfTncni3a5o897vTeR1FJ494aYG/\n\tgLLpds9ejqMjFfeJQCoA63/E+ppD829fkNirXvuyWzNdXHCyL0EYnjn5ZStHvOHvr5nm\n\taaH0AoQw0aR0XtmGx/NEAFpYsU0USMTQSCn/rII4lkZaV9t4+Gm0Z/d5+J/pEZmzlPz7\n\tlzhsdLPrcmN34K3fhawQ7zzBOP3idyBmO8/6dsUn/IBOekDjszNIExNGTrdbIc1D3Tcl\n\tdEVQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20251104; t=1778164544; x=1778769344;\n\th=content-transfer-encoding:in-reply-to:content-language:references\n\t:to:subject:from:user-agent:mime-version:date:message-id:x-gm-gg\n\t:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;\n\tbh=ST1CEAtWzDRasW80f/uET7xk1TrYBLsoaoq1/nosjAM=;\n\tb=V/4kvAP8dHLelzVgdjPe608EJQRJOdBVrDuPx8EgFcI+ZKouDmYrUzSl0pVOZ44m4u\n\tCZcJRCTL4M0PTrISoc87BmYySmwRf6DSg1tHVRqOH87Aww79StT33/Uzu5NW++mUtm0y\n\tfApQUMhSLbtTSJUL5J52RPypTwAapqWbL5VRlHEsOGUDNs0V4VpY2f54jJj2K6biZghC\n\tNr/q5b3Hr1O6OM63oj+0u5Mx6oDWcH/sNjCrdUgk9q6VaSxjbLfk3zu83pC63hcu77fu\n\tMGr9uAjc1dTXzrqUX6q6e4ecXydgqJQWEODgD3Q9J5agGipeEZ1HW6/ukAqqRu8/i/RJ\n\tDvoQ==","X-Forwarded-Encrypted":"i=1;\n\tAFNElJ/K3iRepHfj/7aCvCJfeaQsXetlRP9a3SvgbUqVgp1ZkjR1HR/nVeS1pbjUdIiH18EPyNNiAdP5xsE9HLW/F8A=@lists.libcamera.org","X-Gm-Message-State":"AOJu0Yz4da3YJgAKzw4jcIvU4NNv9eYzUXX2yet3n4ErNo7qlmlJkRRE\n\tHo2sEi8qOI5lop9FRNMZTOJi28ydHma3zV22th+MbUNjfgcJbU0fA6zRtVtTzs5/ALN7c8/ag3x\n\t/X5wnlkOIZbzQldIlSIp/ETZJE8Vy/t0o7tBXFsl9DnzNW7K1LtkNXrEyyzvGu8M75OBqW+Kb83\n\t3MNh81K64qdvQ=","X-Gm-Gg":"AeBDieuEueapWqP+69k3D1z/9UWrwkWLeJqg2+6OzWcgO/+fg0XT0DwtHDQv+eEVB4t\n\tXIfHUoBBT4+kF9aG4emHkF8Rk45qnt/HyYUisM616Hp9cs49UzY4YE08S5HRuDpnhieX9F3AY0g\n\txPDN1x5TybKMszNfDSFNwuQMnNnD3gLCvjbNngA/G2dzE3ahZndbrcZf4cEXGhaCZ6ra8LIQbLJ\n\tGwlbHhomCDoUwj3XfdUpkVlat/gSnmWTnaQ2hExTcyr4jUlivpXt4Ihgkuu9oLpGskRKu+Q3YEV\n\tsmGtNhiR1VJFO3THeTf6j/tg32ibbuOrAWs59sawCdb5HOTu7qh2gw0XSJkQfmfv6w8CSaZJKEQ\n\tIDGyGrqoUN1pgbXgMCGDqp0vRoq83I65MMaTXIQczMQPYR2EePPC966ORhqfIXNW/AvMjLUzXO1\n\tu6azgHLAwT9cnf+wqveL8kzs9WHOOxCTIz1uRx7XbnHzyzyJ1oUC24jpd9cPw73EhDwbP7/EP1n\n\tSY/On9S8BEDOlwaiTHEttz4S09hRbyj22j0LA==","X-Received":["by 2002:a05:6122:1822:b0:56e:f876:5626 with SMTP id\n\t71dfb90a1353d-5755952cc68mr4220198e0c.5.1778164543833; \n\tThu, 07 May 2026 07:35:43 -0700 (PDT)","by 2002:a05:6122:1822:b0:56e:f876:5626 with SMTP id\n\t71dfb90a1353d-5755952cc68mr4220177e0c.5.1778164543394; \n\tThu, 07 May 2026 07:35:43 -0700 (PDT)"],"Message-ID":"<1c222221-e6cf-4702-bc72-e32df3e31b3f@oss.qualcomm.com>","Date":"Thu, 7 May 2026 16:35:41 +0200","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","From":"johannes.goede@oss.qualcomm.com","Subject":"Re: [PATCH 08/13] ipa: libipa: Awb: Provide common context storage","To":"Kieran Bingham ,\n\tlibcamera-devel@lists.libcamera.org","References":"<20260407-kbingham-awb-split-v1-0-a39af3f4dc20@ideasonboard.com>\n\t<20260407-kbingham-awb-split-v1-8-a39af3f4dc20@ideasonboard.com>","Content-Language":"en-US, nl","In-Reply-To":"<20260407-kbingham-awb-split-v1-8-a39af3f4dc20@ideasonboard.com>","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"7bit","X-Proofpoint-Spam-Details-Enc":"AW1haW4tMjYwNTA3MDE0NSBTYWx0ZWRfX1XKtZ1dvUs7w\n\tkDrp/mfVVOZkIaWKcXTpkx6J/3S6vIg5HzUque4YZXT/dM/plZ64O8xFg82Q7NDLWO1y8RY8QpS\n\tp/M1jArQps0YBqcYDV7nQcAXjdmKmhIb5nbwgvGMt34U781YfZcHcItPKwV/BR+R/DhfgV8HGMS\n\tiCfJWHzoafcclfRZ+qPuOUI6BAChxAUcSVU2KdL3eFUdC7Ih/a8oyKaXtzreNv5J1FSf+agzrKF\n\tQbaaDfFqzz5M0U+rCcsEf09u1cPE08UFEmUjVorQNq98DLw7inaEAFDfB6w8/D3YzLuLO1W+MOT\n\tUEFDNR2a9rYjlXG6cYwzbUoFmfCQz7avA44P7CMtDYfZeJxhLUL2UXNhFFre+x+PNJIX+PrIkI6\n\tsRvxycIhXAc1dPne8+piWd1L97TOgNeYglGv3Fmm5NwBJ5jaNbVpPK/mj6jCwpg7uwxSXm9QQmw\n\thYTN14Kzp4LbCavhAeA==","X-Proofpoint-ORIG-GUID":"27LEk575e9UHwjP2UFaGbVn25Z5b412x","X-Authority-Analysis":"v=2.4 cv=SuagLvO0 c=1 sm=1 tr=0 ts=69fca340 cx=c_pps\n\ta=DUEm7b3gzWu7BqY5nP7+9g==:117 a=xqWC_Br6kY4A:10 a=IkcTkHD0fZMA:10\n\ta=NGcC8JguVDcA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22\n\ta=u7WPNUs3qKkmUXheDGA7:22 a=_glEPmIy2e8OvE2BGh3C:22 a=P1BnusSwAAAA:8\n\ta=EUspDBNiAAAA:8 a=F1_QMZE_It5tP_vStZIA:9 a=QEXdDO2ut3YA:10\n\ta=-aSRE8QhW-JAV6biHavz:22 a=D0XLA9XvdZm18NrgonBM:22","X-Proofpoint-GUID":"27LEk575e9UHwjP2UFaGbVn25Z5b412x","X-Proofpoint-Virus-Version":"vendor=baseguard\n\tengine=ICAP:2.0.293, Aquarius:18.0.1143, Hydra:6.1.51,\n\tFMLib:17.12.100.49\n\tdefinitions=2026-05-07_01,2026-05-06_01,2025-10-01_01","X-Proofpoint-Spam-Details":"rule=outbound_notspam policy=outbound score=0\n\tclxscore=1015 malwarescore=0 suspectscore=0 bulkscore=0\n\tlowpriorityscore=0\n\timpostorscore=0 spamscore=0 adultscore=0 phishscore=0\n\tpriorityscore=1501\n\tclassifier=typeunknown authscore=0 authtc= authcc= route=outbound\n\tadjust=0\n\treason=mlx scancount=1 engine=8.22.0-2604200000\n\tdefinitions=main-2605070145","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]