Patch Detail
Show a patch.
GET /api/patches/14687/?format=api
{ "id": 14687, "url": "https://patchwork.libcamera.org/api/patches/14687/?format=api", "web_url": "https://patchwork.libcamera.org/patch/14687/", "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": "<20211123091451.67404-4-jeanmichel.hautbois@ideasonboard.com>", "date": "2021-11-23T09:14:43", "name": "[libcamera-devel,v2,03/11] ipa: Do not modify the sensor limits", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "c6d60a8994957f240a5431fe0244c8e8b5c8cc63", "submitter": { "id": 75, "url": "https://patchwork.libcamera.org/api/people/75/?format=api", "name": "Jean-Michel Hautbois", "email": "jeanmichel.hautbois@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/14687/mbox/", "series": [ { "id": 2741, "url": "https://patchwork.libcamera.org/api/series/2741/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=2741", "date": "2021-11-23T09:14:40", "name": "Introduce AGC for RkISP1", "version": 2, "mbox": "https://patchwork.libcamera.org/series/2741/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/14687/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/14687/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\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id C7B65BDB13\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 23 Nov 2021 09:15:05 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 543B560442;\n\tTue, 23 Nov 2021 10:15:03 +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 7040660376\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 23 Nov 2021 10:14:57 +0100 (CET)", "from tatooine.ideasonboard.com (unknown\n\t[IPv6:2a01:e0a:169:7140:3c3b:9149:b:8aa9])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 2DD04A1B;\n\tTue, 23 Nov 2021 10:14:57 +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=\"H0JonW71\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1637658897;\n\tbh=cDAwKOK7nZf5p1eAa/44NWbZMgG4DKa6W2900WGBzKM=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=H0JonW71WTKsfFlEjPNOBEVXn3wjH1DkIt/VJ4K+er7NoJiqZd/hRKIUAlJLOwjEu\n\tE3dntpqHZdFe/E/ISme7RhAkel4CRgUH//Ocs1w8UdHoTJMIX3O7coLzzljuAxIsGf\n\tu5O1dUMi2SKDBKM0L2gyRuDafEQyijQ3sXQGNc2U=", "From": "Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Tue, 23 Nov 2021 10:14:43 +0100", "Message-Id": "<20211123091451.67404-4-jeanmichel.hautbois@ideasonboard.com>", "X-Mailer": "git-send-email 2.32.0", "In-Reply-To": "<20211123091451.67404-1-jeanmichel.hautbois@ideasonboard.com>", "References": "<20211123091451.67404-1-jeanmichel.hautbois@ideasonboard.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v2 03/11] ipa: Do not modify the sensor\n\tlimits", "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>,\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>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "The driver is responsible of setting the proper limits for its controls.\nFor instance, imx219 has an analogue gain of 1.0 when the gain code is\nset to 0. The minimum analogue is forced to be at least 1, which for\nimx219 sets it to 1.00329. Rework this for both IPU3 and RkISP1.\n\nSigned-off-by: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com>\n---\n src/ipa/ipu3/ipu3.cpp | 4 ++--\n src/ipa/rkisp1/rkisp1.cpp | 4 ++--\n 2 files changed, 4 insertions(+), 4 deletions(-)", "diff": "diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp\nindex a8d54a5d..b0c75541 100644\n--- a/src/ipa/ipu3/ipu3.cpp\n+++ b/src/ipa/ipu3/ipu3.cpp\n@@ -444,11 +444,11 @@ int IPAIPU3::configure(const IPAConfigInfo &configInfo,\n \t\treturn -EINVAL;\n \t}\n \n-\tminExposure_ = std::max(itExp->second.min().get<int32_t>(), 1);\n+\tminExposure_ = itExp->second.min().get<int32_t>();\n \tmaxExposure_ = itExp->second.max().get<int32_t>();\n \texposure_ = minExposure_;\n \n-\tminGain_ = std::max(itGain->second.min().get<int32_t>(), 1);\n+\tminGain_ = itGain->second.min().get<int32_t>();\n \tmaxGain_ = itGain->second.max().get<int32_t>();\n \tgain_ = minGain_;\n \ndiff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp\nindex 7ecbf8ae..910ad952 100644\n--- a/src/ipa/rkisp1/rkisp1.cpp\n+++ b/src/ipa/rkisp1/rkisp1.cpp\n@@ -139,11 +139,11 @@ int IPARkISP1::configure([[maybe_unused]] const IPACameraSensorInfo &info,\n \n \tautoExposure_ = true;\n \n-\tminExposure_ = std::max<uint32_t>(itExp->second.min().get<int32_t>(), 1);\n+\tminExposure_ = itExp->second.min().get<int32_t>();\n \tmaxExposure_ = itExp->second.max().get<int32_t>();\n \texposure_ = minExposure_;\n \n-\tminGain_ = std::max<uint32_t>(itGain->second.min().get<int32_t>(), 1);\n+\tminGain_ = itGain->second.min().get<int32_t>();\n \tmaxGain_ = itGain->second.max().get<int32_t>();\n \tgain_ = minGain_;\n \n", "prefixes": [ "libcamera-devel", "v2", "03/11" ] }