{"id":14600,"url":"https://patchwork.libcamera.org/api/1.1/patches/14600/?format=json","web_url":"https://patchwork.libcamera.org/patch/14600/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20211113084947.21892-11-jeanmichel.hautbois@ideasonboard.com>","date":"2021-11-13T08:49:43","name":"[libcamera-devel,v5,10/14] ipa: ipu3: awb: Add support for color temperature","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"3134f68a04c9112b9d04b81e4ff3a6a24a4a8ce6","submitter":{"id":75,"url":"https://patchwork.libcamera.org/api/1.1/people/75/?format=json","name":"Jean-Michel Hautbois","email":"jeanmichel.hautbois@ideasonboard.com"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/14600/mbox/","series":[{"id":2717,"url":"https://patchwork.libcamera.org/api/1.1/series/2717/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=2717","date":"2021-11-13T08:49:33","name":"IPA: IPU3: Introduce per-frame controls","version":5,"mbox":"https://patchwork.libcamera.org/series/2717/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/14600/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/14600/checks/","tags":{},"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 E4697C3251\n\tfor ;\n\tSat, 13 Nov 2021 08:50:02 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 76C496044F;\n\tSat, 13 Nov 2021 09:50:02 +0100 (CET)","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 D353D6036B\n\tfor ;\n\tSat, 13 Nov 2021 09:49:52 +0100 (CET)","from tatooine.ideasonboard.com (unknown\n\t[IPv6:2a01:e0a:169:7140:4d35:9445:6881:4ffd])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 8D1ADE0C;\n\tSat, 13 Nov 2021 09:49:52 +0100 (CET)"],"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=\"rEJPvtS3\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1636793392;\n\tbh=cSMjPTzkHWd0gpDn7FUtm4fLB+5H5w+vMhEXeUBk4U4=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=rEJPvtS3e/ACOFbqfzPqwJTRjf74Oxxu4JvT7YGuYOyot1wltqNHB2U9rSrNylq9Y\n\tXOshL6NjsrENF5IXJaoF0dhKczrcjqb5iEKJlSxo9R71yMTwXDLehpEaapNnBD1ZqF\n\tAkcA2HbXojhWU0/eRS7/0w7sunP00QNZG082oz68=","From":"Jean-Michel Hautbois ","To":"libcamera-devel@lists.libcamera.org","Date":"Sat, 13 Nov 2021 09:49:43 +0100","Message-Id":"<20211113084947.21892-11-jeanmichel.hautbois@ideasonboard.com>","X-Mailer":"git-send-email 2.32.0","In-Reply-To":"<20211113084947.21892-1-jeanmichel.hautbois@ideasonboard.com>","References":"<20211113084947.21892-1-jeanmichel.hautbois@ideasonboard.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v5 10/14] ipa: ipu3: awb: Add support for\n\tcolor temperature","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\" "},"content":"The AWB estimates the color temperature, but it is not used at all. It\ncan be useful for debug purpose at least, but also for lux estimation\nlater, to be able to know the temperature estimated for a given frame.\n\nAdd a new member to the IPAFrameContext::awb for this purpose, and\nupdate the value in AWB.\n\nSigned-off-by: Jean-Michel Hautbois \nReviewed-by: Kieran Bingham \nReviewed-by: Umang Jain \nReviewed-by: Paul Elder \n---\n src/ipa/ipu3/algorithms/awb.cpp | 1 +\n src/ipa/ipu3/ipa_context.cpp | 3 +++\n src/ipa/ipu3/ipa_context.h | 2 ++\n 3 files changed, 6 insertions(+)","diff":"diff --git a/src/ipa/ipu3/algorithms/awb.cpp b/src/ipa/ipu3/algorithms/awb.cpp\nindex a4114659..c7bcb20e 100644\n--- a/src/ipa/ipu3/algorithms/awb.cpp\n+++ b/src/ipa/ipu3/algorithms/awb.cpp\n@@ -385,6 +385,7 @@ void Awb::process(IPAContext &context, const ipu3_uapi_stats_3a *stats)\n \tcontext.frameContext.awb.gains.blue = asyncResults_.blueGain;\n \tcontext.frameContext.awb.gains.green = asyncResults_.greenGain;\n \tcontext.frameContext.awb.gains.red = asyncResults_.redGain;\n+\tcontext.frameContext.awb.temperatureK = asyncResults_.temperatureK;\n }\n \n constexpr uint16_t Awb::threshold(float value)\ndiff --git a/src/ipa/ipu3/ipa_context.cpp b/src/ipa/ipu3/ipa_context.cpp\nindex a7ff957d..99caf9ad 100644\n--- a/src/ipa/ipu3/ipa_context.cpp\n+++ b/src/ipa/ipu3/ipa_context.cpp\n@@ -117,6 +117,9 @@ namespace libcamera::ipa::ipu3 {\n *\n * \\var IPAFrameContext::awb.gains.blue\n * \\brief White balance gain for B channel\n+ *\n+ * \\var IPAFrameContext::awb.temperatureK\n+ * \\brief Estimated color temperature\n */\n \n /**\ndiff --git a/src/ipa/ipu3/ipa_context.h b/src/ipa/ipu3/ipa_context.h\nindex a5a19800..fd97e240 100644\n--- a/src/ipa/ipu3/ipa_context.h\n+++ b/src/ipa/ipu3/ipa_context.h\n@@ -45,6 +45,8 @@ struct IPAFrameContext {\n \t\t\tdouble green;\n \t\t\tdouble blue;\n \t\t} gains;\n+\n+\t\tdouble temperatureK;\n \t} awb;\n \n \tstruct {\n","prefixes":["libcamera-devel","v5","10/14"]}