Patch Detail
Show a patch.
GET /api/patches/15443/?format=api
{ "id": 15443, "url": "https://patchwork.libcamera.org/api/patches/15443/?format=api", "web_url": "https://patchwork.libcamera.org/patch/15443/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20220311133705.511853-1-umang.jain@ideasonboard.com>", "date": "2022-03-11T13:37:05", "name": "[libcamera-devel,[PATCH,v4.1,4/4] ipa: ipu3: awb: Clamp gain values", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "2c424ac6372d8f60101797b738a0ceb65908bcf2", "submitter": { "id": 86, "url": "https://patchwork.libcamera.org/api/people/86/?format=api", "name": "Umang Jain", "email": "umang.jain@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/15443/mbox/", "series": [ { "id": 2959, "url": "https://patchwork.libcamera.org/api/series/2959/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=2959", "date": "2022-03-11T13:19:29", "name": "[libcamera-devel] ipa: ipu3: awb: Clamp gain values", "version": 1, "mbox": "https://patchwork.libcamera.org/series/2959/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/15443/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/15443/checks/", "tags": {}, "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "X-Original-To": "parsemail@patchwork.libcamera.org", "Delivered-To": "parsemail@patchwork.libcamera.org", "Received": [ "from lancelot.ideasonboard.com (lancelot.ideasonboard.com\r\n\t[92.243.16.209])\r\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 70851BF415\r\n\tfor <parsemail@patchwork.libcamera.org>;\r\n\tFri, 11 Mar 2022 13:37:21 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\r\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D36A2632E2;\r\n\tFri, 11 Mar 2022 14:37:20 +0100 (CET)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\r\n\t[213.167.242.64])\r\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 58799604E8\r\n\tfor <libcamera-devel@lists.libcamera.org>;\r\n\tFri, 11 Mar 2022 14:37:19 +0100 (CET)", "from perceval.ideasonboard.com (unknown [103.251.226.65])\r\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 6759F482;\r\n\tFri, 11 Mar 2022 14:37:17 +0100 (CET)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\r\n\ts=mail; t=1647005840;\r\n\tbh=l+5xh2l8LOEdfvnBv2rdbsky+XwfMeyyWARuHaYlYOY=;\r\n\th=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe:\r\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\r\n\tFrom;\r\n\tb=n1cAMj4nvfRU+YZdXJ4IdCrC5y1Bzgot+Lpb4OlGoZJmv8mH2hbA7qGAFEZeafi5d\r\n\t0zuAwp1rVu7q4yHko84XKhdKaMq7st6usbAkkiG6XNG1csMg2/cqSFvogL88nTZJCg\r\n\tqChwhCnSUQs6140FYrFy4SNROTgv/HoUqWG4zF3HSWqXv/S+ARtSyIWg9krhxaJUWv\r\n\twkP2u39hXprWPCx5Iyw02aCyra5IYcKRukOrWv+qtjU3gUskC9RRVN0A1Qv1uB9f2n\r\n\taVexY4bu5cdTRqJfBo4pdHmT2yHbqEWJBu9mP/yhdDqIvUvh8vZtJG1Vb/0g17roJI\r\n\tXAUagf/oNoWLw==", "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\r\n\ts=mail; t=1647005838;\r\n\tbh=l+5xh2l8LOEdfvnBv2rdbsky+XwfMeyyWARuHaYlYOY=;\r\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\r\n\tb=kHbqggkfstEsygTQrQlLAtPl1PxQq7w3On4Adv+H1B6/idwR76gD942p4jlyiL9XF\r\n\t1tp2OSTkirtivMqZmNu6SGmZz8+3L8pckILVGvNshAXGJzZmWyiHnwSWIrRb7ze6kp\r\n\tVik/vI9ze8F/dmaBmlC1/l+kwdS/EVdMjv63UFPc=" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key; \r\n\tunprotected) header.d=ideasonboard.com\r\n\theader.i=@ideasonboard.com\r\n\theader.b=\"kHbqggkf\"; dkim-atps=neutral", "To": "libcamera-devel@lists.libcamera.org", "Date": "Fri, 11 Mar 2022 19:07:05 +0530", "Message-Id": "<20220311133705.511853-1-umang.jain@ideasonboard.com>", "X-Mailer": "git-send-email 2.31.1", "In-Reply-To": "<20220224151113.109858-5-jeanmichel.hautbois@ideasonboard.com>", "References": "<20220224151113.109858-5-jeanmichel.hautbois@ideasonboard.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [[PATCH v4.1 4/4] ipa: ipu3: awb: Clamp gain\r\n\tvalues", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\r\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\r\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "From": "Umang Jain via libcamera-devel <libcamera-devel@lists.libcamera.org>", "Reply-To": "Umang Jain <umang.jain@ideasonboard.com>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "From: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com>\n\nThe gain values are coded as u3.13 fixed point values, ie they can not\nbe more than 8. Clamp the values in order to avoid any off limits value\nwhich could make the IPU3 behave in a weird manner.\n\nSigned-off-by: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com>\nReviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\nSigned-off-by: Umang Jain <umang.jain@ideasonboard.com>\nReviewed-by: Umang Jain <umang.jain@ideasonboard.com>\n---\n src/ipa/ipu3/algorithms/awb.cpp | 8 ++++++++\n 1 file changed, 8 insertions(+)", "diff": "diff --git a/src/ipa/ipu3/algorithms/awb.cpp b/src/ipa/ipu3/algorithms/awb.cpp\r\nindex 1dc27fc9..87a6cc7a 100644\r\n--- a/src/ipa/ipu3/algorithms/awb.cpp\r\n+++ b/src/ipa/ipu3/algorithms/awb.cpp\r\n@@ -353,6 +353,14 @@ void Awb::awbGreyWorld()\r\n \r\n \t/* Color temperature is not relevant in Grey world but still useful to estimate it :-) */\r\n \tasyncResults_.temperatureK = estimateCCT(sumRed.R, sumRed.G, sumBlue.B);\r\n+\r\n+\t/*\r\n+\t * Gain values are unsigned integer value ranging [0, 8) with 13 bit\r\n+\t * fractional part.\r\n+\t */\r\n+\tredGain = std::clamp(redGain, 0.0, 65535.0 / 8192);\r\n+\tblueGain = std::clamp(blueGain, 0.0, 65535.0 / 8192);\r\n+\r\n \tasyncResults_.redGain = redGain;\r\n \t/* Hardcode the green gain to 1.0. */\r\n \tasyncResults_.greenGain = 1.0;", "prefixes": [ "libcamera-devel", "[PATCH", "v4.1", "4/4" ] }